Migrate TestCoreUpdateIndexInternalServerError from test_core.py to core_test.go

Author: MatteoPologruto

internal/integrationtest/core/core_test.go

@@ -204,3 +204,18 @@ func TestCoreUpdateIndexUrlNotFound(t *testing.T) {
 	require.Contains(t, string(stdout), "Downloading index: test_index.json Server responded with: 404 Not Found")
 	require.Contains(t, string(stderr), "Some indexes could not be updated.")
 }
+
+func TestCoreUpdateIndexInternalServerError(t *testing.T) {
+	env, cli := integrationtest.CreateArduinoCLIWithEnvironment(t)
+	defer env.CleanUp()
+
+	_, _, err := cli.Run("core", "update-index")
+	require.NoError(t, err)
+
+	// Brings up a local server to fake a failure
+	url := env.HTTPServeFileError(8000, paths.New("test_index.json"), 500)
+
+	stdout, _, err := cli.Run("core", "update-index", "--additional-urls="+url.String())
+	require.Error(t, err)
+	require.Contains(t, string(stdout), "Downloading index: test_index.json Server responded with: 500 Internal Server Error")
+}

test/test_core.py

@@ -26,19 +26,6 @@
 import semver
 
 
-def test_core_updateindex_internal_server_error(run_command, httpserver):
-    assert run_command(["core", "update-index"])
-
-    # Brings up a local server to fake a failure
-    httpserver.expect_request("/test_index.json").respond_with_data(status=500)
-    url = httpserver.url_for("/test_index.json")
-
-    result = run_command(["core", "update-index", f"--additional-urls={url}"])
-    assert result.failed
-    lines = [l.strip() for l in result.stdout.splitlines()]
-    assert "Downloading index: test_index.json Server responded with: 500 INTERNAL SERVER ERROR" in lines
-
-
 def test_core_install_without_updateindex(run_command):
     # Missing "core update-index"
     # Download samd core pinned to 1.8.6