Merge pull request #39 from SAK917/master

Replace assert with if statement (Fixes #32)
adafruit · Feb 11, 2021 · c84c901 · c84c901
2 parents a691635 + 1334c4d
commit c84c901
Show file tree

Hide file tree

Showing 3 changed files with 6 additions and 3 deletions.
diff --git a/adafruit_mcp230xx/mcp23008.py b/adafruit_mcp230xx/mcp23008.py
@@ -85,5 +85,6 @@ def get_pin(self, pin):
         """Convenience function to create an instance of the DigitalInOut class
         pointing at the specified pin of this MCP23008 device.
         """
-        assert 0 <= pin <= 7
+        if not 0 <= pin <= 7:
+            raise ValueError("Pin number must be 0-7.")
         return DigitalInOut(pin, self)
diff --git a/adafruit_mcp230xx/mcp23016.py b/adafruit_mcp230xx/mcp23016.py
@@ -126,7 +126,8 @@ def get_pin(self, pin):
         """Convenience function to create an instance of the DigitalInOut class
         pointing at the specified pin of this MCP23016 device.
         """
-        assert 0 <= pin <= 15
+        if not 0 <= pin <= 15:
+            raise ValueError("Pin number must be 0-15.")
         return DigitalInOut(pin, self)
 
     def clear_inta(self):

diff --git a/adafruit_mcp230xx/mcp23017.py b/adafruit_mcp230xx/mcp23017.py
@@ -163,7 +163,8 @@ def get_pin(self, pin):
         """Convenience function to create an instance of the DigitalInOut class
         pointing at the specified pin of this MCP23017 device.
         """
-        assert 0 <= pin <= 15
+        if not 0 <= pin <= 15:
+            raise ValueError("Pin number must be 0-15.")
         return DigitalInOut(pin, self)
 
     @property