Skip to content

Commit d9718c2

Browse files
committed
Add missing unit tests for # noqa:-like cases
1 parent b8f1284 commit d9718c2

10 files changed

Lines changed: 117 additions & 0 deletions

crates/ruff_linter/src/noqa.rs

Lines changed: 54 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -1250,6 +1250,24 @@ mod tests {
12501250
assert_lexed_ranges_match_slices(directive, source);
12511251
}
12521252

1253+
#[test]
1254+
fn noqa_no_code() {
1255+
let source = "# noqa:";
1256+
assert_debug_snapshot!(Directive::try_extract(source, TextSize::default()));
1257+
}
1258+
1259+
#[test]
1260+
fn noqa_no_code_invalid_suffix() {
1261+
let source = "# noqa: foo";
1262+
assert_debug_snapshot!(Directive::try_extract(source, TextSize::default()));
1263+
}
1264+
1265+
#[test]
1266+
fn noqa_no_code_trailing_content() {
1267+
let source = "# noqa: # Foo";
1268+
assert_debug_snapshot!(Directive::try_extract(source, TextSize::default()));
1269+
}
1270+
12531271
#[test]
12541272
fn noqa_code() {
12551273
let source = "# noqa: F401";
@@ -1482,6 +1500,24 @@ mod tests {
14821500
assert_lexed_ranges_match_slices(exemption, source);
14831501
}
14841502

1503+
#[test]
1504+
fn flake8_noqa_no_code() {
1505+
let source = "# flake8: # noqa:";
1506+
assert_debug_snapshot!(Directive::try_extract(source, TextSize::default()));
1507+
}
1508+
1509+
#[test]
1510+
fn flake8_noqa_no_code_invalid_suffix() {
1511+
let source = "# flake8: # noqa: foo";
1512+
assert_debug_snapshot!(Directive::try_extract(source, TextSize::default()));
1513+
}
1514+
1515+
#[test]
1516+
fn flake8_noqa_no_code_trailing_content() {
1517+
let source = "# flake8: # noqa: # Foo";
1518+
assert_debug_snapshot!(Directive::try_extract(source, TextSize::default()));
1519+
}
1520+
14851521
#[test]
14861522
fn ruff_exemption_all() {
14871523
let source = "# ruff: noqa";
@@ -1490,6 +1526,24 @@ mod tests {
14901526
assert_lexed_ranges_match_slices(exemption, source);
14911527
}
14921528

1529+
#[test]
1530+
fn ruff_noqa_no_code() {
1531+
let source = "# ruff: # noqa:";
1532+
assert_debug_snapshot!(Directive::try_extract(source, TextSize::default()));
1533+
}
1534+
1535+
#[test]
1536+
fn ruff_noqa_no_code_invalid_suffix() {
1537+
let source = "# ruff: # noqa: foo";
1538+
assert_debug_snapshot!(Directive::try_extract(source, TextSize::default()));
1539+
}
1540+
1541+
#[test]
1542+
fn ruff_noqa_no_code_trailing_content() {
1543+
let source = "# ruff: # noqa: # Foo";
1544+
assert_debug_snapshot!(Directive::try_extract(source, TextSize::default()));
1545+
}
1546+
14931547
#[test]
14941548
fn flake8_exemption_all_no_space() {
14951549
let source = "#flake8:noqa";
Lines changed: 7 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,7 @@
1+
---
2+
source: crates/ruff_linter/src/noqa.rs
3+
expression: "Directive::try_extract(source, TextSize::default())"
4+
---
5+
Err(
6+
MissingCodes,
7+
)
Lines changed: 7 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,7 @@
1+
---
2+
source: crates/ruff_linter/src/noqa.rs
3+
expression: "Directive::try_extract(source, TextSize::default())"
4+
---
5+
Err(
6+
MissingCodes,
7+
)
Lines changed: 7 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,7 @@
1+
---
2+
source: crates/ruff_linter/src/noqa.rs
3+
expression: "Directive::try_extract(source, TextSize::default())"
4+
---
5+
Err(
6+
MissingCodes,
7+
)
Lines changed: 7 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,7 @@
1+
---
2+
source: crates/ruff_linter/src/noqa.rs
3+
expression: "Directive::try_extract(source, TextSize::default())"
4+
---
5+
Err(
6+
MissingCodes,
7+
)
Lines changed: 7 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,7 @@
1+
---
2+
source: crates/ruff_linter/src/noqa.rs
3+
expression: "Directive::try_extract(source, TextSize::default())"
4+
---
5+
Err(
6+
MissingCodes,
7+
)
Lines changed: 7 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,7 @@
1+
---
2+
source: crates/ruff_linter/src/noqa.rs
3+
expression: "Directive::try_extract(source, TextSize::default())"
4+
---
5+
Err(
6+
MissingCodes,
7+
)
Lines changed: 7 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,7 @@
1+
---
2+
source: crates/ruff_linter/src/noqa.rs
3+
expression: "Directive::try_extract(source, TextSize::default())"
4+
---
5+
Err(
6+
MissingCodes,
7+
)
Lines changed: 7 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,7 @@
1+
---
2+
source: crates/ruff_linter/src/noqa.rs
3+
expression: "Directive::try_extract(source, TextSize::default())"
4+
---
5+
Err(
6+
MissingCodes,
7+
)
Lines changed: 7 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,7 @@
1+
---
2+
source: crates/ruff_linter/src/noqa.rs
3+
expression: "Directive::try_extract(source, TextSize::default())"
4+
---
5+
Err(
6+
MissingCodes,
7+
)

0 commit comments

Comments
 (0)