Migrate TestCoreLoadingPackageManager from test_core.py to core_test.go

Author: MatteoPologruto

internal/integrationtest/core/core_test.go

@@ -855,3 +855,14 @@ func TestCoreListOutdatedCore(t *testing.T) {
 	// Installed version must be older than latest
 	require.True(t, installedVersion.LessThan(latestVersion))
 }
+
+func TestCoreLoadingPackageManager(t *testing.T) {
+	env, cli := integrationtest.CreateArduinoCLIWithEnvironment(t)
+	defer env.CleanUp()
+
+	// Create empty architecture folder (this condition is normally produced by `core uninstall`)
+	require.NoError(t, cli.DataDir().Join("packages", "foovendor", "hardware", "fooarch").MkdirAll())
+
+	_, _, err := cli.Run("core", "list", "--all", "--format", "json")
+	require.NoError(t, err) // this should not make the cli crash
+}

test/test_core.py

@@ -133,14 +133,6 @@ def test_core_download_multiple_platforms(run_command, data_dir):
     assert "Invalid argument passed: Found 2 platform for reference" in res.stderr
 
 
-def test_core_loading_package_manager(run_command, data_dir):
-    # Create empty architecture folder (this condition is normally produced by `core uninstall`)
-    (Path(data_dir) / "packages" / "foovendor" / "hardware" / "fooarch").mkdir(parents=True)
-
-    result = run_command(["core", "list", "--all", "--format", "json"])
-    assert result.ok  # this should not make the cli crash
-
-
 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