test/update commit order

Signed-off-by: shmck <shawn.j.mckay@gmail.com>

src/utils/validateCommits.ts

@@ -13,8 +13,8 @@ export function validateCommitOrder(positions: string[]): boolean {
       current = { level: 0, step: 0 };
       return;
     } else {
-      const levelMatch = position.match(/^L([0-9]+)Q?$/);
-      const stepMatch = position.match(/^L([0-9]+)S([0-9]+)[Q|A]?$/);
+      const levelMatch = position.match(/^L?([0-9]+)Q?$/);
+      const stepMatch = position.match(/^L?([0-9]+)[S|\.]([0-9]+)[Q|A]?$/);
       if (levelMatch) {
         // allows next level or step
         const [_, levelString] = levelMatch;

tests/commitOrder.test.ts

@@ -1,47 +1,95 @@
 import { validateCommitOrder } from "../src/utils/validateCommits";
 
 describe("commitOrder", () => {
-  it("should return true if order is valid", () => {
-    const positions = ["INIT", "1", "1.1", "1.2", "2", "2.1"];
-    const result = validateCommitOrder(positions);
-    expect(result).toBe(true);
+  describe("#.# format", () => {
+    it("should return true if order is valid", () => {
+      const positions = ["INIT", "1", "1.1", "1.2", "2", "2.1"];
+      const result = validateCommitOrder(positions);
+      expect(result).toBe(true);
+    });
+    it("should return true if valid with duplicates", () => {
+      const positions = [
+        "INIT",
+        "INIT",
+        "1",
+        "1",
+        "1.1",
+        "1.1",
+        "1.2",
+        "1.2",
+        "2",
+        "2",
+        "2.1",
+        "2.1",
+      ];
+      const result = validateCommitOrder(positions);
+      expect(result).toBe(true);
+    });
+    it("should return false if INIT is out of order", () => {
+      const positions = ["INIT", "1", "1.1", "1.2", "INIT", "2", "2.1"];
+      const result = validateCommitOrder(positions);
+      expect(result).toBe(false);
+    });
+    it("should return false if level after step is out of order", () => {
+      const positions = ["INIT", "1", "1.1", "1.2", "2.1", "2"];
+      const result = validateCommitOrder(positions);
+      expect(result).toBe(false);
+    });
+    it("should return false if level is out of order", () => {
+      const positions = ["INIT", "1", "L3", "2"];
+      const result = validateCommitOrder(positions);
+      expect(result).toBe(false);
+    });
+    it("should return false if step is out of order", () => {
+      const positions = ["INIT", "1", "1.1", "1.3", "1.2"];
+      const result = validateCommitOrder(positions);
+      expect(result).toBe(false);
+    });
   });
-  it("should return true if valid with duplicates", () => {
-    const positions = [
-      "INIT",
-      "INIT",
-      "1",
-      "1",
-      "1.1",
-      "1.1",
-      "1.2",
-      "1.2",
-      "2",
-      "2",
-      "2.1",
-      "2.1",
-    ];
-    const result = validateCommitOrder(positions);
-    expect(result).toBe(true);
-  });
-  it("should return false if INIT is out of order", () => {
-    const positions = ["INIT", "1", "1.1", "1.2", "INIT", "2", "2.1"];
-    const result = validateCommitOrder(positions);
-    expect(result).toBe(false);
-  });
-  it("should return false if level after step is out of order", () => {
-    const positions = ["INIT", "1", "1.1", "1.2", "2.1", "2"];
-    const result = validateCommitOrder(positions);
-    expect(result).toBe(false);
-  });
-  it("should return false if level is out of order", () => {
-    const positions = ["INIT", "1", "L3", "2"];
-    const result = validateCommitOrder(positions);
-    expect(result).toBe(false);
-  });
-  it("should return false if step is out of order", () => {
-    const positions = ["INIT", "1", "1.1", "1.3", "1.2"];
-    const result = validateCommitOrder(positions);
-    expect(result).toBe(false);
+  // @deprecated
+  describe("L#S# format", () => {
+    it("should return true if order is valid", () => {
+      const positions = ["INIT", "L1", "L1S1", "L1S2", "L2", "L2S1"];
+      const result = validateCommitOrder(positions);
+      expect(result).toBe(true);
+    });
+    it("should return true if valid with duplicates", () => {
+      const positions = [
+        "INIT",
+        "INIT",
+        "L1",
+        "L1",
+        "L1S1",
+        "L1S1",
+        "L1S2",
+        "L1S2",
+        "L2",
+        "L2",
+        "L2S1",
+        "L2S1",
+      ];
+      const result = validateCommitOrder(positions);
+      expect(result).toBe(true);
+    });
+    it("should return false if INIT is out of order", () => {
+      const positions = ["INIT", "L1", "L1S1", "L1S2", "INIT", "L2", "L2S1"];
+      const result = validateCommitOrder(positions);
+      expect(result).toBe(false);
+    });
+    it("should return false if level after step is out of order", () => {
+      const positions = ["INIT", "L1", "L1S1", "L1S2", "L2S1", "L2"];
+      const result = validateCommitOrder(positions);
+      expect(result).toBe(false);
+    });
+    it("should return false if level is out of order", () => {
+      const positions = ["INIT", "L1", "L3", "L2"];
+      const result = validateCommitOrder(positions);
+      expect(result).toBe(false);
+    });
+    it("should return false if step is out of order", () => {
+      const positions = ["INIT", "L1", "L1S1", "L1S3", "L1S2"];
+      const result = validateCommitOrder(positions);
+      expect(result).toBe(false);
+    });
   });
 });