Skip to content

Commit 0061ce7

Browse files
committed
test(linter): add more tests for disable directives in partial loadable files (#14371)
1 parent c257b41 commit 0061ce7

7 files changed

+483
-71
lines changed
Lines changed: 46 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,46 @@
1+
<template>
2+
<div>Hello World</div>
3+
</template>
4+
5+
<script>
6+
// eslint-disable-next-line no-debugger
7+
console.log('regular script');
8+
9+
// eslint-disable-next-line no-console
10+
debugger;
11+
</script>
12+
13+
<script setup>
14+
// eslint-disable
15+
const unusedVariable1 = 42;
16+
17+
// eslint-disable-next-line no-unused-vars
18+
const unusedVariable2 = 100;
19+
20+
function testFunction() {
21+
// eslint-disable-next-line no-console
22+
console.log('Inside test function');
23+
}
24+
25+
// eslint-enable
26+
27+
// eslint-disable-next-line no-console, no-debugger
28+
console.log('yes'); debugger;
29+
30+
// oxlint-disable-next-line no-debugger, no-for-loop
31+
console.log("complete line");
32+
33+
testFunction();
34+
</script>
35+
36+
<script setup lang="ts" generic="T extends Record<string, string>">
37+
let foo: T;
38+
39+
// eslint-disable-next-line no-debugger
40+
console.log('setup with generics');
41+
42+
// eslint-disable-next-line no-console
43+
debugger;
44+
45+
// eslint-enable
46+
</script>
Lines changed: 37 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,37 @@
1+
---
2+
// eslint-disable
3+
const unusedVariable1 = 42;
4+
5+
// eslint-disable-next-line no-debugger
6+
console.log('This is a test');
7+
8+
// eslint-enable
9+
10+
// eslint-disable-next-line no-console
11+
debugger;
12+
---
13+
14+
<div>Hello World</div>
15+
16+
<script>
17+
// eslint-disable-next-line no-unused-vars
18+
const unusedVariable2 = 100;
19+
20+
function testFunction() {
21+
// eslint-disable-next-line no-console
22+
console.log('Inside test function');
23+
}
24+
25+
// eslint-disable-next-line no-console, no-debugger
26+
console.log('yes'); debugger;
27+
28+
// eslint-disable-next-line no-console, no-debugger
29+
console.log('no');
30+
31+
// oxlint-disable-next-line no-debugger, no-for-loop
32+
console.log("complete line");
33+
34+
testFunction();
35+
36+
// eslint-enable
37+
</script>
Lines changed: 35 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,35 @@
1+
<script>
2+
// eslint-disable
3+
const unusedVariable1 = 42;
4+
5+
// eslint-disable-next-line no-debugger
6+
console.log('This is a test');
7+
8+
// eslint-enable
9+
10+
// eslint-disable-next-line no-console
11+
debugger;
12+
13+
// eslint-disable-next-line no-unused-vars
14+
const unusedVariable2 = 100;
15+
16+
function testFunction() {
17+
// eslint-disable-next-line no-console
18+
console.log('Inside test function');
19+
}
20+
21+
// eslint-disable-next-line no-console, no-debugger
22+
console.log('yes'); debugger;
23+
24+
// eslint-disable-next-line no-console, no-debugger
25+
console.log('no');
26+
27+
// oxlint-disable-next-line no-debugger, no-for-loop
28+
console.log("complete line");
29+
30+
testFunction();
31+
32+
// eslint-enable
33+
</script>
34+
35+
<h1>Hello!</h1>
Lines changed: 37 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,37 @@
1+
<template>
2+
<div>Hello World</div>
3+
</template>
4+
5+
<script>
6+
// eslint-disable
7+
const unusedVariable1 = 42;
8+
9+
// eslint-disable-next-line no-debugger
10+
console.log('This is a test');
11+
12+
// eslint-enable
13+
14+
// eslint-disable-next-line no-console
15+
debugger;
16+
17+
// eslint-disable-next-line no-unused-vars
18+
const unusedVariable2 = 100;
19+
20+
function testFunction() {
21+
// eslint-disable-next-line no-console
22+
console.log('Inside test function');
23+
}
24+
25+
// eslint-disable-next-line no-console, no-debugger
26+
console.log('yes'); debugger;
27+
28+
// eslint-disable-next-line no-console, no-debugger
29+
console.log('no');
30+
31+
// oxlint-disable-next-line no-debugger, no-for-loop
32+
console.log("complete line");
33+
34+
testFunction();
35+
36+
// eslint-enable
37+
</script>

apps/oxlint/src/lint.rs

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1097,7 +1097,7 @@ mod test {
10971097

10981098
#[test]
10991099
fn test_report_unused_directives() {
1100-
let args = &["-c", ".oxlintrc.json", "--report-unused-disable-directives", "test.js"];
1100+
let args = &["-c", ".oxlintrc.json", "--report-unused-disable-directives"];
11011101

11021102
Tester::new().with_cwd("fixtures/report_unused_directives".into()).test_and_snapshot(args);
11031103
}

apps/oxlint/src/snapshots/fixtures__report_unused_directives_-c .oxlintrc.json --report-unused-disable-directives test.js@oxlint.snap

Lines changed: 0 additions & 70 deletions
This file was deleted.

0 commit comments

Comments
 (0)