Add CI workflow for Python testing with Tox (#95)

* Add CI workflow for Python testing with Tox

* Remove travis config

* Keep on truckin'

* Revert

* tweaks

Author: rm-hull

.github/dependabot.yml

@@ -0,0 +1,11 @@
+version: 2
+updates:
+  - package-ecosystem: "github-actions"
+    directory: "/"
+    schedule:
+      interval: "weekly"
+
+  - package-ecosystem: "pip"
+    directory: "/"
+    schedule:
+      interval: "weekly"

.github/workflows/ci.yml

@@ -0,0 +1,45 @@
+name: CI
+
+on:
+  push:
+  pull_request:
+
+jobs:
+  test:
+    runs-on: ubuntu-latest
+
+    strategy:
+      fail-fast: false
+      matrix:
+        python-version: [3.11, 3.12, 3.13, 3.14]
+        include:
+          - python-version: 3.11
+            env: TOXENV=qa
+
+    env:
+      TOX_PARALLEL_NO_SPINNER: 1
+
+    steps:
+      - name: Check out code
+        uses: actions/checkout@v4
+
+      - name: Set up Python
+        uses: actions/setup-python@v5
+        with:
+          python-version: ${{ matrix.python-version }}
+
+      - name: Install dependencies
+        run: |
+          python -m pip install --upgrade pip
+          pip install tox coveralls setuptools
+
+      - name: Run tests
+        run: tox -vv
+        env:
+          TOXENV: ${{ matrix.env.TOXENV || format('py{0}', matrix.python-version) }}
+
+      - name: Upload coverage to Coveralls (only for 3.11)
+        if: matrix.python-version == '3.11' && success()
+        run: coveralls
+        env:
+          GITHUB_TOKEN: ${{ secrets.GITHUB_TOKEN }}

.travis.yml

@@ -395,11 +395,11 @@ def my_callback_two(channel):
 import warnings
 
 from OPi.constants import IN, OUT
-from OPi.constants import LOW, HIGH                     # noqa: F401
-from OPi.constants import NONE, RISING, FALLING, BOTH   # noqa: F401
+from OPi.constants import LOW, HIGH  # noqa: F401
+from OPi.constants import NONE, RISING, FALLING, BOTH  # noqa: F401
 from OPi.constants import BCM, BOARD, SUNXI, CUSTOM
-from OPi.constants import PUD_UP, PUD_DOWN, PUD_OFF     # noqa: F401
-from OPi.constants import RED, GREEN                    # LEDs
+from OPi.constants import PUD_UP, PUD_DOWN, PUD_OFF  # noqa: F401
+from OPi.constants import RED, GREEN  # noqa: F401
 from OPi.pin_mappings import get_gpio_pin, set_custom_pin_mappings
 from OPi import event, sysfs
 
@@ -436,7 +436,7 @@ def setmode(mode):
         :py:attr:`GPIO.SUNXI`, or a `dict` or `object` representing a custom
         pin mapping.
     """
-    if hasattr(mode, '__getitem__'):
+    if hasattr(mode, "__getitem__"):
         set_custom_pin_mappings(mode)
         mode = CUSTOM
 
@@ -499,7 +499,10 @@ def setup(channel, direction, initial=None, pull_up_down=None):
 
     if pull_up_down is not None:
         if _gpio_warnings:
-            warnings.warn("Pull up/down setting are not (yet) fully supported, continuing anyway. Use GPIO.setwarnings(False) to disable warnings.", stacklevel=2)
+            warnings.warn(
+                "Pull up/down setting are not (yet) fully supported, continuing anyway. Use GPIO.setwarnings(False) to disable warnings.",
+                stacklevel=2,
+            )
 
     if isinstance(channel, list):
         for ch in channel:
@@ -511,9 +514,14 @@ def setup(channel, direction, initial=None, pull_up_down=None):
         try:
             sysfs.export(pin)
         except (OSError, IOError) as e:
-            if e.errno == 16:   # Device or resource busy
+            if e.errno == 16:  # Device or resource busy
                 if _gpio_warnings:
-                    warnings.warn("Channel {0} is already in use, continuing anyway. Use GPIO.setwarnings(False) to disable warnings.".format(channel), stacklevel=2)
+                    warnings.warn(
+                        "Channel {0} is already in use, continuing anyway. Use GPIO.setwarnings(False) to disable warnings.".format(
+                            channel
+                        ),
+                        stacklevel=2,
+                    )
                 sysfs.unexport(pin)
                 sysfs.export(pin)
             else:
@@ -565,6 +573,7 @@ def output(channel, state):
         pin = get_gpio_pin(_mode, channel)
         return sysfs.output(pin, state)
 
+
 def setled(led, state):
     """
     Set the state of a onboard LEDs.
@@ -583,11 +592,12 @@ def setled(led, state):
        GPIO.led(leds_list, (GPIO.HIGH, GPIO.LOW))      # sets first LED ON and second LED OFF
     """
     if isinstance(led, list):
-        for l in led:
-            setled(l, state)
+        for led_item in led:
+            setled(led_item, state)
     else:
         return sysfs.setled(led, state)
 
+
 def wait_for_edge(channel, trigger, timeout=-1):
     """
     This function is designed to block execution of your program until an edge
@@ -655,7 +665,10 @@ def add_event_detect(channel, trigger, callback=None, bouncetime=None):
 
     if bouncetime is not None:
         if _gpio_warnings:
-            warnings.warn("bouncetime is not (yet) fully supported, continuing anyway. Use GPIO.setwarnings(False) to disable warnings.", stacklevel=2)
+            warnings.warn(
+                "bouncetime is not (yet) fully supported, continuing anyway. Use GPIO.setwarnings(False) to disable warnings.",
+                stacklevel=2,
+            )
 
     pin = get_gpio_pin(_mode, channel)
     event.add_edge_detect(pin, trigger, __wrap(callback, channel))
@@ -682,7 +695,10 @@ def add_event_callback(channel, callback, bouncetime=None):
 
     if bouncetime is not None:
         if _gpio_warnings:
-            warnings.warn("bouncetime is not (yet) fully supported, continuing anyway. Use GPIO.setwarnings(False) to disable warnings.", stacklevel=2)
+            warnings.warn(
+                "bouncetime is not (yet) fully supported, continuing anyway. Use GPIO.setwarnings(False) to disable warnings.",
+                stacklevel=2,
+            )
 
     pin = get_gpio_pin(_mode, channel)
     event.add_edge_callback(pin, __wrap(callback, channel))
@@ -768,8 +784,9 @@ class PWM:
     # 2. find way to check _exports against _exports_pwm to make sure there is no overlap.
     # 3. Create map of pwm pins to various boards.
 
-    def __init__(self, chip, pin, frequency, duty_cycle_percent, invert_polarity=False):  # (pwm pin, frequency in KHz)
-
+    def __init__(
+        self, chip, pin, frequency, duty_cycle_percent, invert_polarity=False
+    ):  # (pwm pin, frequency in KHz)
         """
         Setup the PWM object to control.
 
@@ -791,34 +808,47 @@ def __init__(self, chip, pin, frequency, duty_cycle_percent, invert_polarity=Fal
         try:
             sysfs.PWM_Export(chip, pin)  # creates the pwm sysfs object
         except (OSError, IOError) as e:
-            if e.errno == 16:   # Device or resource busy
-                warnings.warn("Pin {0} is already in use, continuing anyway.".format(pin), stacklevel=2)
+            if e.errno == 16:  # Device or resource busy
+                warnings.warn(
+                    "Pin {0} is already in use, continuing anyway.".format(pin),
+                    stacklevel=2,
+                )
                 sysfs.PWM_Unexport(chip, pin)
                 sysfs.PWM_Export(chip, pin)
             else:
                 raise e
 
         # invert polarity  if needed
         if invert_polarity is True:
-            sysfs.PWM_Polarity(chip, pin, invert=True)  # invert pwm i.e the duty cycle tells you how long the cycle is off
+            sysfs.PWM_Polarity(
+                chip, pin, invert=True
+            )  # invert pwm i.e the duty cycle tells you how long the cycle is off
         else:
-            sysfs.PWM_Polarity(chip, pin, invert=False)  # don't invert the pwm signal. This is the normal way its used.
+            sysfs.PWM_Polarity(
+                chip, pin, invert=False
+            )  # don't invert the pwm signal. This is the normal way its used.
 
         # enable pwm
         sysfs.PWM_Enable(chip, pin)
         sysfs.PWM_Frequency(chip, pin, frequency)
 
-    def start_pwm(self):  # turn on pwm by setting the duty cycle to what the user specified
+    def start_pwm(
+        self,
+    ):  # turn on pwm by setting the duty cycle to what the user specified
         """
         Start PWM Signal.
         """
-        return sysfs.PWM_Duty_Cycle_Percent(self.chip, self.pin, self.duty_cycle_percent)  # duty cycle controls the on-off
+        return sysfs.PWM_Duty_Cycle_Percent(
+            self.chip, self.pin, self.duty_cycle_percent
+        )  # duty cycle controls the on-off
 
     def stop_pwm(self):  # turn on pwm by setting the duty cycle to 0
         """
         Stop PWM Signal.
         """
-        return sysfs.PWM_Duty_Cycle_Percent(self.chip, self.pin, 0)  # duty cycle at 0 is the equivilant of off
+        return sysfs.PWM_Duty_Cycle_Percent(
+            self.chip, self.pin, 0
+        )  # duty cycle at 0 is the equivilant of off
 
     def change_frequency(self, new_frequency):
         # Order of operations:
@@ -842,7 +872,7 @@ def change_frequency(self, new_frequency):
 
         old_pwm_period = int(round((1 / self.frequency) * 1e9, 0))
 
-        if (pwm_period > old_pwm_period):  # if increasing
+        if pwm_period > old_pwm_period:  # if increasing
             sysfs.PWM_Period(self.chip, self.pin, pwm_period)  # update the pwm period
             sysfs.PWM_Duty_Cycle(self.chip, self.pin, duty_cycle)  # update duty cycle
 
@@ -859,18 +889,24 @@ def duty_cycle(self, duty_cycle_percent):  # in percentage (0-100)
         :param duty_cycle_percent: the new PWM duty cycle as a percentage.
         """
 
-        if (0 <= duty_cycle_percent <= 100):
+        if 0 <= duty_cycle_percent <= 100:
             self.duty_cycle_percent = duty_cycle_percent
-            return sysfs.PWM_Duty_Cycle_Percent(self.chip, self.pin, self.duty_cycle_percent)
+            return sysfs.PWM_Duty_Cycle_Percent(
+                self.chip, self.pin, self.duty_cycle_percent
+            )
         else:
-            raise Exception("Duty cycle must br between 0 and 100. Current value: {0} is out of bounds".format(duty_cycle_percent))
+            raise Exception(
+                "Duty cycle must br between 0 and 100. Current value: {0} is out of bounds".format(
+                    duty_cycle_percent
+                )
+            )
 
     def pwm_polarity(self):  # invert the polarity of the pwm
         """
         Invert the signal.
         """
         sysfs.PWM_Disable(self.chip, self.pin)
-        sysfs.PWM_Polarity(self.chip, self.pin, invert=not(self.invert_polarity))
+        sysfs.PWM_Polarity(self.chip, self.pin, invert=not (self.invert_polarity))
         sysfs.PWM_Enable(self.chip, self.pin)
 
     def pwm_close(self):

OPi/GPIO.py

@@ -89,7 +89,9 @@ def blocking_wait_for_edge(pin, trigger, timeout=-1):
     assert trigger in [RISING, FALLING, BOTH]
 
     if pin in _threads:
-        raise RuntimeError("Conflicting edge detection events already exist for this GPIO channel")
+        raise RuntimeError(
+            "Conflicting edge detection events already exist for this GPIO channel"
+        )
 
     try:
         sysfs.edge(pin, trigger)
@@ -133,7 +135,9 @@ def add_edge_detect(pin, trigger, callback=None):
     assert trigger in [RISING, FALLING, BOTH]
 
     if pin in _threads:
-        raise RuntimeError("Conflicting edge detection already enabled for this GPIO channel")
+        raise RuntimeError(
+            "Conflicting edge detection already enabled for this GPIO channel"
+        )
 
     _threads[pin] = _worker(pin, trigger, callback)
     _threads[pin].start()