Avoid excessive switches on startup (#293)

* Prevent initial value for relays.

* Changelog: 5.0.2.

* Asserting turned off in constructor test.

* Fix assertion.

* set initial value to `inverted`

* Updating the serialization test.

* Removing assertions since it has to be xorred.

* Update CHANGELOG.md

CHANGELOG.md

@@ -2,6 +2,10 @@
 
 ## Version 5
 
+### 5.0.2
+
+- Initial state for relays is `OFF` — fixes excessive switches on startup in most cases.
+
 ### 5.0.1
 
 - Preventing installations on environments having unsupported Python versions:

octoprint_octorelay/driver.py

@@ -11,7 +11,7 @@ class Driver():
     def __init__(self, pin: int, inverted: bool, pin_factory=None):
         self.pin = pin # GPIO pin
         self.inverted = inverted # marks the relay as normally closed
-        self.handle = LED(pin, pin_factory=pin_factory)
+        self.handle = LED(pin, pin_factory=pin_factory, initial_value=inverted)
 
     def __repr__(self) -> str:
         return f"{type(self).__name__}(pin={self.pin},inverted={self.inverted},closed={self.is_closed()})"

tests/test_driver.py

@@ -22,7 +22,7 @@ def test_constructor(self):
     def test_serialization(self):
         relay = Driver(18, True, MockFactory())
         serialization = f"{relay}"
-        self.assertEqual(serialization, "Driver(pin=18,inverted=True,closed=True)")
+        self.assertEqual(serialization, "Driver(pin=18,inverted=True,closed=False)")
 
     def test_close(self):
         cases = [