Merge pull request #105 from nodecraft/refactor/tests

Author: Cherry

.eslintignore

@@ -1,3 +1,4 @@
 node_modules/**
 build/**
+coverage/**
 test/json/**

.github/workflows/test.yml

@@ -7,12 +7,12 @@ jobs:
     strategy:
       matrix:
         os: [ubuntu-latest, windows-latest, macOS-latest]
-        node: [20, 22]
+        node: [22, 24]
     name: Node ${{ matrix.node }} ${{ matrix.os }} Test
     steps:
-      - uses: actions/checkout@v4
+      - uses: actions/checkout@v5
       - name: Setup node
-        uses: actions/setup-node@v4
+        uses: actions/setup-node@v5
         with:
           node-version: ${{ matrix.node }}
           cache: "npm"

.mocharc.json

@@ -26,14 +26,23 @@ const helpers = require('./helpers.js');
 class spawnpoint extends EventEmitter {
 	/**
 	 * Creates new instance of spawnpoint
-	 * @param  {string} [configFile] Sets the JSON file spawnpoint uses to setup the framework.
+	 * @param  {string|Object} [configFile] Sets the JSON file spawnpoint uses to setup the framework, or an options object.
+	 * @param  {string} [configFile.configFile='/config/app.json'] Path to the config file when using options object.
+	 * @param  {string} [configFile.cwd=process.cwd()] Working directory for the applicationndash avoids needing process.chdir().
 	 * @return {this}
 	 */
 	constructor(configFile = '/config/app.json') {
 		// init EventEmitter
 		super();
 
-		if (typeof(configFile) !== 'string') {
+		// Support options object for more flexibility
+		let options = {};
+		if (typeof configFile === 'object' && configFile !== null && !Array.isArray(configFile)) {
+			options = configFile;
+			configFile = options.configFile || '/config/app.json';
+		}
+
+		if (typeof configFile !== 'string') {
 			throw new TypeError('`configFile` must be a path string to a Spawnpoint config file.');
 		}
 		if (!configFile.endsWith('.json') && !configFile.endsWith('.js')) {
@@ -51,8 +60,8 @@ class spawnpoint extends EventEmitter {
 			stopAttempts: 0, // how many attempts to stop have been triggered
 		};
 
-		// app CWD
-		this.cwd = process.cwd();
+		// app CWD - can be passed via options to avoid process.chdir()
+		this.cwd = options.cwd || process.cwd();
 
 		// detect if we are in a container (lazy-loaded on first access)
 		this._containerized = null;
@@ -283,6 +292,8 @@ class spawnpoint extends EventEmitter {
 		if (typeof(exts) === 'string') {
 			exts = [exts];
 		}
+		// normalize path separators for cross-platform support
+		dir = dir.replaceAll('\\', '/');
 		const list = [];
 		let stat;
 		try {
@@ -293,8 +304,8 @@ class spawnpoint extends EventEmitter {
 		if (!stat || !stat.isDirectory()) {
 			return list;
 		}
-		// ensure proper trailing slash and normalize path separators
-		dir = String(dir + '/').replaceAll('\\', '/');
+		// ensure proper trailing slash
+		dir = String(dir + '/');
 
 		// Use withFileTypes to avoid a stat for every entry (significantly faster)
 		const stack = [dir];