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

old result sneaks through after the identity check

AnilBell0636 · 2025年10月29日 11:37 UTC

已关闭
AN
AnilBell0636
The identity check works at receipt. That's not where this one goes wrong. In our offline UR5e inspection test, a callback passes the check, waits in a queue, then updates whichever attempt is on screen when it finally runs. I've already kept attempt identity separate from the job name. Where would you put the regression test so this doesn't come back under a different disguise?

13 条回复

IS
IsabelBennett0735

At the state change. Pause the old callback after receipt, select a new attempt, then release it. The new attempt must remain unfinished. A receipt test will happily congratulate you while the queued work does the damage.

17
FE
FelixAli0197

Our display once pulled this trick with a stale success banner. Stored result was right, screen was wrong. Are you changing the saved inspection result as well, or only painting the wrong row?

6
AN
AnilBell0636

Both. The callback looks up currentAttempt before saving. The banner is just the visible symptom.

10
DI
DineshBrown0908

Can the queued item carry its original attempt instead of looking up the current one?

2
AN
AnilBell0636

It carries it already. Then ignores it. Irritatingly small line of code.

7
IS
IsabelBennett0735

Anil, keep the test aimed at the saved state even after that line changes. Felix's banner case deserves its own check too; a correct database does not make a misleading screen harmless to the next operator.

15
FE
FelixAli0197

Yes, please don't hide the screen check in a database assertion. We did that and shipped the same confusing banner again. Two failures with names you can understand beats one grand test that says no.

8
OW
OwenBrooks0834

What will the screen say about the late result itself? Unfinished new attempt is one half. Someone still has to understand where the older answer went.

19
DI
DineshBrown0908

Owen, would a history entry be enough? Assuming the old attempt is still identifiable, I mean.

24
OW
OwenBrooks0834

Maybe. Depends whether the person is waiting on it. I would show that example to whoever uses the inspection page before deciding a quiet history entry is enough.

7
AN
AnilBell0636

State regression is now in: old callback parked, new attempt selected, old callback released. It fails on the old code and passes with the callback using its own identity. Separate display check still fails because the banner reads the selected row.

8
AN
AnilBell0636

Owen and Dinesh, the older attempt remains in history. Whether its late answer needs a notice is still undecided. Thanks Isabel for the exact pause point; I was about to write another timing lottery.

12
FE
FelixAli0197

That's the annoying part made repeatable, at least. Leave the banner failure visible until it's fixed. Very easy to call that cosmetic once the storage test turns green.

14

讨论已关闭

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