Added guard before setting result on reset future

bellows/uart.py

@@ -138,7 +138,9 @@ def rstack_frame_received(self, data):
             LOGGER.warn("Reset future is None")
             return
 
-        self._reset_future.set_result(True)
+        # Make sure that the reset_future is not done
+        if not self._reset_future.done():
+            self._reset_future.set_result(True)
 
     def error_frame_received(self, data):
         """Error frame receive handler"""

tests/test_uart.py

@@ -118,7 +118,9 @@ def test_rst_frame_received(gw):
 
 def test_rstack_frame_received(gw):
     gw._reset_future = mock.MagicMock()
+    gw._reset_future.done = mock.MagicMock(return_value=False)
     gw.data_received(b'\xc1\x02\x0b\nR\x7e')
+    assert gw._reset_future.done.call_count == 1
     assert gw._reset_future.set_result.call_count == 1
 
 
@@ -138,6 +140,14 @@ def test_rstack_frame_received_nofut(gw):
     gw.data_received(b'\xc1\x02\x0b\nR\x7e')
 
 
+def test_rstack_frame_received_out_of_order(gw):
+    gw._reset_future = mock.MagicMock()
+    gw._reset_future.done = mock.MagicMock(return_value=True)
+    gw.data_received(b'\xc1\x02\x0b\nR\x7e')
+    assert gw._reset_future.done.call_count == 1
+    assert gw._reset_future.set_result.call_count == 0
+
+
 def test_error_frame_received(gw):
     gw.frame_received(bytes([0b11000010]))