MetPX / sr_insects

test cases for sarracenia python and c implementations.
GNU General Public License v2.0
0 stars 0 forks source link

fixes 3/4 thing on startup #36

Closed petersilva closed 7 months ago

petersilva commented 7 months ago

When you run the flow test... you see this:

 .
 . 
 .
FAILED, expected 21, but there are 18 queues existing after 1st audit
 . 
 .
 .

OK, as expected 16 exchanges for flow test created
 .
 .
 .

OK: sr start was successful
Overall dynamic_flow: FAILED 3/4 passed.

I changed ti to expect 18 queues, so we get this instead:

   .
   .
   .
OK, as expected 18 queues existing after 1st audit
   .
   .
   .
OK, as expected 16 exchanges for flow test created
  .
  . 

OK: sr start was successful
Overall dynamic_flow: PASSED 4/4 checks passed!

so people don't think it failed. I didn't investigate why the number changed, but it's been changed for a year or so, and when I look at the broker at the end of the test, it's still 18, so it looks right. Somebody could to an extensive review to understand whether 18 is truly correct. I think it is, but there are more critical things to work on for now.