Migrate TestSketchArchiveAbsoluteSketchPathWithAbsoluteZipPathAndNameWithExtension from test_sketch.py to sketch_test.go

Author: MatteoPologruto

internal/integrationtest/sketch/sketch_test.go

@@ -423,3 +423,21 @@ func TestSketchArchiveAbsoluteSketchPathWithAbsoluteZipPathAndNameWithoutExtensi
 	defer require.NoError(t, archive.Close())
 	verifyZipContainsSketchExcludingBuildDir(t, archive.File)
 }
+
+func TestSketchArchiveAbsoluteSketchPathWithAbsoluteZipPathAndNameWithExtension(t *testing.T) {
+	env, cli := integrationtest.CreateArduinoCLIWithEnvironment(t)
+	defer env.CleanUp()
+
+	// Creates a folder where to save the zip
+	archivesFolder := cli.WorkingDir().Join("my_archives")
+	require.NoError(t, archivesFolder.Mkdir())
+
+	cli.SetWorkingDir(cli.CopySketch("sketch_simple"))
+	_, _, err := cli.Run("sketch", "archive", env.RootDir().Join("sketch_simple").String(), archivesFolder.Join("my_custom_sketch.zip").String())
+	require.NoError(t, err)
+
+	archive, err := zip.OpenReader(archivesFolder.Join("my_custom_sketch.zip").String())
+	require.NoError(t, err)
+	defer require.NoError(t, archive.Close())
+	verifyZipContainsSketchExcludingBuildDir(t, archive.File)
+}

test/test_sketch.py

@@ -48,27 +48,6 @@ def verify_zip_contains_sketch_including_build_dir(files):
     assert "sketch_simple/build/arduino.avr.uno/sketch_simple.ino.with_bootloader.hex" in files
 
 
-def test_sketch_archive_absolute_sketch_path_with_absolute_zip_path_and_name_with_extension(
-    run_command, copy_sketch, working_dir
-):
-    # Creates a folder where to save the zip
-    archives_folder = f"{working_dir}/my_archives/"
-    Path(archives_folder).mkdir()
-
-    result = run_command(
-        ["sketch", "archive", f"{working_dir}/sketch_simple", f"{archives_folder}/my_custom_sketch.zip"],
-        copy_sketch("sketch_simple"),
-    )
-    assert result.ok
-
-    archive = zipfile.ZipFile(f"{archives_folder}/my_custom_sketch.zip")
-    archive_files = archive.namelist()
-
-    verify_zip_contains_sketch_excluding_build_dir(archive_files)
-
-    archive.close()
-
-
 def test_sketch_archive_no_args_with_include_build_dir_flag(run_command, copy_sketch, working_dir):
     result = run_command(["sketch", "archive", "--include-build-dir"], copy_sketch("sketch_simple"))
     assert result.ok