Add Python code lint check

This updates the avocado-static-check submodule, and implements the changes necessary to have the lint checks passing. Signed-off-by: Cleber Rosa <[email protected]>
avocado-framework · Feb 22, 2024 · 3e67b7b · 3e67b7b
1 parent 09199b9
commit 3e67b7b
Show file tree

Hide file tree

Showing 6 changed files with 10 additions and 7 deletions.
diff --git a/.github/workflows/modules-tests.yml b/.github/workflows/modules-tests.yml
@@ -15,7 +15,7 @@ jobs:
         run: pip3 install avocado-framework==102.0
 
       - name: Run the ar module test
-        run: ./tests/test-module.py metadata/autils/archive/ar.yml
+        run: ./tests/test_module.py metadata/autils/archive/ar.yml
 
   static-checks:
     runs-on: ubuntu-latest

diff --git a/.github/workflows/schema-validation.yml b/.github/workflows/schema-validation.yml
@@ -12,4 +12,4 @@ jobs:
         uses: actions/checkout@v3
 
       - name: Validating all libraries against our schema
-        run: python3 tests/validate-schema.py
+        run: python3 tests/validate_schema.py
diff --git a/autils/archive/ar.py b/autils/archive/ar.py
@@ -80,8 +80,8 @@ def __next__(self):
                 member = struct.unpack(
                     FILE_HEADER_FMT, open_file.read(FILE_HEADER_SIZE)
                 )
-            except struct.error:
-                raise StopIteration
+            except struct.error as exc:
+                raise StopIteration from exc
 
             # No support for extended file names
             identifier = member[0].decode("ascii").strip()
@@ -105,3 +105,4 @@ def read_member(self, identifier):
                 with self as open_file:
                     open_file.seek(member.offset)
                     return open_file.read(member.size)
+        return None
diff --git a/static-checks b/static-checks
diff --git a/tests/test-module.py → tests/test_module.py b/tests/test-module.py → tests/test_module.py
@@ -12,8 +12,8 @@
     "Fedora 37": "fedora:37",
 }
 
-metadata_path = sys.argv[1]
-with open(metadata_path, "rb") as m:
+METADATA_PATH = sys.argv[1]
+with open(METADATA_PATH, "rb") as m:
     metadata = yaml.load(m, Loader=yaml.SafeLoader)
 
 test_suites = []

diff --git a/tests/validate-schema.py → tests/validate_schema.py b/tests/validate-schema.py → tests/validate_schema.py
@@ -39,6 +39,7 @@ def validate_yaml(filename, schema):
 
 
 def validate_yamls():
+    """Validates all yamls in the repo against the autils schema."""
     print("Starting validation...")
     failed = False
     for file in glob.glob("./metadata/autils/*/*.yml"):
@@ -53,6 +54,7 @@ def validate_yamls():
         return 1
 
     print("All files passed validation.")
+    return 0
 
 
 if __name__ == "__main__":