Why does our offline inspection screen pass everything?

Yes, an accepted coupon and a rejected coupon from the same application version; the programmer says the stand-in bypasses the parsing code, which explains a lot

Then test through the parser. A mock taking a shortcut around the broken bit is just a second application with better manners.

Our packing demo once showed the result of the previous carton, so does your rejected response actually change the displayed coupon identity as well as the colour?

The saved reject exposed it: our wrapper tests whether the response has contents, then treats that as acceptance, without reading the result field; the reject response has contents too, obviously

Expensive way to ask whether a box contains paper, that

What does the corrected code do with an unrecognised result value? Don't let an else branch quietly become another green lamp.

It reports an unreadable result and leaves acceptance unset; the programmer added a test with an unrecognised value as well as the captured accept and reject, but has not checked the display identity yet

Please do the identity check, Aaron, especially if a coupon result can arrive while another coupon is already selected on screen.

Callum, would you include reopening the screen with no new response, so an old colour cannot sneak back in from saved display state?

Yes, that is close to what caught us, though ours happened when changing cartons rather than reopening the screen.

We added reject_after_accept, empty_response and reopen_without_result to the application tests, plus a response for the wrong coupon; the first three now pass, but the wrong-coupon case revealed that the heading changes before the old result is removed

Good find. Does the export have the same old-result problem?

The export already used the response's coupon identity correctly; we fixed the display reset and the wrong-coupon test now leaves the selected coupon pending, with the other response listed against its own identity, and all five saved-response cases pass through the normal parser and screen code

That's a useful bit of work while the bench is unavailable, and considerably more convincing than true with a cheerful face