Skip to content

Commit 7906d64

Browse files
Migrate TestRemoveMultipleArguments from test_config.py to config_test.go
1 parent 2d24f90 commit 7906d64

File tree

2 files changed

+40
-29
lines changed

2 files changed

+40
-29
lines changed

internal/integrationtest/config/config_test.go

Lines changed: 40 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -428,3 +428,43 @@ func TestRemoveSingleArgument(t *testing.T) {
428428
require.NoError(t, err)
429429
requirejson.Query(t, stdout, ".board_manager | .additional_urls", "[\"https://example.com/yet_another_package_example_index.json\"]")
430430
}
431+
432+
func TestRemoveMultipleArguments(t *testing.T) {
433+
env, cli := integrationtest.CreateArduinoCLIWithEnvironment(t)
434+
defer env.CleanUp()
435+
436+
// Create a config file
437+
_, _, err := cli.Run("config", "init", "--dest-dir", ".")
438+
require.NoError(t, err)
439+
440+
// Adds URLs
441+
urls := [2]string{
442+
"https://example.com/package_example_index.json",
443+
"https://example.com/yet_another_package_example_index.json",
444+
}
445+
_, _, err = cli.Run("config", "add", "board_manager.additional_urls", urls[0], urls[1])
446+
require.NoError(t, err)
447+
448+
// Verifies default state
449+
stdout, _, err := cli.Run("config", "dump", "--format", "json")
450+
require.NoError(t, err)
451+
requirejson.Query(t, stdout, ".board_manager | .additional_urls | length", "2")
452+
requirejson.Contains(t, stdout, `
453+
{
454+
"board_manager": {
455+
"additional_urls": [
456+
"https://example.com/package_example_index.json",
457+
"https://example.com/yet_another_package_example_index.json"
458+
]
459+
}
460+
}`)
461+
462+
// Remove all URLs
463+
_, _, err = cli.Run("config", "remove", "board_manager.additional_urls", urls[0], urls[1])
464+
require.NoError(t, err)
465+
466+
// Verifies all URLs have been removed
467+
stdout, _, err = cli.Run("config", "dump", "--format", "json")
468+
require.NoError(t, err)
469+
requirejson.Query(t, stdout, ".board_manager | .additional_urls", "[]")
470+
}

test/test_config.py

Lines changed: 0 additions & 29 deletions
Original file line numberDiff line numberDiff line change
@@ -17,35 +17,6 @@
1717
import yaml
1818

1919

20-
def test_remove_multiple_arguments(run_command):
21-
# Create a config file
22-
assert run_command(["config", "init", "--dest-dir", "."])
23-
24-
# Adds URLs
25-
urls = [
26-
"https://example.com/package_example_index.json",
27-
"https://example.com/yet_another_package_example_index.json",
28-
]
29-
assert run_command(["config", "add", "board_manager.additional_urls"] + urls)
30-
31-
# Verifies default state
32-
result = run_command(["config", "dump", "--format", "json"])
33-
assert result.ok
34-
settings_json = json.loads(result.stdout)
35-
assert 2 == len(settings_json["board_manager"]["additional_urls"])
36-
assert urls[0] in settings_json["board_manager"]["additional_urls"]
37-
assert urls[1] in settings_json["board_manager"]["additional_urls"]
38-
39-
# Remove all URLs
40-
assert run_command(["config", "remove", "board_manager.additional_urls"] + urls)
41-
42-
# Verifies all URLs have been removed
43-
result = run_command(["config", "dump", "--format", "json"])
44-
assert result.ok
45-
settings_json = json.loads(result.stdout)
46-
assert [] == settings_json["board_manager"]["additional_urls"]
47-
48-
4920
def test_remove_on_unsupported_key(run_command):
5021
# Create a config file
5122
assert run_command(["config", "init", "--dest-dir", "."])

0 commit comments

Comments
 (0)