ci: code reformatted and linter tests added (#49)

Signed-off-by: ffyuanda <46557895+ffyuanda@users.noreply.github.com>

Author: ffyuanda

.github/workflows/build.yml

@@ -38,6 +38,20 @@ jobs:
         COVERALLS_FLAG_NAME: ${{ matrix.os }} - ${{ matrix.python-version }}
         COVERALLS_PARALLEL: true
 
+  lint:
+    name: Run Linters
+    runs-on: ubuntu-latest
+    steps:
+      - name: Checkout
+        uses: actions/checkout@v2
+
+      - name: Super-Linter
+        uses: github/super-linter@v4.2.2
+        env:
+          VALIDATE_PYTHON_BLACK: true
+          DEFAULT_BRANCH: master
+          GITHUB_TOKEN: ${{ secrets.GITHUB_TOKEN }}
+
   coveralls:
     name: Indicate completion to coveralls.io
     needs: test

casbin_sqlalchemy_adapter/adapter.py

@@ -160,31 +160,33 @@ def remove_policies(self, sec, ptype, rules):
             query = query.filter(self._db_class.ptype == ptype)
             rules = zip(*rules)
             for i, rule in enumerate(rules):
-                query = query.filter(or_(getattr(self._db_class, "v{}".format(i)) == v for v in rule))
+                query = query.filter(
+                    or_(getattr(self._db_class, "v{}".format(i)) == v for v in rule)
+                )
             query.delete()
 
     def remove_filtered_policy(self, sec, ptype, field_index, *field_values):
         """removes policy rules that match the filter from the storage.
         This is part of the Auto-Save feature.
         """
         with self._session_scope() as session:
-            query = (session
-                     .query(self._db_class)
-                     .filter(self._db_class.ptype == ptype))
+            query = session.query(self._db_class).filter(self._db_class.ptype == ptype)
 
             if not (0 <= field_index <= 5):
                 return False
             if not (1 <= field_index + len(field_values) <= 6):
                 return False
             for i, v in enumerate(field_values):
-                if v != '':
+                if v != "":
                     v_value = getattr(self._db_class, "v{}".format(field_index + i))
                     query = query.filter(v_value == v)
             r = query.delete()
 
         return True if r > 0 else False
 
-    def update_policy(self, sec: str, ptype: str, old_rule: [str], new_rule: [str]) -> None:
+    def update_policy(
+        self, sec: str, ptype: str, old_rule: [str], new_rule: [str]
+    ) -> None:
         """
         Update the old_rule with the new_rule in the database (storage).
 
@@ -197,9 +199,7 @@ def update_policy(self, sec: str, ptype: str, old_rule: [str], new_rule: [str])
         """
 
         with self._session_scope() as session:
-            query = (session
-                     .query(self._db_class)
-                     .filter(self._db_class.ptype == ptype))
+            query = session.query(self._db_class).filter(self._db_class.ptype == ptype)
 
             # locate the old rule
             for index, value in enumerate(old_rule):
@@ -217,7 +217,17 @@ def update_policy(self, sec: str, ptype: str, old_rule: [str], new_rule: [str])
                 else:
                     exec(f"old_rule_line.v{index} = None")
 
-    def update_policies(self, sec: str, ptype: str, old_rules: [[str], ], new_rules: [[str], ]) -> None:
+    def update_policies(
+        self,
+        sec: str,
+        ptype: str,
+        old_rules: [
+            [str],
+        ],
+        new_rules: [
+            [str],
+        ],
+    ) -> None:
         """
         Update the old_rules with the new_rules in the database (storage).
 

setup.py

@@ -21,7 +21,16 @@
     long_description=long_description,
     long_description_content_type="text/markdown",
     url="https://github.com/pycasbin/sqlalchemy-adapter",
-    keywords=["casbin", "SQLAlchemy", "casbin-adapter", "rbac", "access control", "abac", "acl", "permission"],
+    keywords=[
+        "casbin",
+        "SQLAlchemy",
+        "casbin-adapter",
+        "rbac",
+        "access control",
+        "abac",
+        "acl",
+        "permission",
+    ],
     packages=find_packages(),
     install_requires=install_requires,
     python_requires=">=3.3",