db: deflake TestOpenRecovery

jbowens · jbowens · commit 83c41dc965a4 · 2025-10-17T11:38:21.000-04:00
The file numbers are non-deterministic. Fix #5439. Fix #5440. Fix #5441. Fix #5442.
diff --git a/open_test.go b/open_test.go
@@ -325,7 +325,7 @@ func TestOpenRecovery(t *testing.T) {
 				return err.Error()
 			}
 			waitForCompactionsAndTableStats(d)
-			return runLSMCmd(td, d)
+			return "ok"
 		default:
 			return fmt.Sprintf("unrecognized command %q", td.Cmd)
 		}
diff --git a/testdata/open_recovery b/testdata/open_recovery
@@ -35,7 +35,11 @@ set e foo
 
 reopen auto-compactions=on inject-errors=((RandomLatency "5ms" 1 (And OpFileWrite (PathMatch "*.sst"))))
 ----
-L5:
-  000012:[b#3,SET-e#10,SET]
-L6:
-  000006:[c#0,SET-d#0,SET]
+ok
+
+# Ensure closing and re-opening again succeeds (and the previous reopen didn't
+# accidentally delete a necessary sstable).
+
+reopen
+----
+ok

Original file line number	Diff line number	Diff line change
`@@ -325,7 +325,7 @@ func TestOpenRecovery(t *testing.T) {`
`325`	`325`	`return err.Error()`
`326`	`326`	`}`
`327`	`327`	`waitForCompactionsAndTableStats(d)`
`328`		`- return runLSMCmd(td, d)`
	`328`	`+ return "ok"`
`329`	`329`	`default:`
`330`	`330`	`return fmt.Sprintf("unrecognized command %q", td.Cmd)`
`331`	`331`	`}`