Merge pull request Webperf-se#343 from cockroacher/i331

Remaining pylint issues for Webperf-se#331

.github/workflows/verify_result.py

@@ -383,10 +383,12 @@ def validate_translations():
 
     This function performs two main validation steps:
     1. Validates the existence and correctness of .po and .mo files in the project.
-    2. Validates the usage of global_translation() and local_translation() in .py files in the project.
+    2. Validates the usage of global_translation() and
+       local_translation() in .py files in the project.
 
     The function uses the `validate_locales` function to validate .po and .mo files,
-    and the `validate_python_files` function to validate global_translation() and local_translation() usage.
+    and the `validate_python_files` function to validate global_translation() and
+       local_translation() usage.
 
     Returns:
         bool: True if all validations pass, False otherwise.

tests/w3c_base.py

@@ -64,12 +64,12 @@ def get_errors(test_type, params):
         arg = f'--exit-zero-always{test_arg} --stdout --format json --errors-only {file_path}'
 
     command = f'java -jar vnu.jar {arg}'
-    process = subprocess.Popen(command.split(), stdout=subprocess.PIPE)
-    output, _ = process.communicate(timeout=REQUEST_TIMEOUT * 10)
+    with subprocess.Popen(command.split(), stdout=subprocess.PIPE) as process:
+        output, _ = process.communicate(timeout=REQUEST_TIMEOUT * 10)
 
-    json_result = json.loads(output)
-    if 'messages' in json_result:
-        errors = json_result['messages']
+        json_result = json.loads(output)
+        if 'messages' in json_result:
+            errors = json_result['messages']
 
     return errors