rustqual 1.4.2

Comprehensive Rust code quality analyzer — seven dimensions: IOSP, Complexity, DRY, SRP, Coupling, Test Quality, Architecture
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

//! `run_dry_detection` per-detector enable guards. Each closure has an inner
//! `if !config.<flag> { return vec![]; }` short-circuit; deleting the `!` would
//! run the detector when the flag is off.
use super::*;

fn parse(code: &str) -> Vec<(String, String, syn::File)> {
    vec![(
        "test.rs".to_string(),
        code.to_string(),
        syn::parse_file(code).expect("parse failed"),
    )]
}

fn run_dry(config: &crate::config::Config, code: &str) -> DryResults {
    let parsed = parse(code);
    let sups = std::collections::HashMap::new();
    let api = std::collections::HashMap::new();
    let test_helper = std::collections::HashMap::new();
    let annotation_lines = AnnotationLines {
        api: &api,
        test_helper: &test_helper,
    };
    let cfg_test_files = std::collections::HashSet::new();
    run_dry_detection(&parsed, config, &sups, &annotation_lines, &cfg_test_files)
}

#[test]
fn dead_code_detection_skipped_when_disabled() {
    // With `detect_dead_code = false` an uncalled fn must NOT be reported;
    // guards the `if !c.duplicates.detect_dead_code` short-circuit.
    let mut config = crate::config::Config::default();
    config.duplicates.detect_dead_code = false;
    config.compile();
    let dry = run_dry(&config, "fn unused() { let _ = 1; }");
    assert!(
        dry.dead_code.is_empty(),
        "no dead-code warnings when detect_dead_code is off, got {:?}",
        dry.dead_code
    );
}

#[test]
fn wildcard_detection_skipped_when_duplicates_disabled() {
    // `detect_wildcard_imports` stays on but `duplicates.enabled = false`, so the
    // wildcard closure's `if !c.duplicates.enabled` must short-circuit to empty;
    // guards that `!`.
    let mut config = crate::config::Config::default();
    config.duplicates.enabled = false;
    config.compile();
    let dry = run_dry(&config, "use crate::foo::*;");
    assert!(
        dry.wildcard_warnings.is_empty(),
        "no wildcard warnings when duplicates is disabled, got {:?}",
        dry.wildcard_warnings
    );
}