fix: allow file-path to be accepted by disassemble

README.md

@@ -29,7 +29,7 @@ npm install xml-disassembler
 
 ## Disassembling Files
 
-Disassemble 1 or multiple XML files in the immediate directory (`xmlPath`), without recursion. Each XML file will be disassembled into their own sub-directories using their base name (everything before the first `.` in the file-name).
+Disassemble 1 XML file or multiple XML files in the immediate directory, without recursion. Each XML file will be disassembled into their own sub-directories using their base name (everything before the first `.` in the file-name).
 
 Example:
 
@@ -100,7 +100,7 @@ Import the `DisassembleXMLFileHandler` class from the package.
 ```typescript
 /* 
 FLAGS
-- xmlPath: Directory containing the XML files to disassemble (must be directory). This will only disassemble files in the immediate directory.
+- xmlPath: Path to 1 XML file or a directory of XML files to disassemble. If the path provided is a directory, only the files in the immediate directory will be disasssembled.
 - uniqueIdElements: (Optional) Comma-separated list of unique and required ID elements used to name disassembled files for nested elements. 
                                Defaults to SHA-256 hash if unique ID elements are undefined or not found.
 - prePurge:  (Optional) Boolean value. If set to true, purge pre-existing disassembled directories prior to disassembling the file.

src/service/disassembleXMLFileHandler.ts

@@ -19,23 +19,33 @@ export class DisassembleXMLFileHandler {
     } = xmlAttributes;
     const fileStat = await fs.stat(xmlPath);
 
-    if (!fileStat.isDirectory()) {
-      logger.error(
-        `The provided xmlPath ${xmlPath} to disassemble is not a directory.`,
-      );
-      return;
-    }
-    const files = await fs.readdir(xmlPath);
-    for (const file of files) {
-      const filePath = path.join(xmlPath, file);
-      if (filePath.endsWith(".xml")) {
-        await this.processFile({
-          xmlPath,
-          filePath,
-          uniqueIdElements,
-          prePurge,
-          postPurge,
-        });
+    if (fileStat.isFile()) {
+      const filePath = path.resolve(xmlPath);
+      if (!filePath.endsWith(".xml")) {
+        logger.error(`The file path ${filePath} is not an XML file.`);
+        return;
+      }
+      const basePath = path.dirname(filePath);
+      await this.processFile({
+        xmlPath: basePath,
+        filePath,
+        uniqueIdElements,
+        prePurge,
+        postPurge,
+      });
+    } else if (fileStat.isDirectory()) {
+      const files = await fs.readdir(xmlPath);
+      for (const file of files) {
+        const filePath = path.join(xmlPath, file);
+        if (filePath.endsWith(".xml")) {
+          await this.processFile({
+            xmlPath,
+            filePath,
+            uniqueIdElements,
+            prePurge,
+            postPurge,
+          });
+        }
       }
     }
   }

test/main.spec.ts

@@ -148,7 +148,7 @@ describe("main function", () => {
   });
   it("should disassemble a XML file with no namespace.", async () => {
     await disassembleHandler.disassemble({
-      xmlPath: "mock/no-namespace",
+      xmlPath: "mock/no-namespace/HR_Admin.permissionset-meta.xml",
       uniqueIdElements:
         "application,apexClass,name,externalDataSource,flow,object,apexPage,recordType,tab,field",
     });
@@ -167,11 +167,15 @@ describe("main function", () => {
 
     expect(logger.error).not.toHaveBeenCalled();
   });
-  it("should test disassemble error condition (file path provided).", async () => {
+  it("should test disassemble error condition (XML file path not provided).", async () => {
+    let fakeFile = "mock/not-an-xml.txt";
+    fakeFile = path.resolve(fakeFile);
+    const fakeFileContents = "Testing error condition.";
+    fs.writeFileSync(fakeFile, fakeFileContents);
     await disassembleHandler.disassemble({
-      xmlPath: "mock/no-namespace/HR_Admin.permissionset-meta.xml",
+      xmlPath: fakeFile,
     });
-
+    fs.unlinkSync(fakeFile);
     expect(logger.error).toHaveBeenCalled();
   });
   it("should test reassemble error condition (file path provided).", async () => {