Update pytest workflow to make it pass

The order of execution mangled things up. The package installation
actually downgrades some packages (e.g. pylint) which made the
pylint stage fail.

.github/workflows/pytest.yml

@@ -8,7 +8,7 @@ jobs:
     runs-on: ubuntu-latest
     strategy:
       matrix:
-        python-version: ["3.10"]
+        python-version: ["3.10", "3.9"]
     steps:
       - uses: actions/checkout@v3
       - name: Set up Python ${{ matrix.python-version }}
@@ -18,8 +18,8 @@ jobs:
       - name: Install dependencies
         run: |
           python -m pip install --upgrade pip
-          pip install -r tests/requirements.txt
           pip install -e .
+          pip install -r tests/requirements.txt
           pip install coverage pytest
       - name: Execute pytests to avoid regressions
         run: |

src/pyduin/arduino_cli.py

@@ -135,6 +135,13 @@ def main(): # pylint: disable=too-many-locals,too-many-statements,too-many-branc
     digitalpin_parser_pwm.add_argument('value', type=int, help='0-255')
 
     args = parser.parse_args()
+    if args.cmd == "dependencies":
+        utils.dependencies()
+        sys.exit(0)
+    if not args.cmd:
+        print("Nothing to do")
+        sys.exit(1)
+
     config = CliConfig(args)
     log_level = args.log_level or config.log_level
     logger.setLevel(level=getattr(logging, log_level.upper()))
@@ -145,9 +152,6 @@ def main(): # pylint: disable=too-many-locals,too-many-statements,too-many-branc
     if args.cmd in ('versions', 'v'):
         print(versions(arduino, config.workdir))
         sys.exit(0)
-    elif args.cmd == "dependencies":
-        utils.dependencies()
-        sys.exit(0)
     elif args.cmd in ('free', 'f'):
         print(arduino.free_memory)
         sys.exit(0)
@@ -196,9 +200,6 @@ def main(): # pylint: disable=too-many-locals,too-many-statements,too-many-branc
             res = pin.read()
             print(res.split('%')[-1])
         sys.exit(0)
-    else:
-        print("Nothing to do")
-        sys.exit(1)
     sys.exit(0)
 if __name__ == '__main__':
     main()

src/pyduin/utils.py

@@ -33,9 +33,7 @@
 
 
 class DeviceConfigError(BaseException):
-    """
-        Error Class to throw on config errors
-    """
+    """ Error Class to throw on config errors """
 
 class BuildEnvError(BaseException):
     """ Error class to be thrown on errors in the build environment """

tests/test_cli.py

@@ -41,13 +41,18 @@ def test_cli_invalid_choice(cli_runner, script_runner):
 
 def test_no_board(cli_runner, script_runner, tmp_path):
     path = f'{tmp_path}/pyduin.yml'
+    cfg = """
+    buddies:
+      foo:
+        tty: /foo/bar
+    """
     with open(f'{path}', 'w', encoding='utf-8') as cfile:
-        cfile.write("invalid: template")
-    with pytest.raises(DeviceConfigError) as err:
-        script_runner.run(f'pyduin --tty /foo/bar -c {path}', shell=True)
-    assert str(err.value) == "Cannot determine board for desired action."
+        cfile.write(cfg)
+    #with pytest.raises(DeviceConfigError) as err:
+    result = script_runner.run(f'pyduin -c {path}', shell=True)
+    assert result.stdout == "Nothing to do\n"
 
 def test_invalid_board(cli_runner, script_runner):
     with pytest.raises(DeviceConfigError) as err:
-        script_runner.run('pyduin --board=nixexista', shell=True)
+        script_runner.run('pyduin -b nixexista versions', shell=True)
     assert str(err.value).startswith('Board (nixexista) not in supported boards list')