Skip to content

Commit 7cd3ee5

Browse files
committedNov 9, 2022
Migrate TestCoreBrokenInstall from test_core.py to core_test.go
1 parent f3f61e7 commit 7cd3ee5

File tree

2 files changed

+11
-6
lines changed

2 files changed

+11
-6
lines changed
 

‎internal/integrationtest/core/core_test.go

+11
Original file line numberDiff line numberDiff line change
@@ -359,3 +359,14 @@ func TestCoreZipslip(t *testing.T) {
359359
require.Error(t, err)
360360
require.NoFileExists(t, paths.TempDir().Join("evil.txt").String())
361361
}
362+
363+
func TestCoreBrokenInstall(t *testing.T) {
364+
env, cli := integrationtest.CreateArduinoCLIWithEnvironment(t)
365+
defer env.CleanUp()
366+
367+
url := "https://raw.githubusercontent.com/arduino/arduino-cli/master/test/testdata/test_index.json"
368+
_, _, err := cli.Run("core", "update-index", "--additional-urls="+url)
369+
require.NoError(t, err)
370+
_, _, err = cli.Run("core", "install", "brokenchecksum:x86", "--additional-urls="+url)
371+
require.Error(t, err)
372+
}

‎test/test_core.py

-6
Original file line numberDiff line numberDiff line change
@@ -47,12 +47,6 @@ def test_core_install_esp32(run_command, data_dir):
4747
assert (build_dir / f"{sketch_name}.ino.partitions.bin").exists()
4848

4949

50-
def test_core_broken_install(run_command):
51-
url = "https://raw.githubusercontent.com/arduino/arduino-cli/master/test/testdata/test_index.json"
52-
assert run_command(["core", "update-index", f"--additional-urls={url}"])
53-
assert not run_command(["core", "install", "brokenchecksum:x86", "--additional-urls={url}"])
54-
55-
5650
def test_core_install_creates_installed_json(run_command, data_dir):
5751
assert run_command(["core", "update-index"])
5852
assert run_command(["core", "install", "arduino:avr@1.6.23"])

0 commit comments

Comments
 (0)