Integration test for sources w/o auth #649 #651
Conversation
There was a problem hiding this comment.
@dogancanbakir the tests are failing for some reason. can you please fix them?
|
I've refactored the test to make detecting problems easier (will commit it later). The point is that based on that I've noticed that there are 3 or 4 providers that are not working currenlty. One has anti-automation (captcha) and 2 or 3 other services are not accessible anymore (gateway error and temporarily unavailabe). |
|
@Ice3man543 As @forgedhallpass pointed out, some of the providers are not working. And we should see them failing; seeing integration tests are failing means one thing, providers are not working as expected. That will make us investigate them further. Additionally, we should consider separating integration tests from the current pipeline, and maybe running them as a cron job might be a good idea. |
|
@forgedhallpass we should create issues regarding failing providers and ideally remove them since querying them is a waste of time. |
That's was the plan, but only for tomorrow :) |
This PR adds integration tests for sources w/o auth