Merge pull request #55 from ssalonen/ci-fix

CI fixes

Author: hifiberry

.github/workflows/pythonpackage.yml

@@ -1,6 +1,6 @@
 name: Python package
 
-on: [push]
+on: [push, pull_request]
 
 jobs:
   build:
@@ -9,14 +9,15 @@ jobs:
     strategy:
       max-parallel: 4
       matrix:
-        python-version: [3.5, 3.6, 3.7]
+        python-version: ["3.7", "3.8", "3.9", "3.10", "3.11"]
 
     steps:
     - uses: actions/checkout@v1
     - name: Set up Python ${{ matrix.python-version }}
       uses: actions/setup-python@v1
       with:
         python-version: ${{ matrix.python-version }}
+        cache: 'pip' # caching pip dependencies
     - name: Install dependencies
       run: |
         sudo apt install gcc libasound-dev
@@ -29,7 +30,7 @@ jobs:
         flake8 . --count --select=E9,F63,F7,F82 --show-source --statistics
         # exit-zero treats all errors as warnings. The GitHub editor is 127 chars wide
         flake8 . --count --exit-zero --max-complexity=10 --max-line-length=127 --statistics
-#    - name: Test with pytest
-#      run: |
-#        pip install pytest
-#        pytest
+    - name: Test with pytest
+      run: |
+        pip install pytest
+        pytest

hifiberrydsp/__init__.py

@@ -1 +1,2 @@
 __version__ = "0.20"
+_called_from_test = False

hifiberrydsp/conftest.py

@@ -0,0 +1,4 @@
+
+def pytest_configure(config):
+    import hifiberrydsp
+    hifiberrydsp._called_from_test = True

hifiberrydsp/hardware/spi.py

@@ -20,16 +20,21 @@
 SOFTWARE.
 '''
 import logging
-
-
-def init_spi():
-    import spidev
-    spi = spidev.SpiDev()
-    spi.open(0, 0)
-    spi.bits_per_word = 8
-    spi.max_speed_hz = 1000000
-    spi.mode = 0
-    logging.debug("spi initialized %s", spi)
+import hifiberrydsp
+
+def init_spi():        
+    if not hifiberrydsp._called_from_test:
+        # only open the device when not running tests
+        import spidev        
+        spi = spidev.SpiDev()        
+        spi.open(0, 0)
+        spi.bits_per_word = 8
+        spi.max_speed_hz = 1000000
+        spi.mode = 0
+        logging.debug("spi initialized %s", spi)
+    else:
+        spi = None
+        logging.debug("spi not initialized since running tests")
     return spi