test: Enhance CLI tests for output directory handling and add downloa… (#65)

* test: Enhance CLI tests for output directory handling and add downloadFiles tests

* lint

* chore: Update test:ci script to include experimental test module mocks

* fix: download-files.test.js

Author: usrrname

cli/test/cli.test.js

@@ -1,5 +1,5 @@
 // @ts-nocheck
-import * as assert from 'node:assert';
+import assert from 'node:assert/strict';
 import { execFile } from 'node:child_process';
 import { existsSync } from 'node:fs';
 import * as fs from 'node:fs/promises';
@@ -11,11 +11,9 @@ import { projectRoot } from '../index.mjs';
 const execFileAsync = promisify(execFile);
 describe('CLI', () => {
     describe('default', () => {
-        test('should accept default command with no flags and download the cursorrules', async () => {
+        test('should accept default command with no flags', async () => {
             const { stdout, stderr } = await execFileAsync('node', ['./cli/index.mjs']);
-            if (existsSync(path.join(projectRoot, 'output', '.cursor/rules'))) {
-                assert.ok(stdout.includes('Success'));
-            }
+            assert.ok(stdout.includes('Loading @usrrname/cursorrules ...'));
         });
         after(async () => {
             try {
@@ -52,6 +50,14 @@ describe('CLI', () => {
 
     describe('output', () => {
 
+        test('should accept --output flag for output directory and download the cursorrules', async () => {
+            const { stdout, stderr } = await execFileAsync('node', ['./cli/index.mjs', '--output', 'output']);
+
+            if (existsSync(path.join(projectRoot, 'output'))) {
+                assert.ok(stdout.includes('Success'));
+            }
+        })
+
         test('should accept -o flag for output directory and download the cursorrules', async () => {
             const { stdout, stderr } = await execFileAsync('node', ['./cli/index.mjs', '-o', 'output']);
 
@@ -105,22 +111,24 @@ describe('CLI', () => {
                 console.error(`Error: ${err}`);
             }
         })
-        describe('output directory Validation', () => {
+
+        describe('output directory validation', () => {
             const testBaseDir = path.join(projectRoot, 'test-output-validation');
 
-            beforeEach(async () => {
+            beforeEach(async (t) => {
                 await fs.mkdir(testBaseDir, { recursive: true });
             });
 
-            afterEach(async () => {
+            afterEach(async (t) => {
                 if (existsSync(testBaseDir)) {
                     await fs.rm(testBaseDir, { recursive: true, force: true });
                 }
             });
 
-            test('should accept a valid output directory name within project root', async () => {
+            test('should accept a valid output directory name within project root', async (t) => {
                 const validDir = testBaseDir;
                 const { stdout } = await execFileAsync('node', ['./cli/index.mjs', '-o', validDir]);
+
                 assert.ok(stdout.includes('Success'));
                 assert.ok(existsSync(validDir));
             });

cli/test/download-files.test.js

@@ -0,0 +1,81 @@
+// @ts-nocheck
+import { strict as assert } from 'node:assert';
+import { execFile } from 'node:child_process';
+import { existsSync } from 'node:fs';
+import * as fs from 'node:fs/promises';
+import * as path from 'node:path';
+import { afterEach, before, describe, mock, test } from 'node:test';
+import { promisify } from 'node:util';
+import { projectRoot } from '../index.mjs';
+const execFileAsync = promisify(execFile);
+describe('downloadFiles', () => {
+    let downloadFilesMock;
+    let downloadSelectedFilesMock;
+    let validDir = '';
+
+    before(async (t) => {
+        const downloadFilesModule = await import('../utils/download-files.mjs');
+
+        downloadFilesMock = mock.fn((...args) => downloadFilesModule.downloadFiles(...args));
+        downloadSelectedFilesMock = mock.fn((...args) => downloadFilesModule.downloadSelectedFiles(...args));
+
+        mock?.module?.('../utils/download-files.mjs', () => {
+            return {
+                cache: true,
+                downloadFiles: downloadFilesMock,
+                downloadSelectedFiles: downloadSelectedFilesMock,
+            }
+        });
+        validDir = path.join(projectRoot, 'test-download-files');
+    });
+
+    afterEach(async (t) => {
+        if (existsSync(validDir)) {
+            await fs.rm(validDir, { recursive: true, force: true });
+        }
+    });
+
+    test('should call downloadFiles with valid directory', async (t) => {
+        try {
+
+            const { stdout } = await execFileAsync('node', ['./cli/index.mjs', '-o', validDir]);
+            assert.ok(stdout.includes('Downloading'));
+
+            assert.call(downloadFilesMock.mockImplementationOnce((validDir) => {
+                assert.ok(existsSync(validDir));
+                assert.strictEqual(t.error?.code, 1)
+            }));
+
+
+        } catch (error) {
+
+        }
+    })
+
+    test('should call downloadSelectedFiles with valid directory', async (t) => {
+
+        try {
+            const mockSelectedFiles = {
+                'standards': [
+                    'test-rule-1.mdc',
+                    'test-rule-2.mdc',
+                ],
+            }
+
+            const { stdout } = await execFileAsync('node', ['./cli/index.mjs', '-io', validDir]);
+            assert.call(downloadSelectedFilesMock?.mockImplementationOnce((validDir, mockSelectedFiles) => {
+                assert.strictEqual(validDir, validDir);
+                assert.strictEqual(mockSelectedFiles, mockSelectedFiles);
+            }));
+
+
+            assert.ok(stdout.includes('Copied'));
+            assert.ok(existsSync(validDir));
+            assert.equal(downloadSelectedFilesMock?.mock.calls?.length, 1);
+
+        } catch (error) {
+            assert.strictEqual(error.code, 1);
+        }
+
+    })
+})

package.json

@@ -22,7 +22,7 @@
     "clear": "npm unpublish --registry http://localhost:4873/ --scope=@usrrname",
     "lint": "tsc --noEmit --project tsconfig.json",
     "test": "node --test --watch",
-    "test:ci": "node --test cli/test/*"
+    "test:ci": "node --test cli/test/*  --experimental-test-module-mocks"
   },
   "repository": {
     "type": "git",