pytroll / pytroll-collectors

Collector modules for Pytroll
GNU General Public License v3.0
3 stars 18 forks source link

Fix calling s3.ls() to get also new files for the daemon version #135

Closed pnuu closed 1 year ago

pnuu commented 1 year ago

The s3.ls() call was missing refresh=True kwarg that is required by the daemon version of the S3Stalker. Without it, the call results are taken from cache and none of the new files are returned.

coveralls commented 1 year ago

Pull Request Test Coverage Report for Build 4797161966


Totals Coverage Status
Change from base Build 4699890722: 0.0%
Covered Lines: 0
Relevant Lines: 0

💛 - Coveralls
codecov[bot] commented 1 year ago

Codecov Report

Merging #135 (ffc7179) into main (e3b9abc) will increase coverage by 0.01%. The diff coverage is 100.00%.

@@            Coverage Diff             @@
##             main     #135      +/-   ##
==========================================
+ Coverage   91.32%   91.33%   +0.01%     
==========================================
  Files          27       27              
  Lines        4044     4049       +5     
==========================================
+ Hits         3693     3698       +5     
  Misses        351      351              
Flag Coverage Δ
unittests 91.33% <100.00%> (+0.01%) :arrow_up:

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files Coverage Δ
pytroll_collectors/s3stalker.py 100.00% <100.00%> (ø)
pytroll_collectors/tests/test_s3stalker.py 99.47% <100.00%> (+0.01%) :arrow_up:

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.