Migrate TestRemoveMultipleArguments from test_config.py to config_test.go

Author: MatteoPologruto

internal/integrationtest/config/config_test.go

@@ -479,3 +479,43 @@ func TestRemoveSingleArgument(t *testing.T) {
 	require.NoError(t, err)
 	requirejson.Query(t, stdout, ".board_manager | .additional_urls", "[\"https://example.com/yet_another_package_example_index.json\"]")
 }
+
+func TestRemoveMultipleArguments(t *testing.T) {
+	env, cli := integrationtest.CreateArduinoCLIWithEnvironment(t)
+	defer env.CleanUp()
+
+	// Create a config file
+	_, _, err := cli.Run("config", "init", "--dest-dir", ".")
+	require.NoError(t, err)
+
+	// Adds URLs
+	urls := [2]string{
+		"https://example.com/package_example_index.json",
+		"https://example.com/yet_another_package_example_index.json",
+	}
+	_, _, err = cli.Run("config", "add", "board_manager.additional_urls", urls[0], urls[1])
+	require.NoError(t, err)
+
+	// Verifies default state
+	stdout, _, err := cli.Run("config", "dump", "--format", "json")
+	require.NoError(t, err)
+	requirejson.Query(t, stdout, ".board_manager | .additional_urls | length", "2")
+	requirejson.Contains(t, stdout, `
+	{
+		"board_manager": {
+			"additional_urls": [
+				"https://example.com/package_example_index.json",
+				"https://example.com/yet_another_package_example_index.json"
+			]
+		}
+	}`)
+
+	// Remove all URLs
+	_, _, err = cli.Run("config", "remove", "board_manager.additional_urls", urls[0], urls[1])
+	require.NoError(t, err)
+
+	// Verifies all URLs have been removed
+	stdout, _, err = cli.Run("config", "dump", "--format", "json")
+	require.NoError(t, err)
+	requirejson.Query(t, stdout, ".board_manager | .additional_urls", "[]")
+}

test/test_config.py

@@ -17,35 +17,6 @@
 import yaml
 
 
-def test_remove_multiple_arguments(run_command):
-    # Create a config file
-    assert run_command(["config", "init", "--dest-dir", "."])
-
-    # Adds URLs
-    urls = [
-        "https://example.com/package_example_index.json",
-        "https://example.com/yet_another_package_example_index.json",
-    ]
-    assert run_command(["config", "add", "board_manager.additional_urls"] + urls)
-
-    # Verifies default state
-    result = run_command(["config", "dump", "--format", "json"])
-    assert result.ok
-    settings_json = json.loads(result.stdout)
-    assert 2 == len(settings_json["board_manager"]["additional_urls"])
-    assert urls[0] in settings_json["board_manager"]["additional_urls"]
-    assert urls[1] in settings_json["board_manager"]["additional_urls"]
-
-    # Remove all URLs
-    assert run_command(["config", "remove", "board_manager.additional_urls"] + urls)
-
-    # Verifies all URLs have been removed
-    result = run_command(["config", "dump", "--format", "json"])
-    assert result.ok
-    settings_json = json.loads(result.stdout)
-    assert [] == settings_json["board_manager"]["additional_urls"]
-
-
 def test_remove_on_unsupported_key(run_command):
     # Create a config file
     assert run_command(["config", "init", "--dest-dir", "."])