2019-11-12 06:41:02 -06:00
|
|
|
use std::{collections::HashMap, fs, io::prelude::*, io::BufReader, path::Path};
|
2019-09-30 03:58:53 -05:00
|
|
|
|
2020-02-17 09:57:06 -06:00
|
|
|
use anyhow::Context;
|
2019-09-30 03:58:53 -05:00
|
|
|
use walkdir::{DirEntry, WalkDir};
|
2019-10-17 15:01:53 -05:00
|
|
|
use xtask::project_root;
|
2019-09-30 03:58:53 -05:00
|
|
|
|
|
|
|
fn is_exclude_dir(p: &Path) -> bool {
|
2019-10-25 06:16:46 -05:00
|
|
|
// Test hopefully don't really need comments, and for assists we already
|
|
|
|
// have special comments which are source of doc tests and user docs.
|
2020-02-07 08:53:31 -06:00
|
|
|
let exclude_dirs = ["tests", "test_data", "handlers"];
|
2019-09-30 03:58:53 -05:00
|
|
|
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 {
|
2019-10-30 12:36:37 -05:00
|
|
|
entry.file_name().to_str().map(|s| s.starts_with('.')).unwrap_or(false)
|
2019-09-30 03:58:53 -05:00
|
|
|
}
|
|
|
|
|
|
|
|
#[test]
|
|
|
|
fn no_docs_comments() {
|
|
|
|
let crates = project_root().join("crates");
|
|
|
|
let iter = WalkDir::new(crates);
|
2019-11-03 15:11:37 -06:00
|
|
|
let mut missing_docs = Vec::new();
|
2019-11-12 06:41:02 -06:00
|
|
|
let mut contains_fixme = Vec::new();
|
2019-09-30 03:58:53 -05:00
|
|
|
for f in iter.into_iter().filter_entry(|e| !is_hidden(e)) {
|
2019-09-30 04:09:56 -05:00
|
|
|
let f = f.unwrap();
|
2019-09-30 03:58:53 -05:00
|
|
|
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();
|
2020-02-17 09:57:06 -06:00
|
|
|
reader
|
|
|
|
.read_line(&mut line)
|
|
|
|
.with_context(|| format!("Failed to read {}", f.path().display()))
|
|
|
|
.unwrap();
|
2019-11-12 06:41:02 -06:00
|
|
|
|
|
|
|
if line.starts_with("//!") {
|
|
|
|
if line.contains("FIXME") {
|
|
|
|
contains_fixme.push(f.path().to_path_buf())
|
|
|
|
}
|
|
|
|
} else {
|
2019-11-03 15:11:37 -06:00
|
|
|
missing_docs.push(f.path().display().to_string());
|
2019-09-30 03:58:53 -05:00
|
|
|
}
|
|
|
|
}
|
2019-11-03 15:11:37 -06:00
|
|
|
if !missing_docs.is_empty() {
|
|
|
|
panic!(
|
|
|
|
"\nMissing docs strings\n\n\
|
|
|
|
modules:\n{}\n\n",
|
|
|
|
missing_docs.join("\n")
|
|
|
|
)
|
|
|
|
}
|
2019-11-12 06:41:02 -06:00
|
|
|
|
|
|
|
let whitelist = [
|
|
|
|
"ra_db",
|
|
|
|
"ra_hir",
|
|
|
|
"ra_hir_expand",
|
2019-11-27 12:32:33 -06:00
|
|
|
"ra_ide",
|
2019-11-12 06:41:02 -06:00
|
|
|
"ra_mbe",
|
|
|
|
"ra_parser",
|
|
|
|
"ra_prof",
|
|
|
|
"ra_project_model",
|
|
|
|
"ra_syntax",
|
|
|
|
"ra_text_edit",
|
|
|
|
"ra_tt",
|
2019-11-27 08:46:02 -06:00
|
|
|
"ra_hir_ty",
|
2019-11-12 06:41:02 -06:00
|
|
|
];
|
|
|
|
|
|
|
|
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)
|
|
|
|
}
|
|
|
|
}
|
2019-09-30 03:58:53 -05:00
|
|
|
}
|