Optimized test case execution time && Tests to verify the api endpoint.

bin/index.js

@@ -77,8 +77,8 @@ program
       console.error(error);
     }
   });
-
-function initCommand(options) {
+//Changing sync to async to optimize further
+async function initCommand(options) {
   const selectedTemplate = options.template || "basic"; // Default to 'basic' if no template is specified
 
   if (!templates[selectedTemplate]) {
@@ -108,7 +108,7 @@ function initCommand(options) {
 
   const copySpinner = createSpinner("Creating server files...").start();
   try {
-    fs.copySync(templatePath, destinationPath);
+    await fs.copy(templatePath, destinationPath);
 
     copySpinner.success({ text: "Created server files successfully." });
   } catch (err) {
@@ -133,7 +133,7 @@ function initCommand(options) {
   const addDependencies = createSpinner("Adding dependency packages...").start();
   try {
     const packageJsonPath = path.join(targetDir, "package.json");
-    const packageJsonContent = fs.readFileSync(packageJsonPath, "utf8");
+    const packageJsonContent = await fs.readJSON(packageJsonPath, "utf8");
     const packageJson = JSON.parse(packageJsonContent);
     packageJson.dependencies = packageJson.dependencies || {};
 

package.json

@@ -6,19 +6,21 @@
   "main": "index.js",
   "scripts": {
     "dev": "node bin/index.js init",
-    "test": "jest"
+    "test": "node --experimental-vm-modules node_modules/jest/bin/jest.js"
   },
   "author": "Abhinav Ramakrishnan, Ashwin Narayanan S",
   "license": "ISC",
   "bin": "./bin/index.js",
   "dependencies": {
     "chalk": "^5.3.0",
     "commander": "^12.1.0",
+    "express": "^4.21.2",
     "figlet": "^1.7.0",
     "fs-extra": "^11.2.0",
-    "nanospinner": "^1.1.0"
+    "nanospinner": "^1.1.0",
+    "supertest": "^7.0.0"
   },
   "devDependencies": {
     "jest": "^29.7.0"
   }
-}
+}

templates/basic/server.js

@@ -7,6 +7,10 @@ app.get("/", (req, res) => {
   return res.send("Hello World!");
 });
 
+if (process.env.NODE_ENV !== "test") {
 app.listen(port, () => {
   console.log(`Example app listening on port ${port}.`);
 });
+}
+
+export default app; 

templates/basic/server.test.js

@@ -0,0 +1,10 @@
+import app from "./server.js";
+import request from "supertest";
+
+describe("API Endpoints", () => {
+  it("should return Hello World on GET /", async () => {
+    const res = await request(app).get("/");
+    expect(res.statusCode).toEqual(200);
+    expect(res.text).toBe("Hello World!");
+  });
+});

test/help.test.js

@@ -1,5 +1,6 @@
-const util = require('node:util');
-const exec = util.promisify(require('node:child_process').exec);
+import { exec as execCallback } from 'child_process';
+import { promisify } from 'util';
+const exec = promisify(execCallback);
 
 const help = `Usage: qse [options] [command]
 

test/init.test.js

@@ -1,33 +1,37 @@
-const fs = require('fs');
-const path = require('path');
-const crypto = require('crypto');
-const util = require('node:util');
-const exec = util.promisify(require('node:child_process').exec);
+import { createHash } from 'node:crypto';
+import path from 'node:path';
+import { existsSync, mkdirSync, readdirSync, readFileSync, rmSync, statSync } from 'node:fs';
+import { fileURLToPath } from 'url';
+import { exec as execCallback } from 'child_process';
+import { promisify } from 'util';
+
+const exec = promisify(execCallback);
+const __dirname = path.dirname(fileURLToPath(import.meta.url));
 const tempDir = path.join(__dirname, 'temp');
 
 function initTempDirectory() {
-    if (fs.existsSync(tempDir)) {
-        fs.rmSync(tempDir, { recursive: true });
+    if (existsSync(tempDir)) {
+        rmSync(tempDir, { recursive: true });
     }
-    fs.mkdirSync(tempDir);
+    mkdirSync(tempDir);
 }
 
 function clearTempDirectory() {
-    if (fs.existsSync(tempDir)) {
-        fs.rmSync(tempDir, { recursive: true });
+    if (existsSync(tempDir)) {
+        rmSync(tempDir, { recursive: true });
     }
 }
 
 function traverseDirectory(dirName, hash) {
-    const files = fs.readdirSync(dirName);
+    const files = readdirSync(dirName);
 
     for (const file of files) {
         const filePath = path.join(dirName, file);
-        const stats = fs.statSync(filePath);
+        const stats = statSync(filePath);
         if (stats.isDirectory()) {
             traverseDirectory(filePath, hash);
         } else {
-            const data = fs.readFileSync(filePath);
+            const data = readFileSync(filePath);
             hash.update(data);
         }
     }
@@ -36,19 +40,19 @@ function traverseDirectory(dirName, hash) {
 // Ignore node_modules and package-lock.json
 // and compute the SHA256 hash.
 function computeSHA256Hash(dirName) {
-    const hash = crypto.createHash('sha256');
-    const files = fs.readdirSync(dirName);
+    const hash = createHash('sha256');
+    const files = readdirSync(dirName);
 
     for (const file of files) {
         if (file === 'node_modules' || file === 'package-lock.json' || file === 'package.json') {
             continue;
         }
         const filePath = path.join(dirName, file);
-        const stats = fs.statSync(filePath);
+        const stats = statSync(filePath);
         if (stats.isDirectory()) {
             traverseDirectory(filePath, hash);
         } else {
-            const data = fs.readFileSync(filePath);
+            const data = readFileSync(filePath);
             hash.update(data);
         }
     }
@@ -66,9 +70,16 @@ describe('init', () => {
     });
 
     test('no templates passed, should default to basic', async () => {
+
+        console.time('Hash Calculation');
         const originalHash = computeSHA256Hash(path.join(__dirname, '..', 'templates', 'basic'));
+        console.timeEnd('Hash Calculation');
+        console.time('Command Execution');
         await exec(`node ../../bin/index.js init`, { cwd: tempDir });
+        console.timeEnd('Command Execution');
+        console.time('Hash Verification');
         const commandHash = computeSHA256Hash(tempDir);
+        console.timeEnd('Hash Verification');
         expect(commandHash).toEqual(originalHash);
     })
 
@@ -97,4 +108,4 @@ describe('init', () => {
         const { stdout, stderr } = await exec(`node ../../bin/index.js init -t invalid_name`, { cwd: tempDir });
         expect(stderr).toContain(`Template invalid_name does not exist. To see available templates use "qse list".`);
     });
-});
+});

test/list.test.js

@@ -1,5 +1,6 @@
-const util = require('node:util');
-const exec = util.promisify(require('node:child_process').exec);
+import { exec as execCallback } from 'child_process';
+import { promisify } from 'util';
+const exec = promisify(execCallback);
 
 const list = `Available Commands:
 - -v, --version: Prints current qse version

test/version.test.js

@@ -1,6 +1,7 @@
-const util = require('node:util');
-const exec = util.promisify(require('node:child_process').exec);
-const packageJson = require('../package.json')
+import packageJson from '../package.json';
+import { exec as execCallback } from 'child_process';
+import { promisify } from 'util';
+const exec = promisify(execCallback);
 
 describe('Version Command', () => {
     test('package.json has version', () => {