numaproj / numalogic

Collection of operational time series ML models and tools
https://numalogic.numaproj.io/
Apache License 2.0
166 stars 28 forks source link

fix: registry test_case #197

Closed s0nicboOm closed 1 year ago

s0nicboOm commented 1 year ago

Test cases for is_model_stale had freeze time set to fixed date. Now, when redis is fetching data that is older than 10 days (cause ttl is set to 10 days), it automatically deletes this key and hence we were seeing ModelKeyNotFound error.

codecov[bot] commented 1 year ago

Codecov Report

Merging #197 (3ace968) into main (efa1df7) will increase coverage by 0.19%. The diff coverage is n/a.

@@            Coverage Diff             @@
##             main     #197      +/-   ##
==========================================
+ Coverage   96.62%   96.82%   +0.19%     
==========================================
  Files          36       36              
  Lines        1541     1541              
  Branches      119      119              
==========================================
+ Hits         1489     1492       +3     
+ Misses         36       34       -2     
+ Partials       16       15       -1     

see 1 file with indirect coverage changes

:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more