fix: cover copy functionality with tests

kamilkisiela · Jun 27, 2022 · c2c8cf1 · c2c8cf1
1 parent b3c06ea
commit c2c8cf1
Show file tree

Hide file tree

Showing 5 changed files with 27 additions and 3 deletions.
diff --git a/src/commands/build.ts b/src/commands/build.ts
@@ -430,7 +430,7 @@ async function copyToDist(cwd: string, files: string[], distDir: string) {
     allFiles.map(async (file) => {
       if (await fse.pathExists(join(cwd, file))) {
         const sourcePath = join(cwd, file);
-        const destPath = join(cwd, distDir, file.replace("src/", ""));
+        const destPath = join(distDir, file.replace("src/", ""));
         await mkdirp(dirname(destPath));
         await fse.copyFile(sourcePath, destPath);
       }

diff --git a/test/__fixtures__/simple/README.md b/test/__fixtures__/simple/README.md
@@ -0,0 +1 @@
+Hello!
diff --git a/test/__fixtures__/simple/foo.json b/test/__fixtures__/simple/foo.json
@@ -0,0 +1 @@
+{ "hi": 1 }
diff --git a/test/__fixtures__/simple/package.json b/test/__fixtures__/simple/package.json
@@ -41,5 +41,10 @@
     "directory": "dist",
     "access": "public"
   },
-  "type": "module"
-}
+  "type": "module",
+  "buildOptions": {
+    "copy": [
+      "foo.json"
+    ]
+  }
+}
diff --git a/test/integration.spec.ts b/test/integration.spec.ts
@@ -18,6 +18,8 @@ it("can bundle a simple project", async () => {
   const indexDtsFilePath = path.resolve(baseDistPath, "typings", "index.d.ts");
   const indexMjsFilePath = path.resolve(baseDistPath, "esm", "index.js");
   const packageJsonFilePath = path.resolve(baseDistPath, "package.json");
+  const readmeFilePath = path.resolve(baseDistPath, "README.md");
+  const fooFilePath = path.resolve(baseDistPath, "foo.json");
 
   expect(fse.readFileSync(indexJsFilePath, "utf8")).toMatchInlineSnapshot(`
     "\\"use strict\\";
@@ -34,6 +36,14 @@ it("can bundle a simple project", async () => {
     "export var someNumber = 1;
     "
   `);
+  expect(fse.readFileSync(readmeFilePath, "utf8")).toMatchInlineSnapshot(`
+    "Hello!
+    "
+  `);
+  expect(fse.readFileSync(fooFilePath, "utf8")).toMatchInlineSnapshot(`
+    "{ \\"hi\\": 1 }
+    "
+  `);
   expect(fse.readFileSync(packageJsonFilePath, "utf8")).toMatchInlineSnapshot(`
     "{
       \\"name\\": \\"simple\\",
@@ -77,6 +87,9 @@ it("can bundle a simple project", async () => {
       }
     }"
   `);
+  await execa("node", [binaryFolder, "check"], {
+    cwd: path.resolve(fixturesFolder, "simple")
+  });
 });
 
 it("can build a monorepo project", async () => {
@@ -275,4 +288,8 @@ it("can build a monorepo project", async () => {
       }
     }"
   `);
+
+  await execa("node", [binaryFolder, "check"], {
+    cwd: path.resolve(fixturesFolder, "simple-monorepo")
+  });
 });