MetPX / sarracenia

https://MetPX.github.io/sarracenia
GNU General Public License v2.0
45 stars 22 forks source link

Flow test passed although one test failed #183

Closed benlapETS closed 5 years ago

benlapETS commented 5 years ago

This is in flow_check.sh and flow_include.sh where calcres is used for the result calculation and it allows a 10% margin which it should not do:

                | C          routing |
test 18 FAILURE: cpump both pelles (c shovel) should receive about the same number of messages (8959) (5078)
test 19 success: cdnld_f21 subscribe downloaded (364) the same number of files that was published by both van_14 and van_15 (364)
test 20 success: veille_f34 should post twice as many files (728) as subscribe cdnld_f21 downloaded (364)
test 21 success: veille_f34 should post twice as many files (728) as subscribe cfile_f44 downloaded (364)
test 22 success: Overall 20 of 21 passed (sample size: 1075) !
benlapETS commented 5 years ago

I forgot that there was an issue on that and fixed it as of #206.