Merge pull request #15 from scratchrealm/coverage-testing

coverage testing

.flake8

@@ -1,9 +1,10 @@
 [flake8]
-extend-ignore = E124,E128,E301,E302,E305,E501,E261
+extend-ignore = E124,E128,E301,E302,E305,E402,E501,E261
 # E124: closing bracket does not match visual indentation
 # E128: continuation line under-indented for visual indent
 # E301: expected 1 blank line, found 0
 # E302: expected 2 blank lines, found 1
 # E305: expected 2 blank lines after class or function definition, found 1
+# E402: module level import not at top of file
 # E501: line too long (82 > 79 characters)
 # E261: at least two spaces before inline comment

.github/workflows/linter_checks.yml

@@ -11,7 +11,7 @@ jobs:
     - name: Install
       run: cd python && pip install -e .
     - name: Install packages needed for tests
-      run: pip install pytest pytest-cov pyright boto3 kachery_cloud flake8
+      run: pip install pytest pytest-asyncio pytest-cov pyright boto3 kachery_cloud flake8
     - name: Install additional packages used by api_helpers
       run: pip install -r requirements.txt
     - name: Run linter checks

.github/workflows/tests.yml

@@ -11,7 +11,11 @@ jobs:
     - name: Install
       run: cd python && pip install -e .
     - name: Install packages needed for tests
-      run: pip install pytest pytest-cov boto3 kachery_cloud
+      run: pip install pytest pytest-asyncio pytest-cov boto3 kachery_cloud
+    - name: Run non-api tests
+      run: cd python && pytest -m "not api" tests/ # make sure we are not depending on any of the additional packages in requirements.txt
+    - name: Install packages needed for api tests
+      run: pip install -r requirements.txt
     - name: Run tests and collect coverage
       run: cd python && pytest --cov protocaas --cov-report=xml --cov-report=term tests/
     - uses: codecov/codecov-action@v3

doc/for_developers.md

@@ -1,7 +1,7 @@
 ## Running the tests (vscode action)
 
 ```bash
-pip install pytest pytest-cov pyright flake8
+pip install pytest pytest-asyncio pytest-cov pyright flake8
 ```
 
 In vscode: