From 176fc552569a5eb31164970ef2d8dbf2cb7f6d1b Mon Sep 17 00:00:00 2001
From: Bruce Schubert <bruce@emxsys.com>
Date: Wed, 19 Aug 2020 12:09:49 -0700
Subject: [PATCH] Moved inidicators unit test to test_indicators.py - Re: #67

---
 src/hardware/indicators.py | 53 ----------------------------
 tests/test_indicators.py   | 72 ++++++++++++++++++++++++++++++++++++++
 2 files changed, 72 insertions(+), 53 deletions(-)
 create mode 100644 tests/test_indicators.py

diff --git a/src/hardware/indicators.py b/src/hardware/indicators.py
index bb0f63d..83f8a82 100644
--- a/src/hardware/indicators.py
+++ b/src/hardware/indicators.py
@@ -286,56 +286,3 @@ def set_char_layout(self, char, layout):
         self._layouts[char] = layout
 
 
-def test():
-    """ Unit Tests """
-    import os
-
-    print("*** Running Indicator Unit Tests ***")
-
-    try:
-        print("[Constructing Indicators]")
-        ring = RingIndicator()
-        approved = ApprovedIndicator()
-        blocked = BlockedIndicator()
-        message = MessageIndicator()
-
-        print("[Visual Tests]")
-
-        print("Turning on all LEDs for 5 seconds...")
-        ring.turn_on()
-        approved.turn_on()
-        blocked.turn_on()
-        message.turn_on()
-        time.sleep(5)
-
-        print("Blinking on all LEDs for 5 seconds...")
-        ring.blink()
-        time.sleep(.1)
-        approved.blink()
-        time.sleep(.1)
-        blocked.blink()
-        time.sleep(.1)
-        message.blink()
-        time.sleep(5)
-
-        print("Turning off all LEDs...")
-        ring.turn_off()
-        approved.turn_off()
-        blocked.turn_off()
-        message.turn_off()
-        time.sleep(5)
-
-    except Exception as e:
-        print("*** Unit test FAILED ***")
-        pprint(e)
-        return 1
-
-    print("*** Unit tests PASSED ***")
-    return 0
-
-
-if __name__ == '__main__':
-    """ Run the Unit Tests """
-    import sys
-    # Run the tests
-    sys.exit(test())
diff --git a/tests/test_indicators.py b/tests/test_indicators.py
new file mode 100644
index 0000000..d454bc0
--- /dev/null
+++ b/tests/test_indicators.py
@@ -0,0 +1,72 @@
+#!/usr/bin/env python
+# -*- coding: utf-8 -*-
+#
+#  test_indicators.py
+#
+#  Copyright 2020 Bruce Schubert  <bruce@emxsys.com>
+#
+#  Permission is hereby granted, free of charge, to any person obtaining a copy
+#  of this software and associated documentation files (the "Software"), to deal
+#  in the Software without restriction, including without limitation the rights
+#  to use, copy, modify, merge, publish, distribute, sublicense, and/or sell
+#  copies of the Software, and to permit persons to whom the Software is
+#  furnished to do so, subject to the following conditions:
+#
+#  The above copyright notice and this permission notice shall be included in all
+#  copies or substantial portions of the Software.
+#
+#  THE SOFTWARE IS PROVIDED "AS IS", WITHOUT WARRANTY OF ANY KIND, EXPRESS OR
+#  IMPLIED, INCLUDING BUT NOT LIMITED TO THE WARRANTIES OF MERCHANTABILITY,
+#  FITNESS FOR A PARTICULAR PURPOSE AND NONINFRINGEMENT. IN NO EVENT SHALL THE
+#  AUTHORS OR COPYRIGHT HOLDERS BE LIABLE FOR ANY CLAIM, DAMAGES OR OTHER
+#  LIABILITY, WHETHER IN AN ACTION OF CONTRACT, TORT OR OTHERWISE, ARISING FROM,
+#  OUT OF OR IN CONNECTION WITH THE SOFTWARE OR THE USE OR OTHER DEALINGS IN THE
+#  SOFTWARE.
+
+import os
+import sys
+import time
+from pprint import pprint
+
+import pytest
+
+from src.hardware.indicators import RingIndicator, ApprovedIndicator, BlockedIndicator, MessageIndicator
+
+def test_multiple():
+
+    ring = RingIndicator()
+    approved = ApprovedIndicator()
+    blocked = BlockedIndicator()
+    message = MessageIndicator()
+
+    print("[Visual Tests]")
+
+    print("Turning on all LEDs for 5 seconds...")
+    ring.turn_on()
+    approved.turn_on()
+    blocked.turn_on()
+    message.turn_on()
+    time.sleep(5)
+
+    print("Blinking on all LEDs for 5 seconds...")
+    ring.blink()
+    time.sleep(.1)
+    approved.blink()
+    time.sleep(.1)
+    blocked.blink()
+    time.sleep(.1)
+    message.blink()
+    time.sleep(5)
+
+    print("Turning off all LEDs...")
+    ring.turn_off()
+    approved.turn_off()
+    blocked.turn_off()
+    message.turn_off()
+    time.sleep(5)
+
+    # Release GPIO pins
+    ring.close()
+    approved.close()
+    blocked.close()
+    message.close()