2019-09-30 08:58:53 +00:00
|
|
|
use std::fs;
|
|
|
|
use std::io::prelude::*;
|
|
|
|
use std::io::BufReader;
|
|
|
|
use std::path::Path;
|
|
|
|
|
|
|
|
use walkdir::{DirEntry, WalkDir};
|
|
|
|
|
2019-10-17 20:01:53 +00:00
|
|
|
use xtask::project_root;
|
2019-09-30 08:58:53 +00:00
|
|
|
|
|
|
|
fn is_exclude_dir(p: &Path) -> bool {
|
2019-10-25 11:16:46 +00: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.
|
|
|
|
let exclude_dirs = ["tests", "test_data", "assists"];
|
2019-09-30 08:58:53 +00: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 17:36:37 +00:00
|
|
|
entry.file_name().to_str().map(|s| s.starts_with('.')).unwrap_or(false)
|
2019-09-30 08:58:53 +00:00
|
|
|
}
|
|
|
|
|
|
|
|
#[test]
|
|
|
|
fn no_docs_comments() {
|
|
|
|
let crates = project_root().join("crates");
|
|
|
|
let iter = WalkDir::new(crates);
|
|
|
|
for f in iter.into_iter().filter_entry(|e| !is_hidden(e)) {
|
2019-09-30 09:09:56 +00:00
|
|
|
let f = f.unwrap();
|
2019-09-30 08:58:53 +00: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();
|
|
|
|
reader.read_line(&mut line).unwrap();
|
|
|
|
if !line.starts_with("//!") {
|
|
|
|
panic!(
|
|
|
|
"\nMissing docs strings\n\
|
|
|
|
module: {}\n\
|
2019-09-30 10:31:28 +00:00
|
|
|
Need add doc for module\n",
|
2019-09-30 08:58:53 +00:00
|
|
|
f.path().display()
|
|
|
|
)
|
|
|
|
}
|
|
|
|
}
|
|
|
|
}
|