Repository navigation
Commit d540ccb
committed
🔒 Keep this slice's Deno-only suites out of the Node and Bun corpora
Four suites this branch adds open a real `node:sqlite` WorkflowRun database
through the Deno storage adapter, and none of them had an exclusion entry. Bun
has no `node:sqlite` at all, so it would have loaded them and failed on the
import.
They had entries before #293 was cut into a stack. The manifest is one shared
file and each slice was built by applying a set of paths to a fresh branch, so
it took whichever branch's version was applied last rather than accumulating —
the suites moved and their entries did not.
`runtime-exclusions.test.ts` cannot catch this. It checks that every entry names
a file discovery finds; it has no way to know which suites are Deno-only, so a
missing entry looks exactly like a portable test.
What does catch it is asking what the runner would run: derive the corpus the
way `runtimeTests` does, subtract the exclusions, and walk each survivor's
imports for `node:sqlite`. That reports zero here, and reported these four
before.
Refs #2931 parent 638a92c commit d540ccb
1 file changed
Lines changed: 24 additions & 0 deletions
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
144 | 144 | | |
145 | 145 | | |
146 | 146 | | |
| 147 | + | |
| 148 | + | |
| 149 | + | |
| 150 | + | |
| 151 | + | |
| 152 | + | |
| 153 | + | |
| 154 | + | |
| 155 | + | |
| 156 | + | |
| 157 | + | |
| 158 | + | |
| 159 | + | |
| 160 | + | |
| 161 | + | |
| 162 | + | |
| 163 | + | |
| 164 | + | |
| 165 | + | |
| 166 | + | |
| 167 | + | |
| 168 | + | |
| 169 | + | |
| 170 | + | |
147 | 171 | | |
148 | 172 | | |
149 | 173 | | |
| |||
0 commit comments