nu-lint 1.2.0

Linter for Nu shell scripts that helpfully suggests improvements
Documentation 
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
41
42
43
44
45
46
47
48
49
50
51
52
53
54
55
56
57
58
59
60
61
62
63
64
65
66
67
68
69
70
71
72
73
74
75
76
77
78
79

use super::RULE;

#[test]
fn detects_quoted_asterisk_glob_to_ls() {
    let code = r#"ls "*.txt""#;
    RULE.assert_detects(code);
}

#[test]
fn detects_quoted_question_mark_glob_to_ls() {
    let code = r#"ls "file?.rs""#;
    RULE.assert_detects(code);
}

#[test]
fn detects_quoted_glob_to_rm() {
    let code = r#"rm "*.log""#;
    RULE.assert_detects(code);
}

// NOTE: The `glob` command expects a string parameter (not a glob pattern).
// It processes the pattern internally, so quotes should NOT be removed.
// This is different from `ls`, `rm`, etc. which expect actual glob patterns.
// Test removed as the current behavior (not flagging) is correct.

#[test]
fn detects_quoted_glob_with_single_quotes() {
    let code = r#"ls '*.txt'"#;
    RULE.assert_detects(code);
}

#[test]
fn detects_complex_glob_pattern() {
    let code = r#"ls "src/**/*.rs""#;
    RULE.assert_detects(code);
}

#[test]
fn detects_glob_with_question_mark() {
    let code = r#"ls "test?.txt""#;
    RULE.assert_detects(code);
}

#[test]
fn detects_glob_in_cp_source() {
    let code = r#"cp "*.txt" dest/"#;
    RULE.assert_detects(code);
}

#[test]
fn detects_glob_in_mv_source() {
    let code = r#"mv "*.txt" dest/"#;
    RULE.assert_detects(code);
}

#[test]
fn detects_multiple_glob_args() {
    // Only the first arg to rm is glob, but we detect it
    let code = r#"rm "*.txt""#;
    RULE.assert_detects(code);
}

#[test]
fn detects_glob_with_spaces() {
    let code = r#"ls "my files *""#;
    RULE.assert_detects(code);
}

#[test]
fn detects_glob_with_spaces_and_wildcards() {
    let code = r#"ls "test files/*.txt""#;
    RULE.assert_detects(code);
}

#[test]
fn detects_glob_with_special_chars() {
    let code = r#"ls "path|with|pipes*""#;
    RULE.assert_detects(code);
}