harn-lint 0.8.4

Linter for the Harn programming language
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

//! `unnecessary-parentheses` — removes grouping around one value when
//! the parens are not part of a call, declaration, or precedence boundary.

use super::*;

#[test]
fn test_unnecessary_parentheses_autofixes_ternary_list_branch() {
    let source = r#"pipeline default() {
  let args = repo ? (["--repo", repo]) : []
}
"#;
    let diags = lint_source(source);
    assert_eq!(
        count_rule(&diags, "unnecessary-parentheses"),
        1,
        "expected unnecessary-parentheses, got: {diags:?}"
    );
    let fixed = apply_fixes(source, &diags);
    assert!(
        fixed.contains(r#"repo ? ["--repo", repo] : []"#),
        "expected autofix to unwrap list branch, got: {fixed}"
    );
}

#[test]
fn test_unnecessary_parentheses_autofixes_simple_return_value() {
    let source = r#"fn repo_arg(repo) {
  return ( repo )
}
"#;
    let diags = lint_source(source);
    assert_eq!(count_rule(&diags, "unnecessary-parentheses"), 1);
    let fixed = apply_fixes(source, &diags);
    assert_eq!(fixed, "fn repo_arg(repo) {\n  return repo\n}\n");
}

#[test]
fn test_unnecessary_parentheses_ignores_call_and_decl_parens() {
    let source = r#"fn repo_arg(repo) {
  log(repo)
}
"#;
    let diags = lint_source(source);
    assert!(
        !has_rule(&diags, "unnecessary-parentheses"),
        "function declaration and call parens must not fire, got: {diags:?}"
    );
}

#[test]
fn test_unnecessary_parentheses_ignores_precedence_grouping() {
    let source = r#"pipeline default() {
  let x = a * (b + c)
  let y = (a || b) && c
}
"#;
    let diags = lint_source(source);
    assert!(
        !has_rule(&diags, "unnecessary-parentheses"),
        "precedence-preserving parens must not fire, got: {diags:?}"
    );
}

#[test]
fn test_unnecessary_parentheses_only_unwraps_inner_nested_pair() {
    let source = r#"pipeline default() {
  let repo = ((name))
}
"#;
    let diags = lint_source(source);
    assert_eq!(
        count_rule(&diags, "unnecessary-parentheses"),
        1,
        "only the inner parens should be fixed in one pass, got: {diags:?}"
    );
    let fixed = apply_fixes(source, &diags);
    assert!(
        fixed.contains("(name)"),
        "expected one parenthesis layer to remain for the next lint pass, got: {fixed}"
    );
}