aboutsummaryrefslogtreecommitdiff
path: root/xtask/tests/tidy-tests/docs.rs
blob: a8ed9793f2010d4d8babd5dcfe639c4f029a49e8 (plain)
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
94
95
96
97
98
99
100
101
102
103
104
105
106
107
108
use std::{collections::HashMap, fs, io::prelude::*, io::BufReader, path::Path};

use anyhow::Context;
use walkdir::{DirEntry, WalkDir};
use xtask::project_root;

fn is_exclude_dir(p: &Path) -> bool {
    // Test hopefully don't really need comments, and for assists we already
    // have special comments which are source of doc tests and user docs.
    let exclude_dirs = ["tests", "test_data", "handlers"];
    let mut cur_path = p;
    while let Some(path) = cur_path.parent() {
        if exclude_dirs.iter().any(|dir| path.ends_with(dir)) {
            return true;
        }
        cur_path = path;
    }

    false
}

fn is_exclude_file(d: &DirEntry) -> bool {
    let file_names = ["tests.rs"];

    d.file_name().to_str().map(|f_n| file_names.iter().any(|name| *name == f_n)).unwrap_or(false)
}

fn is_hidden(entry: &DirEntry) -> bool {
    entry.file_name().to_str().map(|s| s.starts_with('.')).unwrap_or(false)
}

#[test]
fn no_docs_comments() {
    let crates = project_root().join("crates");
    let iter = WalkDir::new(crates);
    let mut missing_docs = Vec::new();
    let mut contains_fixme = Vec::new();
    for f in iter.into_iter().filter_entry(|e| !is_hidden(e)) {
        let f = f.unwrap();
        if f.file_type().is_dir() {
            continue;
        }
        if f.path().extension().map(|it| it != "rs").unwrap_or(false) {
            continue;
        }
        if is_exclude_dir(f.path()) {
            continue;
        }
        if is_exclude_file(&f) {
            continue;
        }
        let mut reader = BufReader::new(fs::File::open(f.path()).unwrap());
        let mut line = String::new();
        reader
            .read_line(&mut line)
            .with_context(|| format!("Failed to read {}", f.path().display()))
            .unwrap();

        if line.starts_with("//!") {
            if line.contains("FIXME") {
                contains_fixme.push(f.path().to_path_buf())
            }
        } else {
            missing_docs.push(f.path().display().to_string());
        }
    }
    if !missing_docs.is_empty() {
        panic!(
            "\nMissing docs strings\n\n\
             modules:\n{}\n\n",
            missing_docs.join("\n")
        )
    }

    let whitelist = [
        "ra_cli",
        "ra_db",
        "ra_hir",
        "ra_hir_expand",
        "ra_ide",
        "ra_lsp_server",
        "ra_mbe",
        "ra_parser",
        "ra_prof",
        "ra_project_model",
        "ra_syntax",
        "ra_text_edit",
        "ra_tt",
        "ra_hir_ty",
    ];

    let mut has_fixmes = whitelist.iter().map(|it| (*it, false)).collect::<HashMap<&str, bool>>();
    'outer: for path in contains_fixme {
        for krate in whitelist.iter() {
            if path.components().any(|it| it.as_os_str() == *krate) {
                has_fixmes.insert(krate, true);
                continue 'outer;
            }
        }
        panic!("FIXME doc in a fully-documented crate: {}", path.display())
    }

    for (krate, has_fixme) in has_fixmes.iter() {
        if !has_fixme {
            panic!("crate {} is fully documented, remove it from the white list", krate)
        }
    }
}