简体中文界面。帖子、指南及政策正文保留原文;未对原文作自动翻译。

A delayed housing result completes whichever attempt is open

LinBell0618 · 2025年10月1日 08:32 UTC

已关闭
LI
LinBell0618
Our offline UR5e fixture-check test accepts an old completion against a new attempt. I can provoke it with sleeps, but not reliably enough for a regression test. I want to control event order directly and inspect the state after each delivery, including duplicates. The reference housing is unchanged; the attempt is not.

17 条回复

AD
AdaAbbott0042

Use an explicit event queue in the test and release each event when the test says so. Can you start the second attempt while keeping the first completion held back?

17
LI
LinBell0618

Yes. The harness can intercept the callback. I will replace the sleep with a held event and an explicit release step.

12
MA
MayaAli0214

Would you also check the displayed accepted total at each step? I cannot advise on the callback, but a correct internal match could still leave the planning screen counting the same housing twice.

24
TH
TheoAllen0323

Maya, are we counting accepted housings or completed attempts there? On our sheet those got called the same thing and the totals looked wonderful until somebody counted the actual pieces.

-8
MA
MayaAli0214

I meant physical housings with an accepted disposition. The attempt history should remain visible separately, since retries can be useful information without becoming extra output.

18
LI
LinBell0618

The first deterministic sequence now fails consistently: start attempt A, hold its completion, start B, release A. B becomes complete. I have added a state assertion at that point rather than checking only the eventual total. Maya, the current harness does not render the report, so your count check needs a separate test using the same saved history. I will keep those as two checks rather than let a correct total conceal the wrong attempt state.

12
CA
CalebBrown0928

Could the test output name those steps plainly? I mean the report a maintainer reads after failure, not changing the actual attempt identifiers.

17
AD
AdaAbbott0042

Yes, and include both expected and actual attempt states. A failure saying B completed after A arrived is much easier to hand over than a bare assertion number.

19
TH
TheoAllen0323

Lin, does A carry its own attempt identity in the event, or does your harness know it only because it held that callback? That changes what the real code can match later.

23
LI
LinBell0618

The captured event contains the attempt reference. Our matcher currently ignores it and uses the active job label. The harness is not inventing a field absent from the captured reply.

10
AD
AdaAbbott0042

That gives the developer something specific to fix. Test a duplicate A after B has genuinely completed as well, so the corrected matcher cannot quietly reopen or alter B.

13
MA
MayaAli0214

And that saved history can feed the report test. Two deliveries of A's completion should not create two accepted housings, even though the delivery history records both messages.

5
CA
CalebBrown0928

What does the maintainer see for an event with no usable attempt reference? Leaving it unmatched needs an explanation too.

22
LI
LinBell0618

Unknown-reference events currently land in a generic error list. I have added a case for them and asked for a clearer unmatched-event entry. The known-reference matcher repair passes the held-A sequence and duplicate-A-after-B sequence offline.

19
TH
TheoAllen0323

Does closing and reopening the report rerun any completion handling? Ours once counted correctly until you reopened it to show somebody the result. Very efficient way to manufacture output without touching a part.

10
AD
AdaAbbott0042

That's worth a regression too, Theo. Lin, restore the saved state before delivering the duplicates in another case; an in-memory seen set can hide what breaks after restart.

18
LI
LinBell0618

Queued those persistence cases. The current pass covers event ordering in memory, not recovery from disk. Thank you Theo; reopening the report was outside my original test boundary.

20

讨论已关闭

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