fix test in 241

Author: samgst-amazon

plugins/amazonq/shared/jetbrains-community/src/software/aws/toolkits/jetbrains/services/amazonq/lsp/artifacts/LspUtils.kt

@@ -7,6 +7,8 @@ import com.intellij.openapi.util.SystemInfo
 import com.intellij.openapi.util.text.StringUtil
 import com.intellij.util.io.DigestUtil
 import com.intellij.util.system.CpuArch
+import java.nio.file.attribute.PosixFileAttributes
+import java.nio.file.attribute.PosixFileAttributeView
 import software.aws.toolkits.core.utils.ZIP_PROPERTY_POSIX
 import software.aws.toolkits.core.utils.createParentDirectories
 import software.aws.toolkits.core.utils.exists
@@ -18,6 +20,7 @@ import java.nio.file.Files
 import java.nio.file.Path
 import java.nio.file.Paths
 import java.nio.file.StandardCopyOption
+import java.nio.file.attribute.PosixFilePermissions
 import java.security.MessageDigest
 import kotlin.io.path.isDirectory
 import kotlin.io.path.listDirectoryEntries
@@ -91,7 +94,12 @@ fun extractZipFile(zipFilePath: Path, destDir: Path) {
                     .forEach { zipEntry ->
                         val destPath = Paths.get(destDir.toString(), zipEntry.toString())
                         destPath.createParentDirectories()
-                        Files.copy(zipEntry, destPath, StandardCopyOption.REPLACE_EXISTING, StandardCopyOption.COPY_ATTRIBUTES)
+
+                        Files.copy(zipEntry, destPath, StandardCopyOption.REPLACE_EXISTING)
+
+                        // Explicitly set permissions after copy
+                        val permissions = Files.getPosixFilePermissions(zipEntry)
+                        Files.setPosixFilePermissions(destPath, permissions)
                     }
             }
         }

plugins/amazonq/shared/jetbrains-community/tst/software/aws/toolkits/jetbrains/services/amazonq/lsp/artifacts/LspUtilsTest.kt

@@ -22,6 +22,7 @@ import java.nio.file.attribute.PosixFilePermissions
 import java.util.zip.ZipOutputStream
 import kotlin.io.path.isRegularFile
 import kotlin.io.path.setPosixFilePermissions
+import java.nio.file.attribute.PosixFilePermission
 
 class LspUtilsTest {
     @Test
@@ -86,7 +87,12 @@ class LspUtilsTest {
                 paths
                     .filter { it.isRegularFile() }
                     .forEach { file ->
-                        Files.copy(file, zipfs.getPath("/").resolve(source.relativize(file).toString()), StandardCopyOption.COPY_ATTRIBUTES)
+                        val targetPath = zipfs.getPath("/").resolve(source.relativize(file).toString())
+                        // First copy the file
+                        Files.copy(file, targetPath)
+                        // Then explicitly set the permissions from the source file
+                        val sourcePerms = Files.getPosixFilePermissions(file)
+                        Files.setPosixFilePermissions(targetPath, sourcePerms)
                     }
             }
         }