fix(barrels): add integration test and fix issue

Author: Romakita

packages/barrels/.gitignore

@@ -0,0 +1 @@
+**/__mock__/**/index.ts

packages/barrels/bin/barrels.js

@@ -1,89 +1,13 @@
 #!/usr/bin/env node
-import {existsSync} from "node:fs";
-import {readFile, writeFile} from "node:fs/promises";
-import {dirname, join} from "node:path";
-import {globby} from "globby";
-
-function resolveConfig() {
-  return [
-    join(process.cwd(), ".barrelsby.json"),
-    join(process.cwd(), ".barrels.json")
-  ].find((path) => {
-    return existsSync(path);
-  });
-}
-
-async function readJSON(path) {
-  const content = await readFile(path, "utf-8");
-
-  return JSON.parse(content);
-}
-
-function getConfig() {
-  const configPath = resolveConfig();
-
-  if (!configPath) {
-    return {};
-  }
-
-  return readJSON(configPath);
-}
-
-async function cleanIndex(cwd, excluded) {
-  const patterns = [
-    "**/index.ts",
-    ...excluded
-  ];
-
-  const files = await globby(patterns, {
-    cwd: cwd
-  });
-
-  return Promise.all(files.map((file) => fs.unlink(join(cwd, file))));
-}
+import {generateBarrels} from "../src/generate-barrel.js";
+import {getConfig} from "../src/get-config.js";
 
 async function build() {
   const {
     directory = ["./src"],
     exclude = ["**/__mock__", "**/__mocks__", "**/*.spec.ts", "**/*.benchmark.ts"],
-    delete: shouldDelete
   } = await getConfig();
-
-  const excluded = exclude
-    .map((path) => `!${path}`)
-    .concat(directory.map((path) => `!${path}/index.ts`));
-
-  const directories = (
-    await globby(directory.map((d) => {
-        return join(d, "*");
-      }),
-      {
-        cwd: process.cwd()
-      })
-  ).reduce((set, file) => {
-    return set.add(dirname(file));
-  }, new Set());
-
-  const promises = [...directories.keys()].map(async (directory) => {
-    const baseIndex = join(process.cwd(), directory?.path ?? directory);
-
-    const files = await globby(["**/*.ts", "!index.ts", ...excluded], {
-      cwd: directory
-    });
-
-    const exports = files
-      .sort((a, b) => a.localeCompare(b))
-      .map((file) => {
-        // TODO set .js after all configuration are ok to resolve .js
-        return `export * from "./${file.replace(".ts", ".js")}";`;
-      });
-
-    const content = ["/**", " * @file Automatically generated by @tsed/barrels.", " */", ...exports];
-
-    await writeFile(join(baseIndex, "index.ts"), content.join("\n") + "\n", {encoding: "utf8"});
-  });
-
-  await Promise.all(promises);
+  await generateBarrels({exclude, directory, cwd: process.cwd()});
 }
 
 await build();

packages/barrels/package.json

@@ -14,5 +14,8 @@
   },
   "dependencies": {
     "globby": "14.0.2"
+  },
+  "scripts": {
+    "test": "node --test test/barrels.integration.spec.js"
   }
 }

packages/barrels/src/generate-barrel.js

@@ -0,0 +1,54 @@
+import {writeFile} from "node:fs/promises";
+import path, {dirname, join} from "node:path";
+import {globby} from "globby";
+
+// async function cleanIndex(cwd, excluded) {
+//   const patterns = [
+//     "**/index.ts",
+//     ...excluded
+//   ];
+//
+//   const files = await globby(patterns, {
+//     cwd: cwd
+//   });
+//
+//   return Promise.all(files.map((file) => fs.unlink(join(cwd, file))));
+// }
+
+export async function generateBarrels({exclude, directory, cwd}) {
+  const excluded = exclude
+    .map((path) => `!${path}`)
+    .concat(directory.map((path) => `!${path}/index.ts`));
+
+  const directories = (
+    await globby(directory.map((d) => {
+        return join(d, "*");
+      }),
+      {
+        cwd
+      })
+  ).reduce((set, file) => {
+    return set.add(dirname(file));
+  }, new Set());
+
+  const promises = [...directories.keys()].map(async (directory) => {
+    const baseIndex = join(cwd, directory?.path ?? directory);
+
+    const files = await globby(["**/*.{ts,tsx}", "!index.{ts,tsx}", ...excluded], {
+      cwd: path.join(cwd, directory)
+    });
+
+    const exports = files
+      .sort((a, b) => a.localeCompare(b))
+      .map((file) => {
+        // TODO set .js after all configuration are ok to resolve .js
+        return `export * from "./${file.replace(".ts", ".js")}";`;
+      });
+
+    const content = ["/**", " * @file Automatically generated by @tsed/barrels.", " */", ...exports];
+
+    await writeFile(join(baseIndex, "index.ts"), content.join("\n") + "\n", {encoding: "utf8"});
+  });
+
+  await Promise.all(promises);
+}

packages/barrels/src/get-config.js

@@ -0,0 +1,28 @@
+import {join} from "node:path";
+import {existsSync} from "node:fs";
+import {readFile} from "node:fs/promises";
+
+function resolveConfig() {
+  return [
+    join(process.cwd(), ".barrelsby.json"),
+    join(process.cwd(), ".barrels.json")
+  ].find((path) => {
+    return existsSync(path);
+  });
+}
+
+async function readJSON(path) {
+  const content = await readFile(path, "utf-8");
+
+  return JSON.parse(content);
+}
+
+export function getConfig() {
+  const configPath = resolveConfig();
+
+  if (!configPath) {
+    return {};
+  }
+
+  return readJSON(configPath);
+}

packages/barrels/test/__mock__/scenario-1/file1.spec.ts

@@ -0,0 +1,7 @@
+import {file1} from "./file1";
+
+describe("barrels", () => {
+  it("should", () => {
+    expect(file1).toEqual("file1")
+  })
+})

packages/barrels/test/__mock__/scenario-1/file1.ts

@@ -0,0 +1 @@
+export const file1 = "file1"

packages/barrels/test/__mock__/scenario-1/file2.ts

@@ -0,0 +1 @@
+export const file2 = "file2"

packages/barrels/test/__mock__/scenario-1/sub-directory/file2.ts

@@ -0,0 +1 @@
+export const file3 = "file3"

packages/barrels/test/barrels.integration.spec.js

@@ -0,0 +1,27 @@
+import {generateBarrels} from "../src/generate-barrel.js";
+import {describe, it} from "node:test";
+import assert from "node:assert";
+import {join} from "node:path";
+import * as fs from "node:fs/promises";
+
+describe('barrels.integration.ts', () => {
+  it('should generate barrels', async () => {
+    const cwd = join(import.meta.dirname, "__mock__")
+    // Test code here
+    await generateBarrels({
+      cwd,
+      "directory": ["./scenario-1"],
+      "exclude": ["**/__mock__", "**/__mocks__", "**/*.spec.ts"],
+      "delete": true
+    })
+
+    const result = await fs.readFile(join(cwd, "scenario-1", "index.ts"), "utf-8")
+
+    assert.strictEqual(result, "/**\n" +
+      " * @file Automatically generated by @tsed/barrels.\n" +
+      " */\n" +
+      "export * from \"./file1.js\";\n" +
+      "export * from \"./file2.js\";\n" +
+      "export * from \"./sub-directory/file2.js\";\n")
+  })
+})