natlint 0.1.0

A linter for Solidity natspec comments.
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 solang_parser::pt::VariableDefinition;

crate::too_many_comments_rule!(
    TooManyInheritdoc,
    VariableDefinition,
    Inheritdoc,
    "Variables must not have more than one inheritdoc comment."
);

#[cfg(test)]
mod tests {
    use super::{TooManyInheritdoc, VariableDefinition};
    use crate::{
        generate_too_many_comment_test_cases,
        parser::{CommentTag, CommentsRef, Parser},
        rules::{Rule, Violation, ViolationError},
    };
    use forge_fmt::Visitable;
    use solang_parser::parse;

    fn parse_source(src: &str) -> Parser {
        let (mut source, comments) = parse(src, 0).expect("failed to parse source");
        let mut doc = Parser::new(comments, src.to_owned());
        source.visit(&mut doc).expect("failed to visit source");
        doc
    }

    macro_rules! test_too_many_notice {
        ($name:ident, $source:expr, $expected:expr) => {
            #[test]
            fn $name() {
                let src = parse_source($source);

                let parent = src.items_ref().first().unwrap();
                let child = parent.children.first().unwrap();
                let var = child.as_variable().unwrap();
                let comments = CommentsRef::from(&child.comments);

                let expected = $expected(var);

                assert_eq!(
                    TooManyInheritdoc::check(Some(parent), var, &comments),
                    expected
                );
            }
        };
    }

    pub mod pub_const_test {
        use super::*;

        generate_too_many_comment_test_cases!(
            Inheritdoc,
            test_too_many_notice,
            TooManyInheritdoc,
            r#"
                bytes32 public constant SOME_CONST = keccak256("SOME_CONST");
            "#,
            "@inheritdoc",
            VariableDefinition
        );
    }

    pub mod pub_immutable_test {
        use super::*;

        generate_too_many_comment_test_cases!(
            Inheritdoc,
            test_too_many_notice,
            TooManyInheritdoc,
            r"
                bytes32 public immutable SOME_IMMUT;
            ",
            "@inheritdoc",
            VariableDefinition
        );
    }

    pub mod priv_state_test {
        use super::*;

        generate_too_many_comment_test_cases!(
            Inheritdoc,
            test_too_many_notice,
            TooManyInheritdoc,
            r"
                State private state;
            ",
            "@inheritdoc",
            VariableDefinition
        );
    }
}