Added test for check doc strings in crates.

#1856

3 files changed, 71 insertions, 3 deletions

diff --git a/crates/ra_tools/tests/cli.rs b/crates/ra_tools/tests/cli.rs
index 91b19c8f8..609fd4d8b 100644
--- a/crates/ra_tools/tests/cli.rs
+++ b/crates/ra_tools/tests/cli.rs
@@ -1,6 +1,5 @@
-use walkdir::WalkDir;
-
 use ra_tools::{gen_tests, generate_boilerplate, project_root, run_rustfmt, Verify};
+use walkdir::WalkDir;
 
 #[test]
 fn generated_grammar_is_fresh() {
@@ -36,7 +35,7 @@ fn no_todo() {
         let text = std::fs::read_to_string(e.path()).unwrap();
         if text.contains("TODO") || text.contains("TOOD") {
             panic!(
-                "\nTODO markers should not be commited to the master branch,\n\
+                "\nTODO markers should not be committed to the master branch,\n\
                  use FIXME instead\n\
                  {}\n",
                 e.path().display(),
diff --git a/crates/ra_tools/tests/docs.rs b/crates/ra_tools/tests/docs.rs
new file mode 100644
index 000000000..1629247da
--- /dev/null
+++ b/crates/ra_tools/tests/docs.rs
@@ -0,0 +1,67 @@
+use std::fs;
+use std::io::prelude::*;
+use std::io::BufReader;
+use std::path::Path;
+
+use walkdir::{DirEntry, WalkDir};
+
+use ra_tools::project_root;
+
+fn is_exclude_dir(p: &Path) -> bool {
+    let exclude_dirs = ["tests", "test_data"];
+    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);
+    for f in iter.into_iter().filter_entry(|e| !is_hidden(e)) {
+        let f = f.unwrap(); //dbg!(f.unwrap());
+        if f.file_type().is_dir() {
+            continue;
+        }
+        if f.path().extension().map(|it| it != "rs").unwrap_or(false) {
+            //dbg!(f.path());
+            continue;
+        }
+        if is_exclude_dir(f.path()) {
+            //dbg!(f.path());
+            continue;
+        }
+        if is_exclude_file(&f) {
+            //dbg!(f.path());
+            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("//!") {
+            //dbg!(line);
+            panic!(
+                "\nMissing docs strings\n\
+                 module: {}\n\
+                 Need add doc for module or this string \"//! FIXME: write short doc here\"\n",
+                f.path().display()
+            )
+        }
+    }
+}
diff --git a/crates/ra_tools/tests/main.rs b/crates/ra_tools/tests/main.rs
new file mode 100644
index 000000000..56d1318d6
--- /dev/null
+++ b/crates/ra_tools/tests/main.rs
@@ -0,0 +1,2 @@
+mod cli;
+mod docs;