[FMWK-77] Check code coverage in Github actions workflow (#9)

- Fix test case for invalid operate policy
- Fix converting any policy to an operate policy

.github/workflows/python-package.yml

@@ -49,5 +49,6 @@ jobs:
         poetry run python3 cookbook-ex.py
     - name: Run unit tests
       run: |
-        cd tests/
-        poetry run python3 test.py
+        poetry run coverage run -m unittest tests/test.py
+    - name: Run coverage test
+      run: poetry run coverage report --fail-under=100 --include=documentapi/api.py

documentapi/api.py

@@ -425,7 +425,7 @@ def convertToOperatePolicy(policy: Policy) -> Union[Policy, None]:
 
     # Filter out non-operate policies
     operatePolicy = policy.copy()
-    for key in operatePolicy:
+    for key in policy:
         if key not in OPERATE_CONFIG_KEYS:
             operatePolicy.pop(key)
 

pyproject.toml

@@ -14,6 +14,7 @@ packages = [{include = "documentapi"}]
 python = ">= 3.7, < 3.10"
 jsonpath-ng-aerospike = "^1.5.3"
 aerospike = ">= 6.0, < 8.0"
+coverage = "^6.5.0"
 
 [tool.poetry.group.dev.dependencies]
 flake8 = "^5.0.4"

tests/test.py

@@ -215,6 +215,15 @@ def testGetKeyWithBrackets(self):
         results = documentClient.get(keyTuple, MAP_BIN_NAME, "$['key[with.brackets]']")
         self.assertEqual(results, mapJsonObj["key[with.brackets]"])
 
+    def testGetWithInvalidOperatePolicy(self):
+        # This read policy should be filtered out
+        # Since it can't be used in an operate policy
+        readPolicy = {
+            "deserialize": False
+        }
+        results = documentClient.get(keyTuple, MAP_BIN_NAME, "$['key[with.brackets]']", readPolicy)
+        self.assertEqual(results, mapJsonObj["key[with.brackets]"])
+
 
 class TestGetAdvancedOps(TestGets):