10 pre-existing test failures in test_mqtt_handler and test_node_sweeper #10

Closed
opened 2026-08-19 22:06:54 -04:00 by logan · 0 comments
Owner

The pytest baseline is 69 passed / 10 failed. The ten failures predate current work and are treated as the accepted baseline when verifying changes.

That means those two modules provide no signal at all, and any genuine regression in them is masked — every future change has to reason about whether a failure is "the usual ten".

Either fix them or delete them. A permanently red test is worse than no test.

The pytest baseline is 69 passed / 10 failed. The ten failures predate current work and are treated as the accepted baseline when verifying changes. That means those two modules provide no signal at all, and any genuine regression in them is masked — every future change has to reason about whether a failure is "the usual ten". Either fix them or delete them. A permanently red test is worse than no test.
logan closed this issue 2026-08-20 03:06:49 -04:00
Sign in to join this conversation.
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: logan/server-26#10