Commit 9274396
authored
## Summary
Follow-up to #4268. That PR bumped `testTimeout` to 20s for this test
but missed the second 9s ceiling: `configure({ asyncUtilTimeout: 9_000
})` in `packages/graphiql/setup-files.ts`. RTL's `waitFor` gives up at
the asyncUtilTimeout regardless of how big the `it()` timeout is — its
final assertion then runs against a `null` `queryEditor` and fails with
`expect(...).toBeVisible()` instead of the original "Test timed out"
message.
Evidence the previous fix was luck-of-the-draw: #4268 passed because
Monaco cold-booted at 7927ms — 73ms under the 9s cliff. #4266 failed at
9066ms on the same fix because Monaco took longer that run.
This PR adds `{ timeout: 15_000 }` to the `waitFor(...)` call so the
polling window matches the test's 20s ceiling. Keeps `it(..., 20000)` so
the two timeouts don't race in the other direction.
1 parent e4f5148 commit 9274396
1 file changed
Lines changed: 13 additions & 8 deletions
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
167 | 167 | | |
168 | 168 | | |
169 | 169 | | |
170 | | - | |
| 170 | + | |
| 171 | + | |
| 172 | + | |
171 | 173 | | |
172 | 174 | | |
173 | 175 | | |
174 | | - | |
175 | | - | |
176 | | - | |
177 | | - | |
178 | | - | |
179 | | - | |
180 | | - | |
| 176 | + | |
| 177 | + | |
| 178 | + | |
| 179 | + | |
| 180 | + | |
| 181 | + | |
| 182 | + | |
| 183 | + | |
| 184 | + | |
| 185 | + | |
181 | 186 | | |
182 | 187 | | |
183 | 188 | | |
| |||
0 commit comments