Why does our offline inspection screen pass everything?

AaronAllen0262 · 2026年2月25日 12:12 UTC

已关闭

主帖中文翻译暂不可用,以下显示英文原文;回复保留原文。 查看英文原文

AA
AaronAllen0262
Our offline stand-in returns true, so the inspection screen stays green even when I give it no coupon result at all; with the UR5e bench unavailable, I want to replace that cheerful nonsense with saved application responses, but I don't know which examples would expose a bad parser instead of merely keeping the demonstration running

16 条回复

OL
OliverChen1146

Have you got a saved failure response as well as a success, because I would start by making the screen tell those apart before collecting a museum of edge cases

22分
AA
AaronAllen0262

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

2分
MI
MinaBennett0772

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

10分
CA
CallumChen1143

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?

0分
FA
FarahChan1113

Keep an example with no result too, Aaron, since that was the case that first bothered you.

20分
AA
AaronAllen0262

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

18分
OL
OliverChen1146

Expensive way to ask whether a box contains paper, that

10分
MI
MinaBennett0772

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

11分
AA
AaronAllen0262

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

17分
CA
CallumChen1143

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

19分
FA
FarahChan1113

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

0分
CA
CallumChen1143

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

0分
AA
AaronAllen0262

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

8分
MI
MinaBennett0772

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

21分
AA
AaronAllen0262

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

7分
OL
OliverChen1146

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

3分

讨论已关闭

此讨论已连续六个月没有活动,现已关闭新回复。 最后活动: .