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

use super::RULE;
use crate::log::init_test_log;

#[test]
fn test_detect_print_exit_pattern() {
    let bad_code = r#"
def bad-error [] {
    print "Error occurred"
    exit 1
}
"#;
    RULE.assert_count(bad_code, 1);
}

#[test]
fn test_detect_various_error_messages() {
    for msg in [
        "Error: File not found",
        "Failed to connect to server",
        "Cannot parse input",
        "Unable to access resource",
        "Missing configuration file",
        "Access denied",
        "Connection timeout",
    ] {
        let bad_code = format!(r#"print "{msg}"; exit 1"#);
        RULE.assert_detects(&bad_code);
    }
}

#[test]
fn test_detect_print_exit_with_invalid_message() {
    init_test_log();
    let bad_code = r#"
if ($args | is-empty) {
    print "Invalid arguments provided"
    exit 1
}
"#;
    RULE.assert_count(bad_code, 1);
}

#[test]
fn test_detect_nested_in_function() {
    let bad_code = r#"
def main [] {
    def helper [] {
        print "Error in helper function"
        exit 1
    }
    helper
}
"#;
    RULE.assert_detects(bad_code);
}

#[test]
fn test_detect_in_closure() {
    let bad_code = r#"
let checker = { ||
    print "Error in closure"
    exit 1
}
do $checker
"#;
    RULE.assert_detects(bad_code);
}

#[test]
fn test_detect_same_pipeline_variations() {
    RULE.assert_detects(r#"def validate [] { print "Validation failed"; exit 1 }"#);
    RULE.assert_detects(r#"let result = some_command; print "Command failed"; exit 2"#);
    let bad_code = r#"
def check [value: int] {
    if $value < 0 {
        print "Negative value not allowed"; exit 1
    }
}
"#;
    RULE.assert_detects(bad_code);
}