Migrated TestBoardListWithInvalidDiscovery from test_board.py to board_test.go

Originally, stderr was checked to assert that the CLI did not crash if an invalid discovery was installed, but now, under the same conditions, stderr results empty.
I decided to check if stdout contained  instead and the checks have been successful.

Author: MatteoPologruto

internal/integrationtest/board/board_test.go

@@ -46,3 +46,24 @@ func TestBoardList(t *testing.T) {
 		require.Contains(t, port, "protocol_label")
 	}
 }
+
+func TestBoardListWithInvalidDiscovery(t *testing.T) {
+	env, cli := integrationtest.CreateArduinoCLIWithEnvironment(t)
+	defer env.CleanUp()
+
+	_, _, err := cli.Run("core", "update-index")
+	require.NoError(t, err)
+	_, _, err = cli.Run("board", "list")
+	require.NoError(t, err)
+
+	// check that the CLI does not crash if an invalid discovery is installed
+	// (for example if the installation fails midway).
+	// https://github.com/arduino/arduino-cli/issues/1669
+	toolDir := cli.DataDir().Join("packages", "builtin", "tools", "serial-discovery")
+	err = toolDir.RemoveAll()
+	require.NoError(t, err)
+
+	stdout, _, err := cli.Run("board", "list")
+	require.NoError(t, err)
+	require.Contains(t, string(stdout), "builtin:serial-discovery")
+}

test/test_board.py

@@ -394,23 +394,6 @@
 """  # noqa: E501
 
 
-def test_board_list_with_invalid_discovery(run_command, data_dir):
-    run_command(["core", "update-index"])
-    result = run_command(["board", "list"])
-    assert result.ok
-
-    # check that the CLI do no crash if an invalid discovery is installed
-    # (for example if the installation fails midway).
-    # https://github.com/arduino/arduino-cli/issues/1669
-    tool_dir = os.path.join(data_dir, "packages", "builtin", "tools", "serial-discovery")
-    for file_to_delete in glob.glob(tool_dir + "/*/*"):
-        os.remove(file_to_delete)
-
-    result = run_command(["board", "list"])
-    assert result.ok
-    assert "builtin:serial-discovery" in result.stderr
-
-
 def test_board_listall(run_command):
     assert run_command(["update"])
     assert run_command(["core", "install", "arduino:avr@1.8.3"])