fix: don't require hddtemp/hdparm for tests

desbma · Jun 21, 2023 · 8289284 · 8289284
1 parent ec98574
commit 8289284
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 5 deletions.
diff --git a/.github/workflows/ci.yml b/.github/workflows/ci.yml
@@ -20,7 +20,6 @@ jobs:
         with:
           python-version: ${{matrix.python-version}}
       - run: |
-          sudo apt-get install hddtemp hdparm
           pip install -U pip
           pip install coveralls
           pip install -r requirements.txt

diff --git a/hddfancontrol/__init__.py b/hddfancontrol/__init__.py
@@ -1287,6 +1287,9 @@ def cl_main():  # noqa: C901
         logging.getLogger("Startup").error("You need to run this script as root")
         exit(1)
 
+    # check mandatory deps
+    bin_dep.check_bin_dependency(("hdparm",))
+
     if args.test_mode or (args.fan_start_value is None) or (args.fan_stop_value is None):
         if (args.fan_start_value is None) or (args.fan_stop_value is None):
             logging.getLogger("Startup").warning(
@@ -1326,9 +1329,5 @@ def cl_main():  # noqa: C901
             )
 
 
-# check deps
-bin_dep.check_bin_dependency(("hddtemp", "hdparm"))
-
-
 if __name__ == "__main__":
     cl_main()