Skip to content

Commit c6695db

Browse files
Matovidloclaude
andcommitted
test(e2e): fix expected stdout for pull+rename orphaned scheduler test
Remove "Ignoring invalid local state." — the orphaned scheduler "456" is removed by SetRecords at manifest-load time (stderr warning only), so no localErr is produced and the stdout message is not printed. Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
1 parent 7773cea commit c6695db

2 files changed

Lines changed: 5 additions & 1 deletion

File tree

internal/pkg/plan/rename/plan_test.go

Lines changed: 5 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -156,6 +156,11 @@ func TestRenameAllPlan_StatelessParentSkipped(t *testing.T) {
156156
plan, err := NewPlan(projectState.State())
157157
require.NoError(t, err)
158158

159+
// The fixture produces 3 legitimate rename actions (branch, mysql config, mysql row).
160+
// Asserting the count ensures the plan is not vacuously empty — an empty plan would
161+
// make the loop below a no-op and the NotEqual assertions would never fire.
162+
require.Len(t, plan.actions, 3, "plan must contain the fixture's legitimate rename actions")
163+
159164
for _, action := range plan.actions {
160165
assert.NotEqual(t, action.Manifest.Key(), orchKey)
161166
assert.NotEqual(t, action.Manifest.Key(), schedulerKey)

test/cli/pull/pull-force-orphaned-scheduler-with-rename/expected-stdout

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,4 +1,3 @@
1-
Ignoring invalid local state.
21
Plan for "pull" operation:
32
* B main | changed: description
43
* C main/extractor/ex-generic-v2/wrong-name | changed: description, name

0 commit comments

Comments
 (0)