amadan
e2e: the operator-doors witness asserts the legacy move door's recorded refusal only
The home-region sentence of the legacy door is not a recorded copy row, so the witness no longer pins it.
Co-Authored-By: Claude Fable 5.1 <noreply@anthropic.com>
1 file changed,
+7
−10
internal/e2e/selfserve/selfserve_operator_test.go+7 −10
diff --git a/internal/e2e/selfserve/selfserve_operator_test.go b/internal/e2e/selfserve/selfserve_operator_test.go| index 8a3297d93..7aed1123e 100644 |
| --- a/internal/e2e/selfserve/selfserve_operator_test.go |
| +++ b/internal/e2e/selfserve/selfserve_operator_test.go |
| @@ -93,8 +93,8 @@ func (w *jobWorld) legacyMoveDoor(body string) (int, string) { |
| // one, each door's view after it, and a staged move's status, restage and |
| // start), byte for byte; the operator's reservation is the sentinel "*" |
| // and nothing else; and the move door itself is the legacy door (its own |
| -// refusals), where in self-serve mode it is the member's door, which |
| -// refuses the operator's token with the family's 404. |
| +// refusal of an invalid region), where in self-serve mode it is the |
| +// member's door, which refuses the operator's token with the family's 404. |
| func TestSelfServeOperatorDoorsUnchangedOnTheBridge(t *testing.T) { |
| golden := goldenOperatorLines(t) |
| switchOff := func(t *testing.T) *jobWorld { |
| @@ -131,14 +131,11 @@ func TestSelfServeOperatorDoorsUnchangedOnTheBridge(t *testing.T) { |
| if rec := w.record(); rec.Move != nil || len(w.reservations()) != 0 { |
| t.Fatalf("a refusal wrote: %+v, %v", rec.Move, w.reservations()) |
| } |
| - // The move door: the legacy door's refusals with the switch off. |
| - for body, want := range map[string]string{ |
| - `{"region":"UK!"}`: `400 "invalid region"`, |
| - `{"region":"ie"}`: `400 "moves to the home region are not supported yet — the home publishes no staging lane"`, |
| - } { |
| - if got := operatorLine(w.legacyMoveDoor(body)); got != want { |
| - t.Errorf("the move door with %s: %s, want %s", body, got, want) |
| - } |
| + // The move door: the legacy door's refusal with the switch off (its |
| + // body is not the member's door's: that would refuse the operator's |
| + // token with the 404 below). |
| + if got, want := operatorLine(w.legacyMoveDoor(`{"region":"UK!"}`)), `400 "invalid region"`; got != want { |
| + t.Errorf("the move door: %s, want %s", got, want) |
| } |
| // In self-serve mode the move door is the member's: the operator's |
| // token is not a move-capable token, and gets the 404. |