Commit pattern upgrade

index.js

@@ -5,6 +5,9 @@ const { join } = require("path");
 const { spawn } = require("child_process");
 const { readFile } = require("fs");
 
+//This is a common source of truth for both the actual action and unit tests
+const commitRegex = "^v?(\\d+\\.\\d+\\.\\d+(?:-(?:alpha|beta)\\.\\d+)?)$";
+
 async function main() {
   const dir =
     process.env.WORKSPACE ||
@@ -24,8 +27,7 @@ async function main() {
     throw new NeutralExitError();
   }
 
-  const commitPattern =
-    getEnv("COMMIT_PATTERN") || "^(?:Release|Version) (\\S+)";
+  const commitPattern = getEnv("COMMIT_PATTERN") || commitRegex;
 
   const { name, email } = eventObj.repository.owner;
 
@@ -191,3 +193,7 @@ if (require.main === module) {
     }
   });
 }
+
+module.exports = {
+  commitRegex
+};

package.json

@@ -9,10 +9,13 @@
     "npm-publish-action": "./index.js"
   },
   "scripts": {
-    "lint": "eslint ."
+    "lint": "eslint .",
+    "test": "mocha tests.js"
   },
   "devDependencies": {
-    "eslint": "^7.5.0"
+    "chai": "^4.2.0",
+    "eslint": "^7.5.0",
+    "mocha": "^8.0.1"
   },
   "eslintConfig": {
     "parserOptions": {

tests.js

@@ -0,0 +1,75 @@
+const { commitRegex } = require("./index.js");
+const { expect } = require("chai");
+
+const regex = new RegExp(commitRegex);
+
+describe("commit regex", () => {
+  it("will accept single digit semantic versions", () => {
+    expect("1.2.3".match(regex)).to.have.length(2);
+    expect("1.2.3".match(regex)[1]).to.equal("1.2.3");
+
+    expect("v1.2.3".match(regex)).to.have.length(2);
+    expect("v1.2.3".match(regex)[1]).to.equal("1.2.3");
+  });
+
+  it("will accept multi digit semantic versions", () => {
+    expect("10.11.12".match(regex)).to.have.length(2);
+    expect("10.11.12".match(regex)[1]).to.equal("10.11.12");
+
+    expect("v10.11.12".match(regex)).to.have.length(2);
+    expect("v10.11.12".match(regex)[1]).to.equal("10.11.12");
+  });
+
+  it("will accept single digit semantic versions with prerelease suffixes", () => {
+    expect("1.2.3-alpha.0".match(regex)).to.have.length(2);
+    expect("1.2.3-alpha.0".match(regex)[1]).to.equal("1.2.3-alpha.0");
+
+    expect("1.2.3-beta.0".match(regex)).to.have.length(2);
+    expect("1.2.3-beta.0".match(regex)[1]).to.equal("1.2.3-beta.0");
+
+    expect("v1.2.3-alpha.0".match(regex)).to.have.length(2);
+    expect("v1.2.3-alpha.0".match(regex)[1]).to.equal("1.2.3-alpha.0");
+
+    expect("v1.2.3-beta.0".match(regex)).to.have.length(2);
+    expect("v1.2.3-beta.0".match(regex)[1]).to.equal("1.2.3-beta.0");
+  });
+
+  it("will accept multi digit semantic versions with prerelease suffixes", () => {
+    expect("10.11.12-alpha.0".match(regex)).to.have.length(2);
+    expect("10.11.12-alpha.0".match(regex)[1]).to.equal("10.11.12-alpha.0");
+
+    expect("10.11.12-beta.0".match(regex)).to.have.length(2);
+    expect("10.11.12-beta.0".match(regex)[1]).to.equal("10.11.12-beta.0");
+
+    expect("v10.11.12-alpha.0".match(regex)).to.have.length(2);
+    expect("v10.11.12-alpha.0".match(regex)[1]).to.equal("10.11.12-alpha.0");
+
+    expect("v10.11.12-beta.0".match(regex)).to.have.length(2);
+    expect("v10.11.12-beta.0".match(regex)[1]).to.equal("10.11.12-beta.0");
+  });
+
+  it('will not accept non "v" prefixes', () => {
+    expect("x1.2.3".match(regex)).to.be.null;
+    expect("x10.11.12".match(regex)).to.be.null;
+  });
+
+  it("will not accept unknown prerelease suffixes", () => {
+    expect("1.2.3-gamma.0".match(regex)).to.be.null;
+
+    expect("v1.2.3-gamma.0".match(regex)).to.be.null;
+
+    expect("10.11.12-gamma.0".match(regex)).to.be.null;
+
+    expect("v10.11.12-gamma.0".match(regex)).to.be.null;
+  });
+
+  it("will not accept invalid prerelease suffixes", () => {
+    expect("1.2.3-alpha".match(regex)).to.be.null;
+
+    expect("v1.2.3-alpha".match(regex)).to.be.null;
+
+    expect("10.11.12-alpha".match(regex)).to.be.null;
+
+    expect("v10.11.12-alpha".match(regex)).to.be.null;
+  });
+});