From 48ed7690afda63e83cd31bdaaf38e2b8b7fdad37 Mon Sep 17 00:00:00 2001
From: Mike Kinney <mike.kinney@gmail.com>
Date: Wed, 12 Jan 2022 13:46:01 -0800
Subject: [PATCH 1/5] fix the consider-using-f-string warnings

---
 .pylintrc              |  2 +-
 meshtastic/__main__.py |  4 ++--
 meshtastic/util.py     | 23 +++++++++++------------
 3 files changed, 14 insertions(+), 15 deletions(-)

diff --git a/.pylintrc b/.pylintrc
index 5deb236..49c0e03 100644
--- a/.pylintrc
+++ b/.pylintrc
@@ -23,7 +23,7 @@ ignore-patterns=mqtt_pb2.py,channel_pb2.py,environmental_measurement_pb2.py,admi
 # no Warning level messages displayed, use"--disable=all --enable=classes
 # --disable=W"
 #
-disable=invalid-name,fixme,logging-fstring-interpolation,too-many-statements,too-many-branches,too-many-locals,no-member,f-string-without-interpolation,protected-access,no-self-use,pointless-string-statement,too-few-public-methods,consider-using-f-string,broad-except,no-else-return,unused-argument,global-statement,global-variable-not-assigned,too-many-boolean-expressions,no-else-raise,bare-except,c-extension-no-member
+disable=invalid-name,fixme,logging-fstring-interpolation,too-many-statements,too-many-branches,too-many-locals,no-member,f-string-without-interpolation,protected-access,no-self-use,pointless-string-statement,too-few-public-methods,broad-except,no-else-return,unused-argument,global-statement,global-variable-not-assigned,too-many-boolean-expressions,no-else-raise,bare-except,c-extension-no-member
 
 
 [BASIC]
diff --git a/meshtastic/__main__.py b/meshtastic/__main__.py
index 68990df..1c8514e 100644
--- a/meshtastic/__main__.py
+++ b/meshtastic/__main__.py
@@ -40,7 +40,7 @@ def onReceive(packet, interface):
                 rxSnr = packet['rxSnr']
                 hopLimit = packet['hopLimit']
                 print(f"message: {msg}")
-                reply = "got msg \'{}\' with rxSnr: {} and hopLimit: {}".format(msg, rxSnr, hopLimit)
+                reply = f"got msg \'{msg}\' with rxSnr: {rxSnr} and hopLimit: {hopLimit}"
                 print("Sending reply: ", reply)
                 interface.sendText(reply)
 
@@ -192,7 +192,7 @@ def onConnected(interface):
 
             else:
                 print(f"Setting position fields to {allFields}")
-                setPref(prefs, 'position_flags', ('%d' % allFields))
+                setPref(prefs, 'position_flags', f'{allFields:d}')
                 print("Writing modified preferences to device")
                 interface.getNode(args.dest).writeConfig()
 
diff --git a/meshtastic/util.py b/meshtastic/util.py
index 6b0b8c8..0d6a837 100644
--- a/meshtastic/util.py
+++ b/meshtastic/util.py
@@ -189,17 +189,16 @@ def support_info():
     print('or wish to make feature requests, visit:')
     print('https://github.com/meshtastic/Meshtastic-python/issues')
     print('When adding an issue, be sure to include the following info:')
-    print(' System: {0}'.format(platform.system()))
-    print('   Platform: {0}'.format(platform.platform()))
-    print('   Release: {0}'.format(platform.uname().release))
-    print('   Machine: {0}'.format(platform.uname().machine))
-    print('   Encoding (stdin): {0}'.format(sys.stdin.encoding))
-    print('   Encoding (stdout): {0}'.format(sys.stdout.encoding))
+    print(f' System: {platform.system()}')
+    print(f'   Platform: {platform.platform()}')
+    print(f'   Release: {platform.uname().release}')
+    print(f'   Machine: {platform.uname().machine}')
+    print(f'   Encoding (stdin): {sys.stdin.encoding}')
+    print(f'   Encoding (stdout): {sys.stdout.encoding}')
     the_version = pkg_resources.get_distribution("meshtastic").version
-    print(' meshtastic: v{0}'.format(the_version))
-    print(' Executable: {0}'.format(sys.argv[0]))
-    print(' Python: {0} {1} {2}'.format(platform.python_version(),
-          platform.python_implementation(), platform.python_compiler()))
+    print(f' meshtastic: v{the_version}')
+    print(f' Executable: {sys.argv[0]}')
+    print(f' Python: {platform.python_version()} {platform.python_implementation()} {platform.python_compiler()}')
     print('')
     print('Please add the output from the command: meshtastic --info')
 
@@ -221,12 +220,12 @@ def remove_keys_from_dict(keys, adict):
 
 def hexstr(barray):
     """Print a string of hex digits"""
-    return ":".join('{:02x}'.format(x) for x in barray)
+    return ":".join(f'{x:02x}' for x in barray)
 
 
 def ipstr(barray):
     """Print a string of ip digits"""
-    return ".".join('{}'.format(x) for x in barray)
+    return ".".join(f'{x}' for x in barray)
 
 
 def readnet_u16(p, offset):

From 0f89baa36e9def01b8df350597a383f44d409ff4 Mon Sep 17 00:00:00 2001
From: Mike Kinney <mike.kinney@gmail.com>
Date: Wed, 12 Jan 2022 14:41:49 -0800
Subject: [PATCH 2/5] fix the pylint unused-argument warnings

---
 .pylintrc                                 |   2 +-
 meshtastic/__main__.py                    |   2 +-
 meshtastic/ble_interface.py               |   2 +-
 meshtastic/mesh_interface.py              |   4 +-
 meshtastic/tests/test_ble_interface.py    |   1 +
 meshtastic/tests/test_main.py             | 247 +++++++++++++++-------
 meshtastic/tests/test_mesh_interface.py   | 131 ++++++++----
 meshtastic/tests/test_node.py             |  14 +-
 meshtastic/tests/test_serial_interface.py |   4 +
 meshtastic/tests/test_stream_interface.py |   6 +-
 meshtastic/tests/test_tcp_interface.py    |   2 +-
 meshtastic/tests/test_tunnel.py           |  28 +--
 meshtastic/tests/test_util.py             |   1 +
 meshtastic/tunnel.py                      |   2 +-
 14 files changed, 291 insertions(+), 155 deletions(-)

diff --git a/.pylintrc b/.pylintrc
index 49c0e03..8228bf4 100644
--- a/.pylintrc
+++ b/.pylintrc
@@ -23,7 +23,7 @@ ignore-patterns=mqtt_pb2.py,channel_pb2.py,environmental_measurement_pb2.py,admi
 # no Warning level messages displayed, use"--disable=all --enable=classes
 # --disable=W"
 #
-disable=invalid-name,fixme,logging-fstring-interpolation,too-many-statements,too-many-branches,too-many-locals,no-member,f-string-without-interpolation,protected-access,no-self-use,pointless-string-statement,too-few-public-methods,broad-except,no-else-return,unused-argument,global-statement,global-variable-not-assigned,too-many-boolean-expressions,no-else-raise,bare-except,c-extension-no-member
+disable=invalid-name,fixme,logging-fstring-interpolation,too-many-statements,too-many-branches,too-many-locals,no-member,f-string-without-interpolation,protected-access,no-self-use,pointless-string-statement,too-few-public-methods,broad-except,no-else-return,global-statement,global-variable-not-assigned,too-many-boolean-expressions,no-else-raise,bare-except,c-extension-no-member
 
 
 [BASIC]
diff --git a/meshtastic/__main__.py b/meshtastic/__main__.py
index 1c8514e..9ea52f4 100644
--- a/meshtastic/__main__.py
+++ b/meshtastic/__main__.py
@@ -48,7 +48,7 @@ def onReceive(packet, interface):
         print(f'Warning: There is no field {ex} in the packet.')
 
 
-def onConnection(interface, topic=pub.AUTO_TOPIC):
+def onConnection(interface, topic=pub.AUTO_TOPIC): # pylint: disable=W0613
     """Callback invoked when we connect/disconnect from a radio"""
     print(f"Connection changed: {topic.getName()}")
 
diff --git a/meshtastic/ble_interface.py b/meshtastic/ble_interface.py
index 0bafefc..4f476a5 100644
--- a/meshtastic/ble_interface.py
+++ b/meshtastic/ble_interface.py
@@ -39,7 +39,7 @@ class BLEInterface(MeshInterface):
 
         self._readFromRadio()  # read the initial responses
 
-        def handle_data(handle, data):
+        def handle_data(handle, data): # pylint: disable=W0613
             self._handleFromRadio(data)
 
         if self.device:
diff --git a/meshtastic/mesh_interface.py b/meshtastic/mesh_interface.py
index 6f88e98..39e67e0 100644
--- a/meshtastic/mesh_interface.py
+++ b/meshtastic/mesh_interface.py
@@ -73,7 +73,7 @@ class MeshInterface:
             logging.error(f'Traceback: {traceback}')
         self.close()
 
-    def showInfo(self, file=sys.stdout):
+    def showInfo(self, file=sys.stdout): # pylint: disable=W0613
         """Show human readable summary about this object"""
         owner = f"Owner: {self.getLongName()} ({self.getShortName()})"
         myinfo = ''
@@ -100,7 +100,7 @@ class MeshInterface:
         print(infos)
         return infos
 
-    def showNodes(self, includeSelf=True, file=sys.stdout):
+    def showNodes(self, includeSelf=True, file=sys.stdout): # pylint: disable=W0613
         """Show table summary of nodes in mesh"""
         def formatFloat(value, precision=2, unit=''):
             """Format a float value with precsion."""
diff --git a/meshtastic/tests/test_ble_interface.py b/meshtastic/tests/test_ble_interface.py
index f0cd962..bb8d135 100644
--- a/meshtastic/tests/test_ble_interface.py
+++ b/meshtastic/tests/test_ble_interface.py
@@ -12,3 +12,4 @@ def test_BLEInterface(mock_platform):
     """Test that we can instantiate a BLEInterface"""
     iface = BLEInterface('foo', debugOut=True, noProto=True)
     iface.close()
+    mock_platform.assert_called()
diff --git a/meshtastic/tests/test_main.py b/meshtastic/tests/test_main.py
index 41175cc..84721f3 100644
--- a/meshtastic/tests/test_main.py
+++ b/meshtastic/tests/test_main.py
@@ -22,7 +22,8 @@ from ..remote_hardware import onGPIOreceive
 
 
 @pytest.mark.unit
-def test_main_init_parser_no_args(capsys, reset_globals):
+@pytest.mark.usefixtures("reset_globals")
+def test_main_init_parser_no_args(capsys):
     """Test no arguments"""
     sys.argv = ['']
     Globals.getInstance().set_args(sys.argv)
@@ -33,7 +34,8 @@ def test_main_init_parser_no_args(capsys, reset_globals):
 
 
 @pytest.mark.unit
-def test_main_init_parser_version(capsys, reset_globals):
+@pytest.mark.usefixtures("reset_globals")
+def test_main_init_parser_version(capsys):
     """Test --version"""
     sys.argv = ['', '--version']
     Globals.getInstance().set_args(sys.argv)
@@ -48,7 +50,8 @@ def test_main_init_parser_version(capsys, reset_globals):
 
 
 @pytest.mark.unit
-def test_main_main_version(capsys, reset_globals):
+@pytest.mark.usefixtures("reset_globals")
+def test_main_main_version(capsys):
     """Test --version"""
     sys.argv = ['', '--version']
     Globals.getInstance().set_args(sys.argv)
@@ -63,7 +66,8 @@ def test_main_main_version(capsys, reset_globals):
 
 
 @pytest.mark.unit
-def test_main_main_no_args(reset_globals, capsys):
+@pytest.mark.usefixtures("reset_globals")
+def test_main_main_no_args(capsys):
     """Test with no args"""
     sys.argv = ['']
     Globals.getInstance().set_args(sys.argv)
@@ -77,7 +81,8 @@ def test_main_main_no_args(reset_globals, capsys):
 
 
 @pytest.mark.unit
-def test_main_support(capsys, reset_globals):
+@pytest.mark.usefixtures("reset_globals")
+def test_main_support(capsys):
     """Test --support"""
     sys.argv = ['', '--support']
     Globals.getInstance().set_args(sys.argv)
@@ -95,8 +100,9 @@ def test_main_support(capsys, reset_globals):
 
 
 @pytest.mark.unit
+@pytest.mark.usefixtures("reset_globals")
 @patch('meshtastic.util.findPorts', return_value=[])
-def test_main_ch_index_no_devices(patched_find_ports, capsys, reset_globals):
+def test_main_ch_index_no_devices(patched_find_ports, capsys):
     """Test --ch-index 1"""
     sys.argv = ['', '--ch-index', '1']
     Globals.getInstance().set_args(sys.argv)
@@ -113,8 +119,9 @@ def test_main_ch_index_no_devices(patched_find_ports, capsys, reset_globals):
 
 
 @pytest.mark.unit
+@pytest.mark.usefixtures("reset_globals")
 @patch('meshtastic.util.findPorts', return_value=[])
-def test_main_test_no_ports(patched_find_ports, reset_globals, capsys):
+def test_main_test_no_ports(patched_find_ports, capsys):
     """Test --test with no hardware"""
     sys.argv = ['', '--test']
     Globals.getInstance().set_args(sys.argv)
@@ -130,8 +137,9 @@ def test_main_test_no_ports(patched_find_ports, reset_globals, capsys):
 
 
 @pytest.mark.unit
+@pytest.mark.usefixtures("reset_globals")
 @patch('meshtastic.util.findPorts', return_value=['/dev/ttyFake1'])
-def test_main_test_one_port(patched_find_ports, reset_globals, capsys):
+def test_main_test_one_port(patched_find_ports, capsys):
     """Test --test with one fake port"""
     sys.argv = ['', '--test']
     Globals.getInstance().set_args(sys.argv)
@@ -147,8 +155,9 @@ def test_main_test_one_port(patched_find_ports, reset_globals, capsys):
 
 
 @pytest.mark.unit
+@pytest.mark.usefixtures("reset_globals")
 @patch('meshtastic.test.testAll', return_value=True)
-def test_main_test_two_ports_success(patched_test_all, reset_globals, capsys):
+def test_main_test_two_ports_success(patched_test_all, capsys):
     """Test --test two fake ports and testAll() is a simulated success"""
     sys.argv = ['', '--test']
     Globals.getInstance().set_args(sys.argv)
@@ -164,8 +173,9 @@ def test_main_test_two_ports_success(patched_test_all, reset_globals, capsys):
 
 
 @pytest.mark.unit
+@pytest.mark.usefixtures("reset_globals")
 @patch('meshtastic.test.testAll', return_value=False)
-def test_main_test_two_ports_fails(patched_test_all, reset_globals, capsys):
+def test_main_test_two_ports_fails(patched_test_all, capsys):
     """Test --test two fake ports and testAll() is a simulated failure"""
     sys.argv = ['', '--test']
     Globals.getInstance().set_args(sys.argv)
@@ -181,7 +191,8 @@ def test_main_test_two_ports_fails(patched_test_all, reset_globals, capsys):
 
 
 @pytest.mark.unit
-def test_main_info(capsys, caplog, reset_globals):
+@pytest.mark.usefixtures("reset_globals")
+def test_main_info(capsys, caplog):
     """Test --info"""
     sys.argv = ['', '--info']
     Globals.getInstance().set_args(sys.argv)
@@ -201,8 +212,9 @@ def test_main_info(capsys, caplog, reset_globals):
 
 
 @pytest.mark.unit
+@pytest.mark.usefixtures("reset_globals")
 @patch('os.getlogin')
-def test_main_info_with_permission_error(patched_getlogin, capsys, caplog, reset_globals):
+def test_main_info_with_permission_error(patched_getlogin, capsys, caplog):
     """Test --info"""
     sys.argv = ['', '--info']
     Globals.getInstance().set_args(sys.argv)
@@ -224,7 +236,8 @@ def test_main_info_with_permission_error(patched_getlogin, capsys, caplog, reset
 
 
 @pytest.mark.unit
-def test_main_info_with_tcp_interface(capsys, reset_globals):
+@pytest.mark.usefixtures("reset_globals")
+def test_main_info_with_tcp_interface(capsys):
     """Test --info"""
     sys.argv = ['', '--info', '--host', 'meshtastic.local']
     Globals.getInstance().set_args(sys.argv)
@@ -244,7 +257,7 @@ def test_main_info_with_tcp_interface(capsys, reset_globals):
 
 # TODO: comment out ble (for now)
 #@pytest.mark.unit
-#def test_main_info_with_ble_interface(capsys, reset_globals):
+#def test_main_info_with_ble_interface(capsys):
 #    """Test --info"""
 #    sys.argv = ['', '--info', '--ble', 'foo']
 #    Globals.getInstance().set_args(sys.argv)
@@ -263,7 +276,8 @@ def test_main_info_with_tcp_interface(capsys, reset_globals):
 
 
 @pytest.mark.unit
-def test_main_no_proto(capsys, reset_globals):
+@pytest.mark.usefixtures("reset_globals")
+def test_main_no_proto(capsys):
     """Test --noproto (using --info for output)"""
     sys.argv = ['', '--info', '--noproto']
     Globals.getInstance().set_args(sys.argv)
@@ -275,6 +289,7 @@ def test_main_no_proto(capsys, reset_globals):
 
     # Override the time.sleep so there is no loop
     def my_sleep(amount):
+        print(f'amount:{amount}')
         sys.exit(0)
 
     with patch('meshtastic.serial_interface.SerialInterface', return_value=iface):
@@ -290,7 +305,8 @@ def test_main_no_proto(capsys, reset_globals):
 
 
 @pytest.mark.unit
-def test_main_info_with_seriallog_stdout(capsys, reset_globals):
+@pytest.mark.usefixtures("reset_globals")
+def test_main_info_with_seriallog_stdout(capsys):
     """Test --info"""
     sys.argv = ['', '--info', '--seriallog', 'stdout']
     Globals.getInstance().set_args(sys.argv)
@@ -309,7 +325,8 @@ def test_main_info_with_seriallog_stdout(capsys, reset_globals):
 
 
 @pytest.mark.unit
-def test_main_info_with_seriallog_output_txt(capsys, reset_globals):
+@pytest.mark.usefixtures("reset_globals")
+def test_main_info_with_seriallog_output_txt(capsys):
     """Test --info"""
     sys.argv = ['', '--info', '--seriallog', 'output.txt']
     Globals.getInstance().set_args(sys.argv)
@@ -330,7 +347,8 @@ def test_main_info_with_seriallog_output_txt(capsys, reset_globals):
 
 
 @pytest.mark.unit
-def test_main_qr(capsys, reset_globals):
+@pytest.mark.usefixtures("reset_globals")
+def test_main_qr(capsys):
     """Test --qr"""
     sys.argv = ['', '--qr']
     Globals.getInstance().set_args(sys.argv)
@@ -349,7 +367,8 @@ def test_main_qr(capsys, reset_globals):
 
 
 @pytest.mark.unit
-def test_main_nodes(capsys, reset_globals):
+@pytest.mark.usefixtures("reset_globals")
+def test_main_nodes(capsys):
     """Test --nodes"""
     sys.argv = ['', '--nodes']
     Globals.getInstance().set_args(sys.argv)
@@ -368,7 +387,8 @@ def test_main_nodes(capsys, reset_globals):
 
 
 @pytest.mark.unit
-def test_main_set_owner_to_bob(capsys, reset_globals):
+@pytest.mark.usefixtures("reset_globals")
+def test_main_set_owner_to_bob(capsys):
     """Test --set-owner bob"""
     sys.argv = ['', '--set-owner', 'bob']
     Globals.getInstance().set_args(sys.argv)
@@ -384,7 +404,8 @@ def test_main_set_owner_to_bob(capsys, reset_globals):
 
 
 @pytest.mark.unit
-def test_main_set_ham_to_KI123(capsys, reset_globals):
+@pytest.mark.usefixtures("reset_globals")
+def test_main_set_ham_to_KI123(capsys):
     """Test --set-ham KI123"""
     sys.argv = ['', '--set-ham', 'KI123']
     Globals.getInstance().set_args(sys.argv)
@@ -393,7 +414,7 @@ def test_main_set_ham_to_KI123(capsys, reset_globals):
     def mock_turnOffEncryptionOnPrimaryChannel():
         print('inside mocked turnOffEncryptionOnPrimaryChannel')
     def mock_setOwner(name, is_licensed):
-        print('inside mocked setOwner')
+        print(f'inside mocked setOwner name:{name} is_licensed:{is_licensed}')
     mocked_node.turnOffEncryptionOnPrimaryChannel.side_effect = mock_turnOffEncryptionOnPrimaryChannel
     mocked_node.setOwner.side_effect = mock_setOwner
 
@@ -412,7 +433,8 @@ def test_main_set_ham_to_KI123(capsys, reset_globals):
 
 
 @pytest.mark.unit
-def test_main_reboot(capsys, reset_globals):
+@pytest.mark.usefixtures("reset_globals")
+def test_main_reboot(capsys):
     """Test --reboot"""
     sys.argv = ['', '--reboot']
     Globals.getInstance().set_args(sys.argv)
@@ -435,7 +457,8 @@ def test_main_reboot(capsys, reset_globals):
 
 
 @pytest.mark.unit
-def test_main_sendtext(capsys, reset_globals):
+@pytest.mark.usefixtures("reset_globals")
+def test_main_sendtext(capsys):
     """Test --sendtext"""
     sys.argv = ['', '--sendtext', 'hello']
     Globals.getInstance().set_args(sys.argv)
@@ -443,6 +466,7 @@ def test_main_sendtext(capsys, reset_globals):
     iface = MagicMock(autospec=SerialInterface)
     def mock_sendText(text, dest, wantAck, channelIndex):
         print('inside mocked sendText')
+        print(f'{text} {dest} {wantAck} {channelIndex}')
     iface.sendText.side_effect = mock_sendText
 
     with patch('meshtastic.serial_interface.SerialInterface', return_value=iface) as mo:
@@ -456,7 +480,8 @@ def test_main_sendtext(capsys, reset_globals):
 
 
 @pytest.mark.unit
-def test_main_sendtext_with_channel(capsys, reset_globals):
+@pytest.mark.usefixtures("reset_globals")
+def test_main_sendtext_with_channel(capsys):
     """Test --sendtext"""
     sys.argv = ['', '--sendtext', 'hello', '--ch-index', '1']
     Globals.getInstance().set_args(sys.argv)
@@ -464,6 +489,7 @@ def test_main_sendtext_with_channel(capsys, reset_globals):
     iface = MagicMock(autospec=SerialInterface)
     def mock_sendText(text, dest, wantAck, channelIndex):
         print('inside mocked sendText')
+        print(f'{text} {dest} {wantAck} {channelIndex}')
     iface.sendText.side_effect = mock_sendText
 
     with patch('meshtastic.serial_interface.SerialInterface', return_value=iface) as mo:
@@ -478,7 +504,8 @@ def test_main_sendtext_with_channel(capsys, reset_globals):
 
 
 @pytest.mark.unit
-def test_main_sendtext_with_invalid_channel(caplog, capsys, reset_globals):
+@pytest.mark.usefixtures("reset_globals")
+def test_main_sendtext_with_invalid_channel(caplog, capsys):
     """Test --sendtext"""
     sys.argv = ['', '--sendtext', 'hello', '--ch-index', '-1']
     Globals.getInstance().set_args(sys.argv)
@@ -499,7 +526,8 @@ def test_main_sendtext_with_invalid_channel(caplog, capsys, reset_globals):
 
 
 @pytest.mark.unit
-def test_main_sendtext_with_invalid_channel_nine(caplog, capsys, reset_globals):
+@pytest.mark.usefixtures("reset_globals")
+def test_main_sendtext_with_invalid_channel_nine(caplog, capsys):
     """Test --sendtext"""
     sys.argv = ['', '--sendtext', 'hello', '--ch-index', '9']
     Globals.getInstance().set_args(sys.argv)
@@ -520,7 +548,8 @@ def test_main_sendtext_with_invalid_channel_nine(caplog, capsys, reset_globals):
 
 
 @pytest.mark.unit
-def test_main_sendtext_with_dest(capsys, caplog, reset_globals, iface_with_nodes):
+@pytest.mark.usefixtures("reset_globals")
+def test_main_sendtext_with_dest(capsys, caplog, iface_with_nodes):
     """Test --sendtext with --dest"""
     sys.argv = ['', '--sendtext', 'hello', '--dest', 'foo']
     Globals.getInstance().set_args(sys.argv)
@@ -546,7 +575,8 @@ def test_main_sendtext_with_dest(capsys, caplog, reset_globals, iface_with_nodes
 
 
 @pytest.mark.unit
-def test_main_sendping(capsys, reset_globals):
+@pytest.mark.usefixtures("reset_globals")
+def test_main_sendping(capsys):
     """Test --sendping"""
     sys.argv = ['', '--sendping']
     Globals.getInstance().set_args(sys.argv)
@@ -554,6 +584,7 @@ def test_main_sendping(capsys, reset_globals):
     iface = MagicMock(autospec=SerialInterface)
     def mock_sendData(payload, dest, portNum, wantAck, wantResponse):
         print('inside mocked sendData')
+        print(f'{payload} {dest} {portNum} {wantAck} {wantResponse}')
     iface.sendData.side_effect = mock_sendData
 
     with patch('meshtastic.serial_interface.SerialInterface', return_value=iface) as mo:
@@ -567,7 +598,8 @@ def test_main_sendping(capsys, reset_globals):
 
 
 @pytest.mark.unit
-def test_main_setlat(capsys, reset_globals):
+@pytest.mark.usefixtures("reset_globals")
+def test_main_setlat(capsys):
     """Test --sendlat"""
     sys.argv = ['', '--setlat', '37.5']
     Globals.getInstance().set_args(sys.argv)
@@ -580,6 +612,7 @@ def test_main_setlat(capsys, reset_globals):
     iface = MagicMock(autospec=SerialInterface)
     def mock_sendPosition(lat, lon, alt):
         print('inside mocked sendPosition')
+        print(f'{lat} {lon} {alt}')
     iface.sendPosition.side_effect = mock_sendPosition
     iface.localNode.return_value = mocked_node
 
@@ -596,7 +629,8 @@ def test_main_setlat(capsys, reset_globals):
 
 
 @pytest.mark.unit
-def test_main_setlon(capsys, reset_globals):
+@pytest.mark.usefixtures("reset_globals")
+def test_main_setlon(capsys):
     """Test --setlon"""
     sys.argv = ['', '--setlon', '-122.1']
     Globals.getInstance().set_args(sys.argv)
@@ -609,6 +643,7 @@ def test_main_setlon(capsys, reset_globals):
     iface = MagicMock(autospec=SerialInterface)
     def mock_sendPosition(lat, lon, alt):
         print('inside mocked sendPosition')
+        print(f'{lat} {lon} {alt}')
     iface.sendPosition.side_effect = mock_sendPosition
     iface.localNode.return_value = mocked_node
 
@@ -625,7 +660,8 @@ def test_main_setlon(capsys, reset_globals):
 
 
 @pytest.mark.unit
-def test_main_setalt(capsys, reset_globals):
+@pytest.mark.usefixtures("reset_globals")
+def test_main_setalt(capsys):
     """Test --setalt"""
     sys.argv = ['', '--setalt', '51']
     Globals.getInstance().set_args(sys.argv)
@@ -638,6 +674,7 @@ def test_main_setalt(capsys, reset_globals):
     iface = MagicMock(autospec=SerialInterface)
     def mock_sendPosition(lat, lon, alt):
         print('inside mocked sendPosition')
+        print(f'{lat} {lon} {alt}')
     iface.sendPosition.side_effect = mock_sendPosition
     iface.localNode.return_value = mocked_node
 
@@ -654,7 +691,8 @@ def test_main_setalt(capsys, reset_globals):
 
 
 @pytest.mark.unit
-def test_main_set_team_valid(capsys, reset_globals):
+@pytest.mark.usefixtures("reset_globals")
+def test_main_set_team_valid(capsys):
     """Test --set-team"""
     sys.argv = ['', '--set-team', 'CYAN']
     Globals.getInstance().set_args(sys.argv)
@@ -662,6 +700,7 @@ def test_main_set_team_valid(capsys, reset_globals):
     mocked_node = MagicMock(autospec=Node)
     def mock_setOwner(team):
         print('inside mocked setOwner')
+        print(f'{team}')
     mocked_node.setOwner.side_effect = mock_setOwner
 
     iface = MagicMock(autospec=SerialInterface)
@@ -682,7 +721,8 @@ def test_main_set_team_valid(capsys, reset_globals):
 
 
 @pytest.mark.unit
-def test_main_set_team_invalid(capsys, reset_globals):
+@pytest.mark.usefixtures("reset_globals")
+def test_main_set_team_invalid(capsys):
     """Test --set-team using an invalid team name"""
     sys.argv = ['', '--set-team', 'NOTCYAN']
     Globals.getInstance().set_args(sys.argv)
@@ -705,7 +745,8 @@ def test_main_set_team_invalid(capsys, reset_globals):
 
 
 @pytest.mark.unit
-def test_main_seturl(capsys, reset_globals):
+@pytest.mark.usefixtures("reset_globals")
+def test_main_seturl(capsys):
     """Test --seturl (url used below is what is generated after a factory_reset)"""
     sys.argv = ['', '--seturl', 'https://www.meshtastic.org/d/#CgUYAyIBAQ']
     Globals.getInstance().set_args(sys.argv)
@@ -720,7 +761,8 @@ def test_main_seturl(capsys, reset_globals):
 
 
 @pytest.mark.unit
-def test_main_set_valid(capsys, reset_globals):
+@pytest.mark.usefixtures("reset_globals")
+def test_main_set_valid(capsys):
     """Test --set with valid field"""
     sys.argv = ['', '--set', 'wifi_ssid', 'foo']
     Globals.getInstance().set_args(sys.argv)
@@ -740,7 +782,8 @@ def test_main_set_valid(capsys, reset_globals):
 
 
 @pytest.mark.unit
-def test_main_set_with_invalid(capsys, reset_globals):
+@pytest.mark.usefixtures("reset_globals")
+def test_main_set_with_invalid(capsys):
     """Test --set with invalid field"""
     sys.argv = ['', '--set', 'foo', 'foo']
     Globals.getInstance().set_args(sys.argv)
@@ -765,7 +808,8 @@ def test_main_set_with_invalid(capsys, reset_globals):
 
 # TODO: write some negative --configure tests
 @pytest.mark.unit
-def test_main_configure(capsys, reset_globals):
+@pytest.mark.usefixtures("reset_globals")
+def test_main_configure(capsys):
     """Test --configure with valid file"""
     sys.argv = ['', '--configure', 'example_config.yaml']
     Globals.getInstance().set_args(sys.argv)
@@ -790,7 +834,8 @@ def test_main_configure(capsys, reset_globals):
 
 
 @pytest.mark.unit
-def test_main_ch_add_valid(capsys, reset_globals):
+@pytest.mark.usefixtures("reset_globals")
+def test_main_ch_add_valid(capsys):
     """Test --ch-add with valid channel name, and that channel name does not already exist"""
     sys.argv = ['', '--ch-add', 'testing']
     Globals.getInstance().set_args(sys.argv)
@@ -817,7 +862,8 @@ def test_main_ch_add_valid(capsys, reset_globals):
 
 
 @pytest.mark.unit
-def test_main_ch_add_invalid_name_too_long(capsys, reset_globals):
+@pytest.mark.usefixtures("reset_globals")
+def test_main_ch_add_invalid_name_too_long(capsys):
     """Test --ch-add with invalid channel name, name too long"""
     sys.argv = ['', '--ch-add', 'testingtestingtesting']
     Globals.getInstance().set_args(sys.argv)
@@ -847,7 +893,8 @@ def test_main_ch_add_invalid_name_too_long(capsys, reset_globals):
 
 
 @pytest.mark.unit
-def test_main_ch_add_but_name_already_exists(capsys, reset_globals):
+@pytest.mark.usefixtures("reset_globals")
+def test_main_ch_add_but_name_already_exists(capsys):
     """Test --ch-add with a channel name that already exists"""
     sys.argv = ['', '--ch-add', 'testing']
     Globals.getInstance().set_args(sys.argv)
@@ -872,7 +919,8 @@ def test_main_ch_add_but_name_already_exists(capsys, reset_globals):
 
 
 @pytest.mark.unit
-def test_main_ch_add_but_no_more_channels(capsys, reset_globals):
+@pytest.mark.usefixtures("reset_globals")
+def test_main_ch_add_but_no_more_channels(capsys):
     """Test --ch-add with but there are no more channels"""
     sys.argv = ['', '--ch-add', 'testing']
     Globals.getInstance().set_args(sys.argv)
@@ -899,7 +947,8 @@ def test_main_ch_add_but_no_more_channels(capsys, reset_globals):
 
 
 @pytest.mark.unit
-def test_main_ch_del(capsys, reset_globals):
+@pytest.mark.usefixtures("reset_globals")
+def test_main_ch_del(capsys):
     """Test --ch-del with valid secondary channel to be deleted"""
     sys.argv = ['', '--ch-del', '--ch-index', '1']
     Globals.getInstance().set_args(sys.argv)
@@ -919,7 +968,8 @@ def test_main_ch_del(capsys, reset_globals):
 
 
 @pytest.mark.unit
-def test_main_ch_del_no_ch_index_specified(capsys, reset_globals):
+@pytest.mark.usefixtures("reset_globals")
+def test_main_ch_del_no_ch_index_specified(capsys):
     """Test --ch-del without a valid ch-index"""
     sys.argv = ['', '--ch-del']
     Globals.getInstance().set_args(sys.argv)
@@ -942,7 +992,8 @@ def test_main_ch_del_no_ch_index_specified(capsys, reset_globals):
 
 
 @pytest.mark.unit
-def test_main_ch_del_primary_channel(capsys, reset_globals):
+@pytest.mark.usefixtures("reset_globals")
+def test_main_ch_del_primary_channel(capsys):
     """Test --ch-del on ch-index=0"""
     sys.argv = ['', '--ch-del', '--ch-index', '0']
     Globals.getInstance().set_args(sys.argv)
@@ -966,7 +1017,8 @@ def test_main_ch_del_primary_channel(capsys, reset_globals):
 
 
 @pytest.mark.unit
-def test_main_ch_enable_valid_secondary_channel(capsys, reset_globals):
+@pytest.mark.usefixtures("reset_globals")
+def test_main_ch_enable_valid_secondary_channel(capsys):
     """Test --ch-enable with --ch-index"""
     sys.argv = ['', '--ch-enable', '--ch-index', '1']
     Globals.getInstance().set_args(sys.argv)
@@ -987,7 +1039,8 @@ def test_main_ch_enable_valid_secondary_channel(capsys, reset_globals):
 
 
 @pytest.mark.unit
-def test_main_ch_disable_valid_secondary_channel(capsys, reset_globals):
+@pytest.mark.usefixtures("reset_globals")
+def test_main_ch_disable_valid_secondary_channel(capsys):
     """Test --ch-disable with --ch-index"""
     sys.argv = ['', '--ch-disable', '--ch-index', '1']
     Globals.getInstance().set_args(sys.argv)
@@ -1008,7 +1061,8 @@ def test_main_ch_disable_valid_secondary_channel(capsys, reset_globals):
 
 
 @pytest.mark.unit
-def test_main_ch_enable_without_a_ch_index(capsys, reset_globals):
+@pytest.mark.usefixtures("reset_globals")
+def test_main_ch_enable_without_a_ch_index(capsys):
     """Test --ch-enable without --ch-index"""
     sys.argv = ['', '--ch-enable']
     Globals.getInstance().set_args(sys.argv)
@@ -1032,7 +1086,8 @@ def test_main_ch_enable_without_a_ch_index(capsys, reset_globals):
 
 
 @pytest.mark.unit
-def test_main_ch_enable_primary_channel(capsys, reset_globals):
+@pytest.mark.usefixtures("reset_globals")
+def test_main_ch_enable_primary_channel(capsys):
     """Test --ch-enable with --ch-index = 0"""
     sys.argv = ['', '--ch-enable', '--ch-index', '0']
     Globals.getInstance().set_args(sys.argv)
@@ -1056,7 +1111,8 @@ def test_main_ch_enable_primary_channel(capsys, reset_globals):
 
 
 @pytest.mark.unit
-def test_main_ch_range_options(capsys, reset_globals):
+@pytest.mark.usefixtures("reset_globals")
+def test_main_ch_range_options(capsys):
     """Test changing the various range options."""
     range_options = ['--ch-longslow', '--ch-longfast', '--ch-mediumslow',
                      '--ch-mediumfast', '--ch-shortslow', '--ch-shortfast']
@@ -1079,7 +1135,8 @@ def test_main_ch_range_options(capsys, reset_globals):
 
 
 @pytest.mark.unit
-def test_main_ch_longsfast_on_non_primary_channel(capsys, reset_globals):
+@pytest.mark.usefixtures("reset_globals")
+def test_main_ch_longsfast_on_non_primary_channel(capsys):
     """Test --ch-longfast --ch-index 1"""
     sys.argv = ['', '--ch-longfast', '--ch-index', '1']
     Globals.getInstance().set_args(sys.argv)
@@ -1116,7 +1173,8 @@ def test_main_ch_longsfast_on_non_primary_channel(capsys, reset_globals):
 # POS_TIMESTAMP 256
 
 @pytest.mark.unit
-def test_main_pos_fields_no_args(capsys, reset_globals):
+@pytest.mark.usefixtures("reset_globals")
+def test_main_pos_fields_no_args(capsys):
     """Test --pos-fields no args (which shows settings)"""
     sys.argv = ['', '--pos-fields']
     Globals.getInstance().set_args(sys.argv)
@@ -1146,7 +1204,8 @@ def test_main_pos_fields_no_args(capsys, reset_globals):
 
 
 @pytest.mark.unit
-def test_main_pos_fields_arg_of_zero(capsys, reset_globals):
+@pytest.mark.usefixtures("reset_globals")
+def test_main_pos_fields_arg_of_zero(capsys):
     """Test --pos-fields an arg of 0 (which shows list)"""
     sys.argv = ['', '--pos-fields', '0']
     Globals.getInstance().set_args(sys.argv)
@@ -1179,7 +1238,8 @@ def test_main_pos_fields_arg_of_zero(capsys, reset_globals):
 
 
 @pytest.mark.unit
-def test_main_pos_fields_valid_values(capsys, reset_globals):
+@pytest.mark.usefixtures("reset_globals")
+def test_main_pos_fields_valid_values(capsys):
     """Test --pos-fields with valid values"""
     sys.argv = ['', '--pos-fields', 'POS_GEO_SEP', 'POS_ALT_MSL']
     Globals.getInstance().set_args(sys.argv)
@@ -1205,7 +1265,8 @@ def test_main_pos_fields_valid_values(capsys, reset_globals):
 
 
 @pytest.mark.unit
-def test_main_get_with_valid_values(capsys, reset_globals):
+@pytest.mark.usefixtures("reset_globals")
+def test_main_get_with_valid_values(capsys):
     """Test --get with valid values (with string, number, boolean)"""
     sys.argv = ['', '--get', 'ls_secs', '--get', 'wifi_ssid', '--get', 'fixed_position']
     Globals.getInstance().set_args(sys.argv)
@@ -1229,7 +1290,8 @@ def test_main_get_with_valid_values(capsys, reset_globals):
 
 
 @pytest.mark.unit
-def test_main_get_with_invalid(capsys, reset_globals):
+@pytest.mark.usefixtures("reset_globals")
+def test_main_get_with_invalid(capsys):
     """Test --get with invalid field"""
     sys.argv = ['', '--get', 'foo']
     Globals.getInstance().set_args(sys.argv)
@@ -1254,7 +1316,8 @@ def test_main_get_with_invalid(capsys, reset_globals):
 
 
 @pytest.mark.unit
-def test_main_setchan(capsys, reset_globals):
+@pytest.mark.usefixtures("reset_globals")
+def test_main_setchan(capsys):
     """Test --setchan (deprecated)"""
     sys.argv = ['', '--setchan', 'a', 'b']
     Globals.getInstance().set_args(sys.argv)
@@ -1271,7 +1334,8 @@ def test_main_setchan(capsys, reset_globals):
 
 
 @pytest.mark.unit
-def test_main_onReceive_empty(caplog, reset_globals, capsys):
+@pytest.mark.usefixtures("reset_globals")
+def test_main_onReceive_empty(caplog, capsys):
     """Test onReceive"""
     args = MagicMock()
     Globals.getInstance().set_args(args)
@@ -1297,7 +1361,8 @@ def test_main_onReceive_empty(caplog, reset_globals, capsys):
 #            }
 
 @pytest.mark.unit
-def test_main_onReceive_with_sendtext(caplog, capsys, reset_globals):
+@pytest.mark.usefixtures("reset_globals")
+def test_main_onReceive_with_sendtext(caplog, capsys):
     """Test onReceive with sendtext
        The entire point of this test is to make sure the interface.close() call
        is made in onReceive().
@@ -1332,7 +1397,8 @@ def test_main_onReceive_with_sendtext(caplog, capsys, reset_globals):
 
 
 @pytest.mark.unit
-def test_main_onReceive_with_text(caplog, capsys, reset_globals):
+@pytest.mark.usefixtures("reset_globals")
+def test_main_onReceive_with_text(caplog, capsys):
     """Test onReceive with text
     """
     args = MagicMock()
@@ -1371,7 +1437,8 @@ def test_main_onReceive_with_text(caplog, capsys, reset_globals):
 
 
 @pytest.mark.unit
-def test_main_onConnection(reset_globals, capsys):
+@pytest.mark.usefixtures("reset_globals")
+def test_main_onConnection(capsys):
     """Test onConnection"""
     sys.argv = ['']
     Globals.getInstance().set_args(sys.argv)
@@ -1389,7 +1456,8 @@ def test_main_onConnection(reset_globals, capsys):
 
 
 @pytest.mark.unit
-def test_main_export_config(reset_globals, capsys):
+@pytest.mark.usefixtures("reset_globals")
+def test_main_export_config(capsys):
     """Test export_config() function directly"""
     iface = MagicMock(autospec=SerialInterface)
     with patch('meshtastic.serial_interface.SerialInterface', return_value=iface) as mo:
@@ -1425,7 +1493,8 @@ position_flags: 35"""
 
 
 @pytest.mark.unit
-def test_main_export_config_called_from_main(capsys, reset_globals):
+@pytest.mark.usefixtures("reset_globals")
+def test_main_export_config_called_from_main(capsys):
     """Test --export-config"""
     sys.argv = ['', '--export-config']
     Globals.getInstance().set_args(sys.argv)
@@ -1441,7 +1510,8 @@ def test_main_export_config_called_from_main(capsys, reset_globals):
 
 
 @pytest.mark.unit
-def test_main_gpio_rd_no_gpio_channel(capsys, reset_globals):
+@pytest.mark.usefixtures("reset_globals")
+def test_main_gpio_rd_no_gpio_channel(capsys):
     """Test --gpio_rd with no named gpio channel"""
     sys.argv = ['', '--gpio-rd', '0x10', '--dest', '!foo']
     Globals.getInstance().set_args(sys.argv)
@@ -1459,7 +1529,8 @@ def test_main_gpio_rd_no_gpio_channel(capsys, reset_globals):
 
 
 @pytest.mark.unit
-def test_main_gpio_rd_no_dest(capsys, reset_globals):
+@pytest.mark.usefixtures("reset_globals")
+def test_main_gpio_rd_no_dest(capsys):
     """Test --gpio_rd with a named gpio channel but no dest was specified"""
     sys.argv = ['', '--gpio-rd', '0x2000']
     Globals.getInstance().set_args(sys.argv)
@@ -1481,7 +1552,8 @@ def test_main_gpio_rd_no_dest(capsys, reset_globals):
 
 
 @pytest.mark.unit
-def test_main_gpio_rd(caplog, capsys, reset_globals):
+@pytest.mark.usefixtures("reset_globals")
+def test_main_gpio_rd(caplog, capsys):
     """Test --gpio_rd with a named gpio channel"""
     # Note: On the Heltec v2.1, there is a GPIO pin GPIO 13 that does not have a
     # red arrow (meaning ok to use for our purposes)
@@ -1538,7 +1610,8 @@ def test_main_gpio_rd(caplog, capsys, reset_globals):
 
 
 @pytest.mark.unit
-def test_main_getPref_valid_field(capsys, reset_globals):
+@pytest.mark.usefixtures("reset_globals")
+def test_main_getPref_valid_field(capsys):
     """Test getPref() with a valid field"""
     prefs = MagicMock()
     prefs.DESCRIPTOR.fields_by_name.get.return_value = 'ls_secs'
@@ -1553,7 +1626,8 @@ def test_main_getPref_valid_field(capsys, reset_globals):
 
 
 @pytest.mark.unit
-def test_main_getPref_valid_field_string(capsys, reset_globals):
+@pytest.mark.usefixtures("reset_globals")
+def test_main_getPref_valid_field_string(capsys):
     """Test getPref() with a valid field and value as a string"""
     prefs = MagicMock()
     prefs.DESCRIPTOR.fields_by_name.get.return_value = 'wifi_ssid'
@@ -1568,7 +1642,8 @@ def test_main_getPref_valid_field_string(capsys, reset_globals):
 
 
 @pytest.mark.unit
-def test_main_getPref_valid_field_bool(capsys, reset_globals):
+@pytest.mark.usefixtures("reset_globals")
+def test_main_getPref_valid_field_bool(capsys):
     """Test getPref() with a valid field and value as a bool"""
     prefs = MagicMock()
     prefs.DESCRIPTOR.fields_by_name.get.return_value = 'fixed_position'
@@ -1583,7 +1658,8 @@ def test_main_getPref_valid_field_bool(capsys, reset_globals):
 
 
 @pytest.mark.unit
-def test_main_getPref_invalid_field(capsys, reset_globals):
+@pytest.mark.usefixtures("reset_globals")
+def test_main_getPref_invalid_field(capsys):
     """Test getPref() with an invalid field"""
 
     class Field:
@@ -1614,7 +1690,8 @@ def test_main_getPref_invalid_field(capsys, reset_globals):
 
 
 @pytest.mark.unit
-def test_main_setPref_valid_field_int(capsys, reset_globals):
+@pytest.mark.usefixtures("reset_globals")
+def test_main_setPref_valid_field_int(capsys):
     """Test setPref() with a valid field"""
 
     class Field:
@@ -1636,7 +1713,8 @@ def test_main_setPref_valid_field_int(capsys, reset_globals):
 
 
 @pytest.mark.unit
-def test_main_setPref_invalid_field(capsys, reset_globals):
+@pytest.mark.usefixtures("reset_globals")
+def test_main_setPref_invalid_field(capsys):
     """Test setPref() with a invalid field"""
 
 
@@ -1667,7 +1745,8 @@ def test_main_setPref_invalid_field(capsys, reset_globals):
 
 
 @pytest.mark.unit
-def test_main_ch_set_psk_no_ch_index(capsys, reset_globals):
+@pytest.mark.usefixtures("reset_globals")
+def test_main_ch_set_psk_no_ch_index(capsys):
     """Test --ch-set psk """
     sys.argv = ['', '--ch-set', 'psk', 'foo', '--host', 'meshtastic.local']
     Globals.getInstance().set_args(sys.argv)
@@ -1686,7 +1765,8 @@ def test_main_ch_set_psk_no_ch_index(capsys, reset_globals):
 
 
 @pytest.mark.unit
-def test_main_ch_set_psk_with_ch_index(capsys, reset_globals):
+@pytest.mark.usefixtures("reset_globals")
+def test_main_ch_set_psk_with_ch_index(capsys):
     """Test --ch-set psk """
     sys.argv = ['', '--ch-set', 'psk', 'foo', '--host', 'meshtastic.local', '--ch-index', '0']
     Globals.getInstance().set_args(sys.argv)
@@ -1702,7 +1782,8 @@ def test_main_ch_set_psk_with_ch_index(capsys, reset_globals):
 
 
 @pytest.mark.unit
-def test_main_ch_set_name_with_ch_index(capsys, reset_globals):
+@pytest.mark.usefixtures("reset_globals")
+def test_main_ch_set_name_with_ch_index(capsys):
     """Test --ch-set setting other than psk"""
     sys.argv = ['', '--ch-set', 'name', 'foo', '--host', 'meshtastic.local', '--ch-index', '0']
     Globals.getInstance().set_args(sys.argv)
@@ -1719,7 +1800,8 @@ def test_main_ch_set_name_with_ch_index(capsys, reset_globals):
 
 
 @pytest.mark.unit
-def test_onNode(capsys, reset_globals):
+@pytest.mark.usefixtures("reset_globals")
+def test_onNode(capsys):
     """Test onNode"""
     onNode('foo')
     out, err = capsys.readouterr()
@@ -1728,7 +1810,8 @@ def test_onNode(capsys, reset_globals):
 
 
 @pytest.mark.unit
-def test_tunnel_no_args(capsys, reset_globals):
+@pytest.mark.usefixtures("reset_globals")
+def test_tunnel_no_args(capsys):
     """Test tunnel no arguments"""
     sys.argv = ['']
     Globals.getInstance().set_args(sys.argv)
@@ -1741,9 +1824,10 @@ def test_tunnel_no_args(capsys, reset_globals):
 
 
 @pytest.mark.unit
+@pytest.mark.usefixtures("reset_globals")
 @patch('meshtastic.util.findPorts', return_value=[])
 @patch('platform.system')
-def test_tunnel_tunnel_arg_with_no_devices(mock_platform_system, patched_find_ports, caplog, capsys, reset_globals):
+def test_tunnel_tunnel_arg_with_no_devices(mock_platform_system, caplog, capsys):
     """Test tunnel with tunnel arg (act like we are on a linux system)"""
     a_mock = MagicMock()
     a_mock.return_value = 'Linux'
@@ -1763,9 +1847,10 @@ def test_tunnel_tunnel_arg_with_no_devices(mock_platform_system, patched_find_po
 
 
 @pytest.mark.unit
+@pytest.mark.usefixtures("reset_globals")
 @patch('meshtastic.util.findPorts', return_value=[])
 @patch('platform.system')
-def test_tunnel_subnet_arg_with_no_devices(mock_platform_system, patched_find_ports, caplog, capsys, reset_globals):
+def test_tunnel_subnet_arg_with_no_devices(mock_platform_system, caplog, capsys):
     """Test tunnel with subnet arg (act like we are on a linux system)"""
     a_mock = MagicMock()
     a_mock.return_value = 'Linux'
@@ -1785,11 +1870,13 @@ def test_tunnel_subnet_arg_with_no_devices(mock_platform_system, patched_find_po
 
 
 @pytest.mark.unit
+@pytest.mark.usefixtures("reset_globals")
 @patch('platform.system')
-def test_tunnel_tunnel_arg(mock_platform_system, caplog, reset_globals, iface_with_nodes, capsys):
+def test_tunnel_tunnel_arg(mock_platform_system, caplog, iface_with_nodes, capsys):
     """Test tunnel with tunnel arg (act like we are on a linux system)"""
     # Override the time.sleep so there is no loop
     def my_sleep(amount):
+        print(f'{amount}')
         sys.exit(3)
 
     a_mock = MagicMock()
diff --git a/meshtastic/tests/test_mesh_interface.py b/meshtastic/tests/test_mesh_interface.py
index b0b9d76..3a703df 100644
--- a/meshtastic/tests/test_mesh_interface.py
+++ b/meshtastic/tests/test_mesh_interface.py
@@ -15,7 +15,8 @@ from ..util import Timeout
 
 
 @pytest.mark.unit
-def test_MeshInterface(capsys, reset_globals):
+@pytest.mark.usefixtures("reset_globals")
+def test_MeshInterface(capsys):
     """Test that we can instantiate a MeshInterface"""
     iface = MeshInterface(noProto=True)
     anode = Node('foo', 'bar')
@@ -56,7 +57,8 @@ def test_MeshInterface(capsys, reset_globals):
 
 
 @pytest.mark.unit
-def test_getMyUser(reset_globals, iface_with_nodes):
+@pytest.mark.usefixtures("reset_globals")
+def test_getMyUser(iface_with_nodes):
     """Test getMyUser()"""
     iface = iface_with_nodes
     iface.myInfo.my_node_num = 2475227164
@@ -66,7 +68,8 @@ def test_getMyUser(reset_globals, iface_with_nodes):
 
 
 @pytest.mark.unit
-def test_getLongName(reset_globals, iface_with_nodes):
+@pytest.mark.usefixtures("reset_globals")
+def test_getLongName(iface_with_nodes):
     """Test getLongName()"""
     iface = iface_with_nodes
     iface.myInfo.my_node_num = 2475227164
@@ -75,7 +78,8 @@ def test_getLongName(reset_globals, iface_with_nodes):
 
 
 @pytest.mark.unit
-def test_getShortName(reset_globals, iface_with_nodes):
+@pytest.mark.usefixtures("reset_globals")
+def test_getShortName(iface_with_nodes):
     """Test getShortName()."""
     iface = iface_with_nodes
     iface.myInfo.my_node_num = 2475227164
@@ -84,7 +88,8 @@ def test_getShortName(reset_globals, iface_with_nodes):
 
 
 @pytest.mark.unit
-def test_handlePacketFromRadio_no_from(capsys, reset_globals):
+@pytest.mark.usefixtures("reset_globals")
+def test_handlePacketFromRadio_no_from(capsys):
     """Test _handlePacketFromRadio with no 'from' in the mesh packet."""
     iface = MeshInterface(noProto=True)
     meshPacket = mesh_pb2.MeshPacket()
@@ -95,7 +100,8 @@ def test_handlePacketFromRadio_no_from(capsys, reset_globals):
 
 
 @pytest.mark.unit
-def test_handlePacketFromRadio_with_a_portnum(caplog, reset_globals):
+@pytest.mark.usefixtures("reset_globals")
+def test_handlePacketFromRadio_with_a_portnum(caplog):
     """Test _handlePacketFromRadio with a portnum
        Since we have an attribute called 'from', we cannot simply 'set' it.
        Had to implement a hack just to be able to test some code.
@@ -110,7 +116,8 @@ def test_handlePacketFromRadio_with_a_portnum(caplog, reset_globals):
 
 
 @pytest.mark.unit
-def test_handlePacketFromRadio_no_portnum(caplog, reset_globals):
+@pytest.mark.usefixtures("reset_globals")
+def test_handlePacketFromRadio_no_portnum(caplog):
     """Test _handlePacketFromRadio without a portnum"""
     iface = MeshInterface(noProto=True)
     meshPacket = mesh_pb2.MeshPacket()
@@ -121,7 +128,8 @@ def test_handlePacketFromRadio_no_portnum(caplog, reset_globals):
 
 
 @pytest.mark.unit
-def test_getNode_with_local(reset_globals):
+@pytest.mark.usefixtures("reset_globals")
+def test_getNode_with_local():
     """Test getNode"""
     iface = MeshInterface(noProto=True)
     anode = iface.getNode(LOCAL_ADDR)
@@ -129,7 +137,8 @@ def test_getNode_with_local(reset_globals):
 
 
 @pytest.mark.unit
-def test_getNode_not_local(reset_globals, caplog):
+@pytest.mark.usefixtures("reset_globals")
+def test_getNode_not_local(caplog):
     """Test getNode not local"""
     iface = MeshInterface(noProto=True)
     anode = MagicMock(autospec=Node)
@@ -141,7 +150,8 @@ def test_getNode_not_local(reset_globals, caplog):
 
 
 @pytest.mark.unit
-def test_getNode_not_local_timeout(reset_globals, capsys):
+@pytest.mark.usefixtures("reset_globals")
+def test_getNode_not_local_timeout(capsys):
     """Test getNode not local, simulate timeout"""
     iface = MeshInterface(noProto=True)
     anode = MagicMock(autospec=Node)
@@ -157,7 +167,8 @@ def test_getNode_not_local_timeout(reset_globals, capsys):
 
 
 @pytest.mark.unit
-def test_sendPosition(reset_globals, caplog):
+@pytest.mark.usefixtures("reset_globals")
+def test_sendPosition(caplog):
     """Test sendPosition"""
     iface = MeshInterface(noProto=True)
     with caplog.at_level(logging.DEBUG):
@@ -167,7 +178,8 @@ def test_sendPosition(reset_globals, caplog):
 
 
 @pytest.mark.unit
-def test_close_with_heartbeatTimer(reset_globals, caplog):
+@pytest.mark.usefixtures("reset_globals")
+def test_close_with_heartbeatTimer(caplog):
     """Test close() with heartbeatTimer"""
     iface = MeshInterface(noProto=True)
     anode = Node('foo', 'bar')
@@ -183,7 +195,8 @@ def test_close_with_heartbeatTimer(reset_globals, caplog):
 
 
 @pytest.mark.unit
-def test_handleFromRadio_empty_payload(reset_globals, caplog):
+@pytest.mark.usefixtures("reset_globals")
+def test_handleFromRadio_empty_payload(caplog):
     """Test _handleFromRadio"""
     iface = MeshInterface(noProto=True)
     with caplog.at_level(logging.DEBUG):
@@ -193,7 +206,8 @@ def test_handleFromRadio_empty_payload(reset_globals, caplog):
 
 
 @pytest.mark.unit
-def test_handleFromRadio_with_my_info(reset_globals, caplog):
+@pytest.mark.usefixtures("reset_globals")
+def test_handleFromRadio_with_my_info(caplog):
     """Test _handleFromRadio with my_info"""
     # Note: I captured the '--debug --info' for the bytes below.
     # It "translates" to this:
@@ -218,7 +232,8 @@ def test_handleFromRadio_with_my_info(reset_globals, caplog):
 
 
 @pytest.mark.unit
-def test_handleFromRadio_with_node_info(reset_globals, caplog, capsys):
+@pytest.mark.usefixtures("reset_globals")
+def test_handleFromRadio_with_node_info(caplog, capsys):
     """Test _handleFromRadio with node_info"""
     # Note: I captured the '--debug --info' for the bytes below.
     # It "translates" to this:
@@ -254,7 +269,8 @@ def test_handleFromRadio_with_node_info(reset_globals, caplog, capsys):
 
 
 @pytest.mark.unit
-def test_handleFromRadio_with_node_info_tbeam1(reset_globals, caplog, capsys):
+@pytest.mark.usefixtures("reset_globals")
+def test_handleFromRadio_with_node_info_tbeam1(caplog, capsys):
     """Test _handleFromRadio with node_info"""
     # Note: Captured the '--debug --info' for the bytes below.
     # pylint: disable=C0301
@@ -277,7 +293,8 @@ def test_handleFromRadio_with_node_info_tbeam1(reset_globals, caplog, capsys):
 
 
 @pytest.mark.unit
-def test_handleFromRadio_with_node_info_tbeam_with_bad_data(reset_globals, caplog, capsys):
+@pytest.mark.usefixtures("reset_globals")
+def test_handleFromRadio_with_node_info_tbeam_with_bad_data(caplog):
     """Test _handleFromRadio with node_info with some bad data (issue#172) - ensure we do not throw exception"""
     # Note: Captured the '--debug --info' for the bytes below.
     from_radio_bytes = b'"\x17\x08\xdc\x8a\x8a\xae\x02\x12\x08"\x06\x00\x00\x00\x00\x00\x00\x1a\x00=\x00\x00\xb8@'
@@ -288,7 +305,8 @@ def test_handleFromRadio_with_node_info_tbeam_with_bad_data(reset_globals, caplo
 
 
 @pytest.mark.unit
-def test_MeshInterface_sendToRadioImpl(caplog, reset_globals):
+@pytest.mark.usefixtures("reset_globals")
+def test_MeshInterface_sendToRadioImpl(caplog):
     """Test _sendToRadioImp()"""
     iface = MeshInterface(noProto=True)
     with caplog.at_level(logging.DEBUG):
@@ -298,7 +316,8 @@ def test_MeshInterface_sendToRadioImpl(caplog, reset_globals):
 
 
 @pytest.mark.unit
-def test_MeshInterface_sendToRadio_no_proto(caplog, reset_globals):
+@pytest.mark.usefixtures("reset_globals")
+def test_MeshInterface_sendToRadio_no_proto(caplog):
     """Test sendToRadio()"""
     iface = MeshInterface()
     with caplog.at_level(logging.DEBUG):
@@ -308,7 +327,8 @@ def test_MeshInterface_sendToRadio_no_proto(caplog, reset_globals):
 
 
 @pytest.mark.unit
-def test_sendData_too_long(caplog, reset_globals):
+@pytest.mark.usefixtures("reset_globals")
+def test_sendData_too_long(caplog):
     """Test when data payload is too big"""
     iface = MeshInterface(noProto=True)
     some_large_text = b'This is a long text that will be too long for send text.'
@@ -332,7 +352,8 @@ def test_sendData_too_long(caplog, reset_globals):
 
 
 @pytest.mark.unit
-def test_sendData_unknown_app(capsys, reset_globals):
+@pytest.mark.usefixtures("reset_globals")
+def test_sendData_unknown_app(capsys):
     """Test sendData when unknown app"""
     iface = MeshInterface(noProto=True)
     with pytest.raises(SystemExit) as pytest_wrapped_e:
@@ -345,7 +366,8 @@ def test_sendData_unknown_app(capsys, reset_globals):
 
 
 @pytest.mark.unit
-def test_sendPosition_with_a_position(caplog, reset_globals):
+@pytest.mark.usefixtures("reset_globals")
+def test_sendPosition_with_a_position(caplog):
     """Test sendPosition when lat/long/alt"""
     iface = MeshInterface(noProto=True)
     with caplog.at_level(logging.DEBUG):
@@ -356,7 +378,8 @@ def test_sendPosition_with_a_position(caplog, reset_globals):
 
 
 @pytest.mark.unit
-def test_sendPacket_with_no_destination(capsys, reset_globals):
+@pytest.mark.usefixtures("reset_globals")
+def test_sendPacket_with_no_destination(capsys):
     """Test _sendPacket()"""
     iface = MeshInterface(noProto=True)
     with pytest.raises(SystemExit) as pytest_wrapped_e:
@@ -369,7 +392,8 @@ def test_sendPacket_with_no_destination(capsys, reset_globals):
 
 
 @pytest.mark.unit
-def test_sendPacket_with_destination_as_int(caplog, reset_globals):
+@pytest.mark.usefixtures("reset_globals")
+def test_sendPacket_with_destination_as_int(caplog):
     """Test _sendPacket() with int as a destination"""
     iface = MeshInterface(noProto=True)
     with caplog.at_level(logging.DEBUG):
@@ -379,7 +403,8 @@ def test_sendPacket_with_destination_as_int(caplog, reset_globals):
 
 
 @pytest.mark.unit
-def test_sendPacket_with_destination_starting_with_a_bang(caplog, reset_globals):
+@pytest.mark.usefixtures("reset_globals")
+def test_sendPacket_with_destination_starting_with_a_bang(caplog):
     """Test _sendPacket() with int as a destination"""
     iface = MeshInterface(noProto=True)
     with caplog.at_level(logging.DEBUG):
@@ -389,7 +414,8 @@ def test_sendPacket_with_destination_starting_with_a_bang(caplog, reset_globals)
 
 
 @pytest.mark.unit
-def test_sendPacket_with_destination_as_BROADCAST_ADDR(caplog, reset_globals):
+@pytest.mark.usefixtures("reset_globals")
+def test_sendPacket_with_destination_as_BROADCAST_ADDR(caplog):
     """Test _sendPacket() with BROADCAST_ADDR as a destination"""
     iface = MeshInterface(noProto=True)
     with caplog.at_level(logging.DEBUG):
@@ -399,7 +425,8 @@ def test_sendPacket_with_destination_as_BROADCAST_ADDR(caplog, reset_globals):
 
 
 @pytest.mark.unit
-def test_sendPacket_with_destination_as_LOCAL_ADDR_no_myInfo(capsys, reset_globals):
+@pytest.mark.usefixtures("reset_globals")
+def test_sendPacket_with_destination_as_LOCAL_ADDR_no_myInfo(capsys):
     """Test _sendPacket() with LOCAL_ADDR as a destination with no myInfo"""
     iface = MeshInterface(noProto=True)
     with pytest.raises(SystemExit) as pytest_wrapped_e:
@@ -413,7 +440,8 @@ def test_sendPacket_with_destination_as_LOCAL_ADDR_no_myInfo(capsys, reset_globa
 
 
 @pytest.mark.unit
-def test_sendPacket_with_destination_as_LOCAL_ADDR_with_myInfo(caplog, reset_globals):
+@pytest.mark.usefixtures("reset_globals")
+def test_sendPacket_with_destination_as_LOCAL_ADDR_with_myInfo(caplog):
     """Test _sendPacket() with LOCAL_ADDR as a destination with myInfo"""
     iface = MeshInterface(noProto=True)
     myInfo = MagicMock()
@@ -426,7 +454,8 @@ def test_sendPacket_with_destination_as_LOCAL_ADDR_with_myInfo(caplog, reset_glo
 
 
 @pytest.mark.unit
-def test_sendPacket_with_destination_is_blank_with_nodes(capsys, reset_globals, iface_with_nodes):
+@pytest.mark.usefixtures("reset_globals")
+def test_sendPacket_with_destination_is_blank_with_nodes(capsys, iface_with_nodes):
     """Test _sendPacket() with '' as a destination with myInfo"""
     iface = iface_with_nodes
     meshPacket = mesh_pb2.MeshPacket()
@@ -440,7 +469,8 @@ def test_sendPacket_with_destination_is_blank_with_nodes(capsys, reset_globals,
 
 
 @pytest.mark.unit
-def test_sendPacket_with_destination_is_blank_without_nodes(caplog, reset_globals, iface_with_nodes):
+@pytest.mark.usefixtures("reset_globals")
+def test_sendPacket_with_destination_is_blank_without_nodes(caplog, iface_with_nodes):
     """Test _sendPacket() with '' as a destination with myInfo"""
     iface = iface_with_nodes
     iface.nodes = None
@@ -451,7 +481,8 @@ def test_sendPacket_with_destination_is_blank_without_nodes(caplog, reset_global
 
 
 @pytest.mark.unit
-def test_getMyNodeInfo(reset_globals):
+@pytest.mark.usefixtures("reset_globals")
+def test_getMyNodeInfo():
     """Test getMyNodeInfo()"""
     iface = MeshInterface(noProto=True)
     anode = iface.getNode(LOCAL_ADDR)
@@ -465,7 +496,8 @@ def test_getMyNodeInfo(reset_globals):
 
 
 @pytest.mark.unit
-def test_generatePacketId(capsys, reset_globals):
+@pytest.mark.usefixtures("reset_globals")
+def test_generatePacketId(capsys):
     """Test _generatePacketId() when no currentPacketId (not connected)"""
     iface = MeshInterface(noProto=True)
     # not sure when this condition would ever happen... but we can simulate it
@@ -480,7 +512,8 @@ def test_generatePacketId(capsys, reset_globals):
 
 
 @pytest.mark.unit
-def test_fixupPosition_empty_pos(capsys, reset_globals):
+@pytest.mark.usefixtures("reset_globals")
+def test_fixupPosition_empty_pos():
     """Test _fixupPosition()"""
     iface = MeshInterface(noProto=True)
     pos = {}
@@ -489,7 +522,8 @@ def test_fixupPosition_empty_pos(capsys, reset_globals):
 
 
 @pytest.mark.unit
-def test_fixupPosition_no_changes_needed(capsys, reset_globals):
+@pytest.mark.usefixtures("reset_globals")
+def test_fixupPosition_no_changes_needed():
     """Test _fixupPosition()"""
     iface = MeshInterface(noProto=True)
     pos = {"latitude": 101, "longitude": 102}
@@ -498,7 +532,8 @@ def test_fixupPosition_no_changes_needed(capsys, reset_globals):
 
 
 @pytest.mark.unit
-def test_fixupPosition(capsys, reset_globals):
+@pytest.mark.usefixtures("reset_globals")
+def test_fixupPosition():
     """Test _fixupPosition()"""
     iface = MeshInterface(noProto=True)
     pos = {"latitudeI": 1010000000, "longitudeI": 1020000000}
@@ -510,7 +545,8 @@ def test_fixupPosition(capsys, reset_globals):
 
 
 @pytest.mark.unit
-def test_nodeNumToId(capsys, reset_globals, iface_with_nodes):
+@pytest.mark.usefixtures("reset_globals")
+def test_nodeNumToId(iface_with_nodes):
     """Test _nodeNumToId()"""
     iface = iface_with_nodes
     iface.myInfo.my_node_num = 2475227164
@@ -519,7 +555,8 @@ def test_nodeNumToId(capsys, reset_globals, iface_with_nodes):
 
 
 @pytest.mark.unit
-def test_nodeNumToId_not_found(capsys, reset_globals, iface_with_nodes):
+@pytest.mark.usefixtures("reset_globals")
+def test_nodeNumToId_not_found(iface_with_nodes):
     """Test _nodeNumToId()"""
     iface = iface_with_nodes
     iface.myInfo.my_node_num = 2475227164
@@ -528,7 +565,8 @@ def test_nodeNumToId_not_found(capsys, reset_globals, iface_with_nodes):
 
 
 @pytest.mark.unit
-def test_nodeNumToId_to_all(capsys, reset_globals, iface_with_nodes):
+@pytest.mark.usefixtures("reset_globals")
+def test_nodeNumToId_to_all(iface_with_nodes):
     """Test _nodeNumToId()"""
     iface = iface_with_nodes
     iface.myInfo.my_node_num = 2475227164
@@ -537,7 +575,8 @@ def test_nodeNumToId_to_all(capsys, reset_globals, iface_with_nodes):
 
 
 @pytest.mark.unit
-def test_getOrCreateByNum_minimal(capsys, reset_globals, iface_with_nodes):
+@pytest.mark.usefixtures("reset_globals")
+def test_getOrCreateByNum_minimal(iface_with_nodes):
     """Test _getOrCreateByNum()"""
     iface = iface_with_nodes
     iface.myInfo.my_node_num = 2475227164
@@ -546,7 +585,8 @@ def test_getOrCreateByNum_minimal(capsys, reset_globals, iface_with_nodes):
 
 
 @pytest.mark.unit
-def test_getOrCreateByNum_not_found(capsys, reset_globals, iface_with_nodes):
+@pytest.mark.usefixtures("reset_globals")
+def test_getOrCreateByNum_not_found(iface_with_nodes):
     """Test _getOrCreateByNum()"""
     iface = iface_with_nodes
     iface.myInfo.my_node_num = 2475227164
@@ -556,7 +596,8 @@ def test_getOrCreateByNum_not_found(capsys, reset_globals, iface_with_nodes):
 
 
 @pytest.mark.unit
-def test_getOrCreateByNum(capsys, reset_globals, iface_with_nodes):
+@pytest.mark.usefixtures("reset_globals")
+def test_getOrCreateByNum(iface_with_nodes):
     """Test _getOrCreateByNum()"""
     iface = iface_with_nodes
     iface.myInfo.my_node_num = 2475227164
@@ -582,7 +623,7 @@ def test_exit_with_exception(caplog):
 
 
 @pytest.mark.unit
-def test_showNodes_exclude_self(capsys, caplog, reset_globals, iface_with_nodes):
+def test_showNodes_exclude_self(capsys, caplog, iface_with_nodes):
     """Test that we hit that continue statement"""
     with caplog.at_level(logging.DEBUG):
         iface = iface_with_nodes
@@ -593,7 +634,7 @@ def test_showNodes_exclude_self(capsys, caplog, reset_globals, iface_with_nodes)
 
 
 @pytest.mark.unitslow
-def test_waitForConfig(caplog, capsys):
+def test_waitForConfig(capsys):
     """Test waitForConfig()"""
     iface = MeshInterface(noProto=True)
     # override how long to wait
@@ -607,7 +648,7 @@ def test_waitForConfig(caplog, capsys):
 
 
 @pytest.mark.unit
-def test_waitConnected_raises_an_exception(caplog, capsys):
+def test_waitConnected_raises_an_exception(capsys):
     """Test waitConnected()"""
     iface = MeshInterface(noProto=True)
     with pytest.raises(Exception) as pytest_wrapped_e:
@@ -620,7 +661,7 @@ def test_waitConnected_raises_an_exception(caplog, capsys):
 
 
 @pytest.mark.unit
-def test_waitConnected_isConnected_timeout(caplog, capsys):
+def test_waitConnected_isConnected_timeout(capsys):
     """Test waitConnected()"""
     with pytest.raises(Exception) as pytest_wrapped_e:
         iface = MeshInterface()
diff --git a/meshtastic/tests/test_node.py b/meshtastic/tests/test_node.py
index bac95c4..96c2b71 100644
--- a/meshtastic/tests/test_node.py
+++ b/meshtastic/tests/test_node.py
@@ -150,7 +150,7 @@ def test_setURL_valid_URL(caplog):
 
 
 @pytest.mark.unit
-def test_setURL_valid_URL_but_no_settings(caplog, capsys):
+def test_setURL_valid_URL_but_no_settings(capsys):
     """Test setURL"""
     iface = MagicMock(autospec=SerialInterface)
     url = "https://www.meshtastic.org/d/#"
@@ -430,7 +430,7 @@ def test_deleteChannel_secondary_with_admin_channel_before_testing():
 
 
 @pytest.mark.unit
-def test_getChannelByName(capsys):
+def test_getChannelByName():
     """Get a channel by the name."""
     anode = Node('foo', 'bar')
 
@@ -457,7 +457,7 @@ def test_getChannelByName(capsys):
 
 
 @pytest.mark.unit
-def test_getChannelByName_invalid_name(capsys):
+def test_getChannelByName_invalid_name():
     """Get a channel by the name but one that is not present."""
     anode = Node('foo', 'bar')
 
@@ -484,7 +484,7 @@ def test_getChannelByName_invalid_name(capsys):
 
 
 @pytest.mark.unit
-def test_getDisabledChannel(capsys):
+def test_getDisabledChannel():
     """Get the first disabled channel."""
     anode = Node('foo', 'bar')
 
@@ -514,7 +514,7 @@ def test_getDisabledChannel(capsys):
 
 
 @pytest.mark.unit
-def test_getDisabledChannel_where_all_channels_are_used(capsys):
+def test_getDisabledChannel_where_all_channels_are_used():
     """Get the first disabled channel."""
     anode = Node('foo', 'bar')
 
@@ -538,7 +538,7 @@ def test_getDisabledChannel_where_all_channels_are_used(capsys):
 
 
 @pytest.mark.unit
-def test_getAdminChannelIndex(capsys):
+def test_getAdminChannelIndex():
     """Get the 'admin' channel index."""
     anode = Node('foo', 'bar')
 
@@ -565,7 +565,7 @@ def test_getAdminChannelIndex(capsys):
 
 
 @pytest.mark.unit
-def test_getAdminChannelIndex_when_no_admin_named_channel(capsys):
+def test_getAdminChannelIndex_when_no_admin_named_channel():
     """Get the 'admin' channel when there is not one."""
     anode = Node('foo', 'bar')
 
diff --git a/meshtastic/tests/test_serial_interface.py b/meshtastic/tests/test_serial_interface.py
index 7e91b21..ac16cba 100644
--- a/meshtastic/tests/test_serial_interface.py
+++ b/meshtastic/tests/test_serial_interface.py
@@ -23,6 +23,10 @@ def test_SerialInterface_single_port(mocked_findPorts, mocked_serial, mocked_ope
     iface.close()
     mocked_findPorts.assert_called()
     mocked_serial.assert_called()
+    mocked_open.assert_called()
+    mock_get.assert_called()
+    mock_set.assert_called()
+    mock_sleep.assert_called()
     out, err = capsys.readouterr()
     assert re.search(r'Nodes in mesh', out, re.MULTILINE)
     assert re.search(r'Preferences', out, re.MULTILINE)
diff --git a/meshtastic/tests/test_stream_interface.py b/meshtastic/tests/test_stream_interface.py
index d87580e..1bf3451 100644
--- a/meshtastic/tests/test_stream_interface.py
+++ b/meshtastic/tests/test_stream_interface.py
@@ -19,7 +19,8 @@ def test_StreamInterface():
 
 # Note: This takes a bit, so moving from unit to slow
 @pytest.mark.unitslow
-def test_StreamInterface_with_noProto(caplog, reset_globals):
+@pytest.mark.usefixtures("reset_globals")
+def test_StreamInterface_with_noProto(caplog):
     """Test that we can instantiate a StreamInterface based on nonProto
        and we can read/write bytes from a mocked stream
     """
@@ -38,7 +39,8 @@ def test_StreamInterface_with_noProto(caplog, reset_globals):
 ## Tip: If you want to see the print output, run with '-s' flag:
 ##      pytest -s meshtastic/tests/test_stream_interface.py::test_sendToRadioImpl
 @pytest.mark.unitslow
-def test_sendToRadioImpl(caplog, reset_globals):
+@pytest.mark.usefixtures("reset_globals")
+def test_sendToRadioImpl(caplog):
     """Test _sendToRadioImpl()"""
 
 #    def add_header(b):
diff --git a/meshtastic/tests/test_tcp_interface.py b/meshtastic/tests/test_tcp_interface.py
index 2848087..574dfd1 100644
--- a/meshtastic/tests/test_tcp_interface.py
+++ b/meshtastic/tests/test_tcp_interface.py
@@ -28,7 +28,7 @@ def test_TCPInterface(capsys):
 
 
 @pytest.mark.unit
-def test_TCPInterface_without_connecting(capsys):
+def test_TCPInterface_without_connecting():
     """Test that we can instantiate a TCPInterface with connectNow as false"""
     with patch('socket.socket'):
         iface = TCPInterface(hostname='localhost', noProto=True, connectNow=False)
diff --git a/meshtastic/tests/test_tunnel.py b/meshtastic/tests/test_tunnel.py
index f977a39..14b9dd4 100644
--- a/meshtastic/tests/test_tunnel.py
+++ b/meshtastic/tests/test_tunnel.py
@@ -14,7 +14,7 @@ from ..globals import Globals
 
 @pytest.mark.unit
 @patch('platform.system')
-def test_Tunnel_on_non_linux_system(mock_platform_system, reset_globals):
+def test_Tunnel_on_non_linux_system(mock_platform_system):
     """Test that we cannot instantiate a Tunnel on a non Linux system"""
     a_mock = MagicMock()
     a_mock.return_value = 'notLinux'
@@ -29,7 +29,7 @@ def test_Tunnel_on_non_linux_system(mock_platform_system, reset_globals):
 
 @pytest.mark.unit
 @patch('platform.system')
-def test_Tunnel_without_interface(mock_platform_system, reset_globals):
+def test_Tunnel_without_interface(mock_platform_system):
     """Test that we can not instantiate a Tunnel without a valid interface"""
     a_mock = MagicMock()
     a_mock.return_value = 'Linux'
@@ -41,7 +41,7 @@ def test_Tunnel_without_interface(mock_platform_system, reset_globals):
 
 @pytest.mark.unitslow
 @patch('platform.system')
-def test_Tunnel_with_interface(mock_platform_system, caplog, reset_globals, iface_with_nodes):
+def test_Tunnel_with_interface(mock_platform_system, caplog, iface_with_nodes):
     """Test that we can not instantiate a Tunnel without a valid interface"""
     iface = iface_with_nodes
     iface.myInfo.my_node_num = 2475227164
@@ -60,7 +60,7 @@ def test_Tunnel_with_interface(mock_platform_system, caplog, reset_globals, ifac
 
 @pytest.mark.unitslow
 @patch('platform.system')
-def test_onTunnelReceive_from_ourselves(mock_platform_system, caplog, reset_globals, iface_with_nodes):
+def test_onTunnelReceive_from_ourselves(mock_platform_system, caplog, iface_with_nodes):
     """Test onTunnelReceive"""
     iface = iface_with_nodes
     iface.myInfo.my_node_num = 2475227164
@@ -81,7 +81,7 @@ def test_onTunnelReceive_from_ourselves(mock_platform_system, caplog, reset_glob
 
 @pytest.mark.unit
 @patch('platform.system')
-def test_onTunnelReceive_from_someone_else(mock_platform_system, caplog, reset_globals, iface_with_nodes):
+def test_onTunnelReceive_from_someone_else(mock_platform_system, caplog, iface_with_nodes):
     """Test onTunnelReceive"""
     iface = iface_with_nodes
     iface.myInfo.my_node_num = 2475227164
@@ -101,7 +101,7 @@ def test_onTunnelReceive_from_someone_else(mock_platform_system, caplog, reset_g
 
 @pytest.mark.unitslow
 @patch('platform.system')
-def test_shouldFilterPacket_random(mock_platform_system, caplog, reset_globals, iface_with_nodes):
+def test_shouldFilterPacket_random(mock_platform_system, caplog, iface_with_nodes):
     """Test _shouldFilterPacket()"""
     iface = iface_with_nodes
     iface.noProto = True
@@ -119,7 +119,7 @@ def test_shouldFilterPacket_random(mock_platform_system, caplog, reset_globals,
 
 @pytest.mark.unitslow
 @patch('platform.system')
-def test_shouldFilterPacket_in_blacklist(mock_platform_system, caplog, reset_globals, iface_with_nodes):
+def test_shouldFilterPacket_in_blacklist(mock_platform_system, caplog, iface_with_nodes):
     """Test _shouldFilterPacket()"""
     iface = iface_with_nodes
     iface.noProto = True
@@ -137,7 +137,7 @@ def test_shouldFilterPacket_in_blacklist(mock_platform_system, caplog, reset_glo
 
 @pytest.mark.unitslow
 @patch('platform.system')
-def test_shouldFilterPacket_icmp(mock_platform_system, caplog, reset_globals, iface_with_nodes):
+def test_shouldFilterPacket_icmp(mock_platform_system, caplog, iface_with_nodes):
     """Test _shouldFilterPacket()"""
     iface = iface_with_nodes
     iface.noProto = True
@@ -156,7 +156,7 @@ def test_shouldFilterPacket_icmp(mock_platform_system, caplog, reset_globals, if
 
 @pytest.mark.unit
 @patch('platform.system')
-def test_shouldFilterPacket_udp(mock_platform_system, caplog, reset_globals, iface_with_nodes):
+def test_shouldFilterPacket_udp(mock_platform_system, caplog, iface_with_nodes):
     """Test _shouldFilterPacket()"""
     iface = iface_with_nodes
     iface.noProto = True
@@ -175,7 +175,7 @@ def test_shouldFilterPacket_udp(mock_platform_system, caplog, reset_globals, ifa
 
 @pytest.mark.unitslow
 @patch('platform.system')
-def test_shouldFilterPacket_udp_blacklisted(mock_platform_system, caplog, reset_globals, iface_with_nodes):
+def test_shouldFilterPacket_udp_blacklisted(mock_platform_system, caplog, iface_with_nodes):
     """Test _shouldFilterPacket()"""
     iface = iface_with_nodes
     iface.noProto = True
@@ -196,7 +196,7 @@ def test_shouldFilterPacket_udp_blacklisted(mock_platform_system, caplog, reset_
 
 @pytest.mark.unit
 @patch('platform.system')
-def test_shouldFilterPacket_tcp(mock_platform_system, caplog, reset_globals, iface_with_nodes):
+def test_shouldFilterPacket_tcp(mock_platform_system, caplog, iface_with_nodes):
     """Test _shouldFilterPacket()"""
     iface = iface_with_nodes
     iface.noProto = True
@@ -215,7 +215,7 @@ def test_shouldFilterPacket_tcp(mock_platform_system, caplog, reset_globals, ifa
 
 @pytest.mark.unitslow
 @patch('platform.system')
-def test_shouldFilterPacket_tcp_blacklisted(mock_platform_system, caplog, reset_globals, iface_with_nodes):
+def test_shouldFilterPacket_tcp_blacklisted(mock_platform_system, caplog, iface_with_nodes):
     """Test _shouldFilterPacket()"""
     iface = iface_with_nodes
     iface.noProto = True
@@ -236,7 +236,7 @@ def test_shouldFilterPacket_tcp_blacklisted(mock_platform_system, caplog, reset_
 
 @pytest.mark.unitslow
 @patch('platform.system')
-def test_ipToNodeId_none(mock_platform_system, caplog, reset_globals, iface_with_nodes):
+def test_ipToNodeId_none(mock_platform_system, caplog, iface_with_nodes):
     """Test _ipToNodeId()"""
     iface = iface_with_nodes
     iface.noProto = True
@@ -252,7 +252,7 @@ def test_ipToNodeId_none(mock_platform_system, caplog, reset_globals, iface_with
 
 @pytest.mark.unitslow
 @patch('platform.system')
-def test_ipToNodeId_all(mock_platform_system, caplog, reset_globals, iface_with_nodes):
+def test_ipToNodeId_all(mock_platform_system, caplog, iface_with_nodes):
     """Test _ipToNodeId()"""
     iface = iface_with_nodes
     iface.noProto = True
diff --git a/meshtastic/tests/test_util.py b/meshtastic/tests/test_util.py
index c812d64..b933143 100644
--- a/meshtastic/tests/test_util.py
+++ b/meshtastic/tests/test_util.py
@@ -242,6 +242,7 @@ def test_readnet_u16():
 def test_findPorts_when_none_found(patch_comports):
     """Test findPorts()"""
     assert not findPorts()
+    patch_comports.assert_called()
 
 
 @pytest.mark.unitslow
diff --git a/meshtastic/tunnel.py b/meshtastic/tunnel.py
index 6e235d8..8f9ff87 100644
--- a/meshtastic/tunnel.py
+++ b/meshtastic/tunnel.py
@@ -27,7 +27,7 @@ from meshtastic.util import ipstr, readnet_u16
 from meshtastic.globals import Globals
 
 
-def onTunnelReceive(packet, interface):
+def onTunnelReceive(packet, interface): # pylint: disable=W0613
     """Callback for received tunneled messages from mesh."""
     logging.debug(f'in onTunnelReceive()')
     our_globals = Globals.getInstance()

From e7680e07c23346fae4bb65081975bc5fcfc11a78 Mon Sep 17 00:00:00 2001
From: Mike Kinney <mike.kinney@gmail.com>
Date: Wed, 12 Jan 2022 14:46:21 -0800
Subject: [PATCH 3/5] fix pylint global-statement warnings

---
 .pylintrc          | 2 +-
 meshtastic/test.py | 4 ++++
 2 files changed, 5 insertions(+), 1 deletion(-)

diff --git a/.pylintrc b/.pylintrc
index 8228bf4..6f3f472 100644
--- a/.pylintrc
+++ b/.pylintrc
@@ -23,7 +23,7 @@ ignore-patterns=mqtt_pb2.py,channel_pb2.py,environmental_measurement_pb2.py,admi
 # no Warning level messages displayed, use"--disable=all --enable=classes
 # --disable=W"
 #
-disable=invalid-name,fixme,logging-fstring-interpolation,too-many-statements,too-many-branches,too-many-locals,no-member,f-string-without-interpolation,protected-access,no-self-use,pointless-string-statement,too-few-public-methods,broad-except,no-else-return,global-statement,global-variable-not-assigned,too-many-boolean-expressions,no-else-raise,bare-except,c-extension-no-member
+disable=invalid-name,fixme,logging-fstring-interpolation,too-many-statements,too-many-branches,too-many-locals,no-member,f-string-without-interpolation,protected-access,no-self-use,pointless-string-statement,too-few-public-methods,broad-except,no-else-return,global-variable-not-assigned,too-many-boolean-expressions,no-else-raise,bare-except,c-extension-no-member
 
 
 [BASIC]
diff --git a/meshtastic/test.py b/meshtastic/test.py
index bedae30..c41ee60 100644
--- a/meshtastic/test.py
+++ b/meshtastic/test.py
@@ -63,6 +63,7 @@ def testSend(fromInterface, toInterface, isBroadcast=False, asBinary=False, want
     Returns:
         boolean -- True for success
     """
+    # pylint: disable=W0603
     global receivedPackets
     receivedPackets = []
     fromNode = fromInterface.myInfo.my_node_num
@@ -74,6 +75,7 @@ def testSend(fromInterface, toInterface, isBroadcast=False, asBinary=False, want
 
     logging.debug(
         f"Sending test wantAck={wantAck} packet from {fromNode} to {toNode}")
+    # pylint: disable=W0603
     global sendingInterface
     sendingInterface = fromInterface
     if not asBinary:
@@ -94,6 +96,7 @@ def runTests(numTests=50, wantAck=False, maxFailures=0):
     numFail = 0
     numSuccess = 0
     for _ in range(numTests):
+        # pylint: disable=W0603
         global testNumber
         testNumber = testNumber + 1
         isBroadcast = True
@@ -152,6 +155,7 @@ def testAll(numTests=5):
 
     pub.subscribe(onConnection, "meshtastic.connection")
     pub.subscribe(onReceive, "meshtastic.receive")
+    # pylint: disable=W0603
     global interfaces
     interfaces = list(map(lambda port: SerialInterface(
         port, debugOut=openDebugLog(port), connectNow=True), ports))

From ed8510468d0ff6d27c069c744b5db8c7a25a3b4a Mon Sep 17 00:00:00 2001
From: Mike Kinney <mike.kinney@gmail.com>
Date: Wed, 12 Jan 2022 14:47:49 -0800
Subject: [PATCH 4/5] no need to disable these pylint warnings anymore

---
 .pylintrc | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/.pylintrc b/.pylintrc
index 6f3f472..77dc59a 100644
--- a/.pylintrc
+++ b/.pylintrc
@@ -23,7 +23,7 @@ ignore-patterns=mqtt_pb2.py,channel_pb2.py,environmental_measurement_pb2.py,admi
 # no Warning level messages displayed, use"--disable=all --enable=classes
 # --disable=W"
 #
-disable=invalid-name,fixme,logging-fstring-interpolation,too-many-statements,too-many-branches,too-many-locals,no-member,f-string-without-interpolation,protected-access,no-self-use,pointless-string-statement,too-few-public-methods,broad-except,no-else-return,global-variable-not-assigned,too-many-boolean-expressions,no-else-raise,bare-except,c-extension-no-member
+disable=invalid-name,fixme,logging-fstring-interpolation,too-many-statements,too-many-branches,too-many-locals,no-member,f-string-without-interpolation,protected-access,no-self-use,pointless-string-statement,too-few-public-methods,broad-except,no-else-return,no-else-raise,bare-except,c-extension-no-member
 
 
 [BASIC]

From d2d93fbe803fe8f6db34118ad4be13e1708ce72e Mon Sep 17 00:00:00 2001
From: Mike Kinney <mike.kinney@gmail.com>
Date: Wed, 12 Jan 2022 14:50:10 -0800
Subject: [PATCH 5/5] no longer need to disable this pylint warning

---
 .pylintrc | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/.pylintrc b/.pylintrc
index 77dc59a..2935153 100644
--- a/.pylintrc
+++ b/.pylintrc
@@ -23,7 +23,7 @@ ignore-patterns=mqtt_pb2.py,channel_pb2.py,environmental_measurement_pb2.py,admi
 # no Warning level messages displayed, use"--disable=all --enable=classes
 # --disable=W"
 #
-disable=invalid-name,fixme,logging-fstring-interpolation,too-many-statements,too-many-branches,too-many-locals,no-member,f-string-without-interpolation,protected-access,no-self-use,pointless-string-statement,too-few-public-methods,broad-except,no-else-return,no-else-raise,bare-except,c-extension-no-member
+disable=invalid-name,fixme,logging-fstring-interpolation,too-many-statements,too-many-branches,too-many-locals,no-member,f-string-without-interpolation,protected-access,no-self-use,pointless-string-statement,too-few-public-methods,broad-except,no-else-return,no-else-raise,bare-except
 
 
 [BASIC]