Test for check_spec function, for tidy.py #9152

python/tidy.py

@@ -69,6 +69,8 @@ ignored_dirs = [
     os.path.join(".", "."),
 ]
 
+spec_base_path = "components/script/dom/"
+
 
 def is_iter_empty(iterator):
     try:
@@ -525,10 +527,9 @@ def check_json(filename, contents):
 
 
 def check_spec(file_name, lines):
-    base_path = "components/script/dom/"
+    if spec_base_path not in file_name:
-    if base_path not in file_name:
         raise StopIteration
-    file_name = os.path.relpath(os.path.splitext(file_name)[0], base_path)
+    file_name = os.path.relpath(os.path.splitext(file_name)[0], spec_base_path)
     patt = re.compile("^\s*\/\/.+")
 
     # Pattern representing a line with a macro

python/tidy_self_test/speclink.rs

@@ -0,0 +1,9 @@
+/* This Source Code Form is subject to the terms of the Mozilla Public
+ * License, v. 2.0. If a copy of the MPL was not distributed with this
+ * file, You can obtain one at http://mozilla.org/MPL/2.0/. */
+
+impl SpecLinkMethods for SpecLink {
+    fn Test(&self) -> f32 {
+        0
+    }
+}

python/tidy_self_test/tidy_self_test.py

@@ -59,6 +59,11 @@ class CheckTidiness(unittest.TestCase):
         self.assertEqual('use &[T] instead of &Vec<T>', errors.next()[2])
         self.assertEqual('use &str instead of &String', errors.next()[2])
 
+    def test_spec_link(self):
+        tidy.spec_base_path = "python/tidy_self_test/"
+        errors = tidy.collect_errors_for_files(iterFile('speclink.rs'), [], [tidy.check_spec])
+        self.assertEqual('method declared in webidl is missing a comment with a specification link', errors.next()[2])
+
     def test_webidl(self):
         errors = tidy.collect_errors_for_files(iterFile('spec.webidl'), [tidy.check_webidl_spec], [])
         self.assertEqual('No specification link found.', errors.next()[2])