Improve error message for space between <TYPE> and <MODULE>

Signed-off-by: Matthew Peveler <[email protected]>

Author: MasterOdin

dist/index.js

@@ -49,6 +49,12 @@ export function checkTitle(fullTitle: string): true {
     }
   }
 
+  if (/^\[[a-zA-Z0-9\\/]+:[ ]+[a-zA-Z0-9\\/]+?\]/.test(title)) {
+    throw new Error(
+      "Unexpected space between <TYPE> and <MODULE> (e.g. [<TYPE>: <MODULE>]), there should be no space (e.g. [<TYPE>:<MODULE>])."
+    );
+  }
+
   if (!/^\[[a-zA-Z0-9\\/]+(?::[a-zA-Z0-9\\/]+)?\] /.test(title)) {
     throw new Error(
       `Invalid title format, must start with ${

dist/index.js.map

@@ -52,6 +52,10 @@ describe("validate", () => {
             "[SECURITY][Refactor:Autograding]foo",
             "Invalid title format, must start with [SECURITY][<TYPE>:<MODULE>] and have space before description.",
           ],
+          [
+            "[Bugfix: Submission] xxx",
+            "Unexpected space between <TYPE> and <MODULE> (e.g. [<TYPE>: <MODULE>]), there should be no space (e.g. [<TYPE>:<MODULE>]).",
+          ],
           [
             "[SYSADMIN ACTION][SECURITY][Refactor:Autograding]foo",
             "Invalid title format, must start with [SYSADMIN ACTION][SECURITY][<TYPE>:<MODULE>] and have space before description.",