Skip to content

Commit 7085ad4

Browse files
divybotlittledivy
andauthored
chore: test for describe.only without only failure (#34590)
Adds a regression spec for `node:test` `describe.only()` to ensure it runs successfully without Deno reporting `Test failed because the "only" option was used`. The current node:test implementation already registers top-level node:test entries with `sanitizeOnly: false`; this locks the issue repro into the spec suite. Closes denoland/orchid#342 Closes #31862 Verification: - `/home/orchid/orch-work/issue-342/target/debug/deno test describe_only.test.js` - `cargo test -p specs_tests --test specs node_test_describe_only` Co-authored-by: divybot <divybot@users.noreply.github.com> Co-authored-by: Divy Srivastava <me@littledivy.com>
1 parent 17e4d5e commit 7085ad4

3 files changed

Lines changed: 18 additions & 0 deletions

File tree

Lines changed: 4 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,4 @@
1+
{
2+
"args": "test describe_only.test.js",
3+
"output": "describe_only.out"
4+
}
Lines changed: 9 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,9 @@
1+
running 1 test from ./describe_only.test.js
2+
a suite ...
3+
------- output -------
4+
Test suite is running
5+
----- output end -----
6+
a suite ... ok ([WILDCARD])
7+
8+
ok | 1 passed | 0 failed ([WILDCARD])
9+
Lines changed: 5 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,5 @@
1+
import { describe } from "node:test";
2+
3+
describe.only("a suite", () => {
4+
console.log("Test suite is running");
5+
});

0 commit comments

Comments
 (0)