Need more tests! about goinsta HOT 20 CLOSED

That's right.
We need more test methods.
Coveralls.io result is low.

from goinsta.

I'm very busy. Can you write test methods ?

from goinsta.

@ahmdrz After my big commit, I'll take a look.

from goinsta.

I'll start writing tests :)

from goinsta.

Thank you

from goinsta.

I wrote a few tests, they started to fall, it's good.

from goinsta.

Did you test again after changing response structs ?

from goinsta.

Run 3 tests(Travis CI) at the same time, bad, the API gives errors due to frequent requests.

from goinsta.

@ahmdrz
Yes, you do not see that these are different commits, I did it at the same time.
Errors in tests because of frequent requests. Could you restart the test in Travis?

from goinsta.

@ahmdrz

from goinsta.

goinsta_test.go:181: Invalid status code {"message": "User not found", "status": "fail"}

from goinsta.

@ahmdrz in this this test no error "user not found"
https://travis-ci.org/ahmdrz/goinsta/jobs/222286588

from goinsta.

It seems to me that the instagram gave us erroneous data.

from goinsta.

Could you restart the test in Travis?

You have to change user id in line 181...
If I restart travis , we will fail again...

from goinsta.

I think it will be good if we search user by username...

from goinsta.

2 identical tests for Travis, and the result is different.

from goinsta.

@ahmdrz Can you restart the tests separately, not all together?

from goinsta.

No I can't because it is not easy...
I will restart travis...

from goinsta.

One test fail

=== RUN TestSetPublicAndUnPublicAccount
--- FAIL: TestSetPublicAndUnPublicAccount (0.06s)
goinsta_test.go:623: Invalid status code {"message": "Please wait a few minutes before you try again.", "status": "fail"}

from goinsta.

Instagram does not change the status of account 6 times...

from goinsta.