Add TestCoreBrokenDependency to core_test.go

Author: MatteoPologruto

Diff for: internal/integrationtest/core/core_test.go

@@ -80,7 +80,7 @@ func TestCoreSearch(t *testing.T) {
 	// show all versions
 	out, _, err = cli.Run("core", "search", "test_core", "--all", "--format", "json", "--additional-urls="+url.String())
 	require.NoError(t, err)
-	requirejson.Len(t, out, 2)
+	requirejson.Len(t, out, 3)
 
 	checkPlatformIsInJSONOutput := func(stdout []byte, id, version string) {
 		jqquery := fmt.Sprintf(`[{id:"%s", latest:"%s"}]`, id, version)
@@ -150,7 +150,7 @@ func TestCoreSearchNoArgs(t *testing.T) {
 	// update custom index and install test core (installed cores affect `core search`)
 	_, _, err := cli.Run("core", "update-index", "--additional-urls="+url.String())
 	require.NoError(t, err)
-	_, _, err = cli.Run("core", "install", "test:x86", "--additional-urls="+url.String())
+	_, _, err = cli.Run("core", "install", "test:x86@2.0.0", "--additional-urls="+url.String())
 	require.NoError(t, err)
 
 	// list all with no additional urls, ensure the test core won't show up
@@ -725,7 +725,7 @@ func TestCoreListSortedResults(t *testing.T) {
 	require.NoError(t, err)
 
 	// install some core for testing
-	_, _, err = cli.Run("core", "install", "test:x86", "Retrokits-RK002:arm", "Package:x86", "--additional-urls="+url.String())
+	_, _, err = cli.Run("core", "install", "test:x86@2.0.0", "Retrokits-RK002:arm", "Package:x86", "--additional-urls="+url.String())
 	require.NoError(t, err)
 
 	// list all with additional url specified
@@ -1006,3 +1006,21 @@ func TestCoreInstallRunsToolPostInstallScript(t *testing.T) {
 	require.NoError(t, err)
 	require.Contains(t, string(stdout), "Skipping tool configuration.")
 }
+
+func TestCoreBrokenDependency(t *testing.T) {
+	env, cli := integrationtest.CreateArduinoCLIWithEnvironment(t)
+	defer env.CleanUp()
+
+	// Set up an http server to serve our custom index file
+	test_index := paths.New("..", "testdata", "test_index.json")
+	url := env.HTTPServeFile(8000, test_index)
+
+	// Run update-index with our test index
+	_, _, err := cli.Run("core", "update-index", "--additional-urls="+url.String())
+	require.NoError(t, err)
+
+	// Check that the download fails and the correct message is displayed
+	_, stderr, err := cli.Run("core", "install", "test:[email protected]", "--additional-urls="+url.String())
+	require.Error(t, err)
+	require.Contains(t, string(stderr), "try contacting [email protected]")
+}

Diff for: internal/integrationtest/testdata/test_index.json

@@ -44,9 +44,40 @@
               "name": "Test Board"
             }
           ]
+        },
+        {
+          "category": "Test Category",
+          "help": {
+            "online": "https://github.com/Arduino/arduino-cli"
+          },
+          "url": "https://raw.githubusercontent.com/arduino/arduino-cli/master/internal/integrationtest/testdata/core.zip",
+          "checksum": "SHA-256:6a338cf4d6d501176a2d352c87a8d72ac7488b8c5b82cdf2a4e2cef630391092",
+          "name": "test_core",
+          "version": "3.0.0",
+          "architecture": "x86",
+          "archiveFileName": "core.zip",
+          "size": "486",
+          "toolsDependencies": [
+            {
+              "packager": "test",
+              "version": "1.0.6",
+              "name": "rp2040tools"
+            }
+          ],
+          "boards": [
+            {
+              "name": "Test Board"
+            }
+          ]
+        }
+      ],
+      "tools": [
+        {
+          "name": "rp2040tools",
+          "version": "1.0.6",
+          "systems": []
         }
       ],
-      "tools": [],
       "email": "[email protected]",
       "name": "test"
     },