ops

Webperf-se · Oct 15, 2023 · bd88e26 · bd88e26
1 parent f2b1f0f
commit bd88e26
Show file tree

Hide file tree

Showing 2 changed files with 7 additions and 5 deletions.
diff --git a/tests/html_validator_w3c.py b/tests/html_validator_w3c.py
@@ -74,7 +74,6 @@ def run_test(_, langCode, url):
 
 
         error_message_grouped_dict = {}
-        error_message_grouped_for_rating_dict = {}
         if number_of_errors > 0:
             regex = r"(“[^”]+”)"
             for item in errors:

diff --git a/tests/w3c_base.py b/tests/w3c_base.py
@@ -25,11 +25,13 @@ def get_errors(test_type, params):
     arg = ''
     test_arg = ''
     errors = list()
+    is_html = False
 
     if 'css' in params or test_type == 'css':
         test_arg = ' --css --skip-non-css'
     if 'html' in params or test_type == 'html':
         test_arg = ' --html --skip-non-html'
+        is_html = True
 
     if 'doc' in params:
         url = params['doc']
@@ -39,10 +41,11 @@ def get_errors(test_type, params):
                 'Tested url must start with \'https://\' or \'http://\': {0}'.format(url))
 
         file_path = get_cache_path(url, True)
-        html_file_ending_fix = file_path.replace('.cache', '.cache.html')
-        if has_cache_file(url, True, cache_time_delta) and not os.path.exists(file_path):
-            os.rename(file_path, html_file_ending_fix)
-        file_path = html_file_ending_fix
+        if is_html:
+            html_file_ending_fix = file_path.replace('.cache', '.cache.html')
+            if has_cache_file(url, True, cache_time_delta) and not os.path.exists(html_file_ending_fix):
+                os.rename(file_path, html_file_ending_fix)
+            file_path = html_file_ending_fix
 
         arg = '--exit-zero-always{1} --stdout --format json --errors-only {0}'.format(
             file_path, test_arg)