covers all the requirements in readme file

covers readme file

Author: ebrahimbeiati

implement-shell-tools/cat/cat.js

@@ -8,6 +8,10 @@ function printFile(filePath, options) {
     const content = fs.readFileSync(filePath, "utf-8");
     const lines = content.split("\n");
 
+    if (lines.length && lines[lines.length - 1] === "") {
+        lines.pop();
+    }
+
     lines.forEach((line) => {
       let prefix = "";
 
@@ -24,6 +28,7 @@ function printFile(filePath, options) {
     });
   } catch (error) {
     console.error(`cat: ${filePath}: ${error.message}`);
+    process.exitCode = 1;
   }
 }
 
@@ -51,6 +56,9 @@ function main() {
   }
 
   files.forEach((file) => {
+    if(options.numberMode) {
+      globalLineCounter = 1; //reset line counter for each file if numbering is enabled
+    }
     printFile(file, options);
   });
 }

implement-shell-tools/ls/ls.js

@@ -20,13 +20,11 @@ function listDirectory(dirPath, showAll, onePerLine) {
 }
 function main() {
   const args = process.argv.slice(2);
-  // Check for options
   const showAll = args.includes("-a");
   const onePerLine = args.includes("-1");
   //remove options from args
   let directories = args.filter((arg) => arg !== "-a" && arg !== "-1");
 
-  // Default to current directory if no directories are specified
   if (directories.length === 0) {
     directories = [process.cwd()];
   }
@@ -35,7 +33,6 @@ function main() {
       const stats = fs.statSync(arg);
 
       if (stats.isDirectory()) {
-        //Print header if multiple directories are listed
         if (directories.length > 1) {
           console.log(`${arg}:`)
         };

implement-shell-tools/wc/wc.js

@@ -2,7 +2,8 @@
 const fs = require("node:fs");
 // Function to count lines, words, and bytes in a file
 function countFileContent(content) {
-  const lines = content.split("\n").length; // Count lines by splitting on newline characters
+  const rawLines = content.split("\n");
+  const lines = rawLines[rawLines.length - 1] === "" ? rawLines.length - 1 : rawLines.length;
   const words = content.trim().split(/\s+/).filter(Boolean).length; // Split by whitespace and filter out empty strings
   const bytes = Buffer.byteLength(content, "utf8");
   return { lines, words, bytes };

package.json

@@ -0,0 +1,15 @@
+{
+  "name": "module-tools",
+  "version": "1.0.0",
+  "description": "Exercises to practice and solidify your understanding of the Tools module of the Software Development Course.",
+  "main": "index.js",
+  "directories": {
+    "test": "tests"
+  },
+  "scripts": {
+    "test": "node tests/runIntegrationTests.js"
+  },
+  "keywords": [],
+  "author": "",
+  "license": "ISC"
+}

tests/runIntegrationTests.js

@@ -0,0 +1,96 @@
+const { execSync } = require("node:child_process");
+
+function runCommand(command) {
+  return execSync(command, { encoding: "utf-8" }).trim();
+}
+
+function normalizeOutput(output) {
+  return output.replaceAll(/\s+/g, " ").trim();
+}
+
+function test(name, systemCommand, nodeCommand) {
+  try {
+    const expected = normalizeOutput(runCommand(systemCommand));
+    const actual = normalizeOutput(runCommand(nodeCommand));
+    if (expected === actual) {
+      console.log(`✅ ${name} passed`);
+    } else {
+      console.error(`❌ ${name} failed`);
+      console.error(`Expected: "${expected}"`);
+      console.error(`Actual:   "${actual}"`);
+    }
+  } catch (error) {
+    console.error(`❌ ${name} failed with error: ${error.message}`);
+  }
+}
+
+/* CAT TESTS */
+test(
+  "cat single file",
+  "cat implement-shell-tools/cat/sample-files/1.txt",
+  "node implement-shell-tools/cat/cat.js implement-shell-tools/cat/sample-files/1.txt",
+);
+
+test(
+  "cat -n single file",
+  "cat -n implement-shell-tools/cat/sample-files/1.txt",
+  "node implement-shell-tools/cat/cat.js -n implement-shell-tools/cat/sample-files/1.txt",
+);
+
+test(
+  "cat multiple files",
+  "cat implement-shell-tools/cat/sample-files/*.txt",
+  "node implement-shell-tools/cat/cat.js implement-shell-tools/cat/sample-files/*.txt",
+);
+
+test(
+  "cat -n multiple files",
+  "cat -n implement-shell-tools/cat/sample-files/*.txt",
+  "node implement-shell-tools/cat/cat.js -n implement-shell-tools/cat/sample-files/*.txt",
+);
+
+test(
+  "cat -b file",
+  "cat -b implement-shell-tools/cat/sample-files/3.txt",
+  "node implement-shell-tools/cat/cat.js -b implement-shell-tools/cat/sample-files/3.txt",
+);
+
+/* LS TESTS */
+
+test(
+  "ls sample-files",
+  "ls implement-shell-tools/ls/sample-files",
+  "node implement-shell-tools/ls/ls.js implement-shell-tools/ls/sample-files",
+);
+
+/* WC TESTS */
+
+test(
+  "wc all files",
+  "wc implement-shell-tools/wc/sample-files/*",
+  "node implement-shell-tools/wc/wc.js implement-shell-tools/wc/sample-files/*",
+);
+
+test(
+  "wc -l single file",
+  "wc -l implement-shell-tools/wc/sample-files/3.txt",
+  "node implement-shell-tools/wc/wc.js -l implement-shell-tools/wc/sample-files/3.txt",
+);
+
+test(
+  "wc -w single file",
+  "wc -w implement-shell-tools/wc/sample-files/3.txt",
+  "node implement-shell-tools/wc/wc.js -w implement-shell-tools/wc/sample-files/3.txt",
+);
+
+test(
+  "wc -c single file",
+  "wc -c implement-shell-tools/wc/sample-files/3.txt",
+  "node implement-shell-tools/wc/wc.js -c implement-shell-tools/wc/sample-files/3.txt",
+);
+
+test(
+  "wc -l multiple files",
+  "wc -l implement-shell-tools/wc/sample-files/*",
+  "node implement-shell-tools/wc/wc.js -l implement-shell-tools/wc/sample-files/*",
+);