Migrate TestSketchArchiveAbsoluteSketchPathWithAbsoluteZipPathAndNameWithExtensionWithIncludeBuildDirFlag from test_sketch.py to sketch_test.go

Author: MatteoPologruto

internal/integrationtest/sketch/sketch_test.go

@@ -809,3 +809,21 @@ func TestSketchArchiveAbsoluteSketchPathWithAbsoluteZipPathAndNameWithoutExtensi
 	defer require.NoError(t, archive.Close())
 	verifyZipContainsSketchIncludingBuildDir(t, archive.File)
 }
+
+func TestSketchArchiveAbsoluteSketchPathWithAbsoluteZipPathAndNameWithExtensionWithIncludeBuildDirFlag(t *testing.T) {
+	env, cli := integrationtest.CreateArduinoCLIWithEnvironment(t)
+	defer env.CleanUp()
+
+	_ = cli.CopySketch("sketch_simple")
+	// Creates a folder where to save the zip
+	archivesFolder := cli.WorkingDir().Join("my_archives")
+	require.NoError(t, archivesFolder.Mkdir())
+
+	_, _, err := cli.Run("sketch", "archive", cli.WorkingDir().Join("sketch_simple").String(), archivesFolder.Join("my_custom_sketch.zip").String(), "--include-build-dir")
+	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())
+	verifyZipContainsSketchIncludingBuildDir(t, archive.File)
+}

test/test_sketch.py

@@ -48,33 +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_with_include_build_dir_flag(
-    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",
-            "--include-build-dir",
-        ],
-        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_including_build_dir(archive_files)
-
-    archive.close()
-
-
 def test_sketch_archive_with_pde_main_file(run_command, copy_sketch, working_dir):
     sketch_name = "sketch_pde_main_file"
     sketch_dir = copy_sketch(sketch_name)