Migrate TestCoreIndexWithoutChecksum from test_core.py to core_test.go

Author: MatteoPologruto

internal/integrationtest/core/core_test.go

@@ -851,3 +851,19 @@ func TestCoreLoadingPackageManager(t *testing.T) {
 	_, _, err := cli.Run("core", "list", "--all", "--format", "json")
 	require.NoError(t, err) // this should not make the cli crash
 }
+
+func TestCoreIndexWithoutChecksum(t *testing.T) {
+	env, cli := integrationtest.CreateArduinoCLIWithEnvironment(t)
+	defer env.CleanUp()
+
+	_, _, err := cli.Run("config", "init", "--dest-dir", ".")
+	require.NoError(t, err)
+	url := "https://raw.githubusercontent.com/keyboardio/ArduinoCore-GD32-Keyboardio/ae5938af2f485910729e7d27aa233032a1cb4734/package_gd32_index.json" // noqa: E501
+	_, _, err = cli.Run("config", "add", "board_manager.additional_urls", url)
+	require.NoError(t, err)
+
+	_, _, err = cli.Run("core", "update-index")
+	require.NoError(t, err)
+	_, _, err = cli.Run("core", "list", "--all")
+	require.NoError(t, err) // this should not make the cli crash
+}

test/test_core.py

@@ -130,14 +130,4 @@ def test_core_download_multiple_platforms(run_command, data_dir):
     # The cli should not allow it since optimizing the casing results in finding two cores
     res = run_command(["core", "upgrade", "Packager:Arch"])
     assert res.failed
-    assert "Invalid argument passed: Found 2 platform for reference" in res.stderr
-
-
-def test_core_index_without_checksum(run_command):
-    assert run_command(["config", "init", "--dest-dir", "."])
-    url = "https://raw.githubusercontent.com/keyboardio/ArduinoCore-GD32-Keyboardio/ae5938af2f485910729e7d27aa233032a1cb4734/package_gd32_index.json"  # noqa: E501
-    assert run_command(["config", "add", "board_manager.additional_urls", url])
-
-    assert run_command(["core", "update-index"])
-    result = run_command(["core", "list", "--all"])
-    assert result.ok  # this should not make the cli crash
+    assert "Invalid argument passed: Found 2 platform for reference" in res.stderr