Migrate TestLibDownload from test_lib.py to lib_test.go

Author: MatteoPologruto

internal/integrationtest/lib/lib_test.go

@@ -494,3 +494,21 @@ func TestListProvidesIncludesFallback(t *testing.T) {
 			]
 	  	}`)
 }
+
+func TestLibDownload(t *testing.T) {
+	env, cli := integrationtest.CreateArduinoCLIWithEnvironment(t)
+	defer env.CleanUp()
+
+	// Download a specific lib version
+	_, _, err := cli.Run("lib", "download", "AudioZero@1.0.0")
+	require.NoError(t, err)
+	require.FileExists(t, cli.DownloadDir().Join("libraries", "AudioZero-1.0.0.zip").String())
+
+	// Wrong lib version
+	_, _, err = cli.Run("lib", "download", "AudioZero@69.42.0")
+	require.Error(t, err)
+
+	// Wrong lib
+	_, _, err = cli.Run("lib", "download", "AudioZ")
+	require.Error(t, err)
+}

test/test_lib.py

@@ -49,21 +49,6 @@ def download_lib(url, download_dir):
     z.close()
 
 
-def test_lib_download(run_command, downloads_dir):
-
-    # Download a specific lib version
-    assert run_command(["lib", "download", "AudioZero@1.0.0"])
-    assert Path(downloads_dir, "libraries", "AudioZero-1.0.0.zip").exists()
-
-    # Wrong lib version
-    result = run_command(["lib", "download", "AudioZero@69.42.0"])
-    assert result.failed
-
-    # Wrong lib
-    result = run_command(["lib", "download", "AudioZ"])
-    assert result.failed
-
-
 def test_install(run_command):
     libs = ["Arduino_BQ24195", "CMMC MQTT Connector", "WiFiNINA"]
     # Should be safe to run install multiple times