Fixed flaky test

[antonpirker]

The URL www.squirrelchasers.com is actually existing, so we should not access it in our tests. Hope this make the test more stable.

[codecov]

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 79.57%. Comparing base (f6db981) to head (7fb4b42).
Report is 2 commits behind head on master.

✅ All tests successful. No failed tests found.

Additional details and impacted files

@@            Coverage Diff             @@
##           master    #4165      +/-   ##
==========================================
+ Coverage   79.49%   79.57%   +0.07%     
==========================================
  Files         141      141              
  Lines       15732    15732              
  Branches     2675     2675              
==========================================
+ Hits        12506    12518      +12     
+ Misses       2379     2368      -11     
+ Partials      847      846       -1     

see 5 files with indirect coverage changes

[szokeasaurusrex]

Wouldn't a more robust solution be to use a mock server? I feel like tests in general should not be depending on a real network connection, as it introduces to many opportunities for flakiness

[sentrivana]

😄