tests: add new test cases

Author: d-gubert

tests/AppPackager.spec.ts

@@ -360,6 +360,38 @@ describe("AppPackager", () => {
         // along with the custom ignored file
     });
 
+    it("should use forward slashes in zip entry paths for nested files", async () => {
+        // Create files in subdirectories to exercise the path separator fix
+        await fs.mkdirp(path.join(tempDir, "lib"));
+        await fs.mkdirp(path.join(tempDir, "assets", "icons"));
+        await fs.writeFile(
+            path.join(tempDir, "lib", "helper.json"),
+            '{"ok":true}',
+        );
+        await fs.writeFile(
+            path.join(tempDir, "assets", "icons", "logo.png"),
+            "fake png",
+        );
+
+        const outputFile = path.join(tempDir, "output.zip");
+        const packager = new AppPackager(
+            compilerDesc,
+            folderDetails,
+            compilationResult,
+            outputFile,
+        );
+
+        await packager.zipItUp();
+
+        const zipContents = await getZipFileList(outputFile);
+
+        // Regardless of the host OS separator, zip entry paths must use "/"
+        expect(zipContents).to.include("lib/helper.json");
+        expect(zipContents).to.include("assets/icons/logo.png");
+        expect(zipContents).to.not.include("lib\\helper.json");
+        expect(zipContents).to.not.include("assets\\icons\\logo.png");
+    });
+
     it("should handle .rcappsconfig without ignoredFiles property", async () => {
         // Create .rcappsconfig without ignoredFiles property
         const rcAppsConfig = {

tests/TypescriptCompiler.spec.ts

@@ -118,6 +118,30 @@ describe("TypescriptCompiler", () => {
         expect(result.implemented).to.deep.equal(["IPostMessageSent"]);
     });
 
+    it("normalizes backslash separators in sourceFiles keys and classFile (Windows paths)", () => {
+        // Simulate an IAppSource constructed on Windows where path.join produces
+        // backslash-separated keys and classFile comes from app.json with backslashes.
+        const appInfo = { ...baseAppInfo, classFile: "lib\\Foo.ts" };
+
+        const sourceFiles: Record<string, ICompilerFile> = {
+            "lib\\Foo.ts": {
+                name: "lib\\Foo.ts",
+                content: "export class Foo {}",
+                version: 1,
+            },
+        };
+
+        const result = compiler.transpileSource({ appInfo, sourceFiles });
+
+        expect(result.diagnostics).to.be.empty;
+        // Keys in result.files must use forward slashes regardless of input
+        expect(Object.keys(result.files)).to.include("lib/Foo.js");
+        expect(Object.keys(result.files)).to.not.include("lib\\Foo.js");
+        // mainFile must be resolved correctly via the normalised classFile
+        expect(result.mainFile).to.exist;
+        expect(result.mainFile.name).to.equal("lib/Foo.js");
+    });
+
     it("throws on invalid permission names", () => {
         const badAppInfo = {
             ...baseAppInfo,