Merge pull request #4 from tychenjiajun/fix/remove-windows-flag

Fix/remove windows flag

Author: tychenjiajun

src/__tests__/raw-therapee-wrap-windows.test.ts

@@ -61,10 +61,17 @@ describe("Windows platform tests", () => {
 
     await convertDngToImage(parameters);
 
-    // Check that the arguments include the Windows flag
-    expect(spawn).toHaveBeenCalledWith(
-      "rawtherapee-cli",
-      expect.arrayContaining(["-w"]),
-    );
+    // Check that the arguments no longer include the Windows flag
+    const expectedArgs = [
+      "-Y",
+      "-O",
+      path.join(TEST_OUTPUT_DIR, "output.jpg"),
+      "-j90",
+      "-js3",
+      "-b16",
+      "-c",
+      TEST_INPUT_FILE,
+    ];
+    expect(spawn).toHaveBeenCalledWith("rawtherapee-cli", expectedArgs);
   });
 });

src/__tests__/raw-therapee-wrap.test.ts

@@ -97,7 +97,7 @@ it("convertDngToImage should throw error when output directory does not exist",
   };
 
   await expect(convertDngToImage(parameters)).rejects.toThrow(
-    /Output directory does not exist/,
+    /Failed to create output directory/,
   );
 });
 

src/raw-therapee-wrap.ts

@@ -153,7 +153,6 @@ function buildCliArguments({
   input: string;
 }) {
   const cliArguments = [
-    ...(os.platform() === "win32" ? ["-w"] : []),
     "-Y",
     pp3Path ? "-o" : "-O",
     output,