Tests and linting passing (locally!)

Author: multiplemonomials

tools/python/mbed_tools/cli/list_connected_devices.py

@@ -16,8 +16,14 @@
 
 
 @click.command()
+# Note: We need to give --format an alias so it doesn't shadow a builtin
 @click.option(
-    "--format", type=click.Choice(["table", "json"]), default="table", show_default=True, help="Set output format."
+    "--format",
+    "format_type",
+    type=click.Choice(["table", "json"]),
+    default="table",
+    show_default=True,
+    help="Set output format.",
 )
 @click.option(
     "--show-all",

tools/python_tests/mbed_tools/cli/test_devices_command_integration.py

@@ -31,11 +31,3 @@ def callback():
 
         self.assertEqual(1, result.exit_code)
         logger_error.assert_called_once()
-
-
-class TestVersionCommand(TestCase):
-    def test_version_command(self):
-        runner = CliRunner()
-        result = runner.invoke(main.cli, ["--version"])
-        self.assertTrue(result.output)
-        self.assertEqual(0, result.exit_code)

tools/python_tests/mbed_tools/cli/test_project_management.py

@@ -18,7 +18,7 @@ def test_returns_data_from_ioreg_call(self, check_output):
         </plist>"""
 
         self.assertEqual(get_data("some device"), ["foo"])
-        check_output.assert_called_once_with(["ioreg", "-a", "-r", "-n", "some device", "-l"])
+        check_output.assert_called_once_with(["/usr/sbin/ioreg", "-a", "-r", "-n", "some device", "-l"])
 
     def test_handles_corrupt_data_gracefully(self, check_output):
         check_output.return_value = b"""<plist version="1.0">