nu-lint 1.3.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
80
81
82
83
84
85
86
87
88
89
90
91
92
93

use super::RULE;

#[test]
fn test_ignore_already_using_default() {
    let good_code = r#"
def test [x] {
    $x | default "fallback"
}
"#;
    RULE.assert_ignores(good_code);
}

#[test]
fn test_ignore_modified_variable_in_else() {
    // else block modifies the variable, not just returns it
    let good_code = r#"
def test [x] {
    if $x == null { 0 } else { $x + 1 }
}
"#;
    RULE.assert_ignores(good_code);
}

#[test]
fn test_ignore_different_variable_in_else() {
    // else block returns a different variable
    let good_code = r#"
def test [x, y, z] {
    if $x == null { $y } else { $z }
}
"#;
    RULE.assert_ignores(good_code);
}

#[test]
fn test_ignore_no_else_branch() {
    let good_code = r#"
def test [x] {
    if $x == null { print "missing" }
}
"#;
    RULE.assert_ignores(good_code);
}

#[test]
fn test_ignore_else_if_chain() {
    let good_code = r#"
def test [x] {
    if $x == null { 0 } else if $x > 10 { 10 } else { $x }
}
"#;
    RULE.assert_ignores(good_code);
}

#[test]
fn test_ignore_non_null_comparison() {
    let good_code = r#"
def test [x] {
    if $x == 0 { "zero" } else { $x }
}
"#;
    RULE.assert_ignores(good_code);
}

#[test]
fn test_ignore_multiple_statements_in_then() {
    let good_code = r#"
def test [x] {
    if $x == null { print "null"; 0 } else { $x }
}
"#;
    RULE.assert_ignores(good_code);
}

#[test]
fn test_ignore_multiple_statements_in_else() {
    let good_code = r#"
def test [x] {
    if $x == null { 0 } else { print $x; $x }
}
"#;
    RULE.assert_ignores(good_code);
}

#[test]
fn test_ignore_boolean_comparison() {
    let good_code = r#"
def test [flag] {
    if $flag == true { "yes" } else { $flag }
}
"#;
    RULE.assert_ignores(good_code);
}