fix: rules regex (!~)[does not match operator] returning wrong result (#1633)

Author: ANGkeith

src/utils.ts

@@ -294,9 +294,8 @@ export class Utils {
             return `.matchRE2JS(${_rhs}) ${_operator} null`;
         });
 
-        // Convert all null.match functions to false
         evalStr = evalStr.replace(/null.matchRE2JS\(.+?\)\s*!=\s*null/g, "false");
-        evalStr = evalStr.replace(/null.matchRE2JS\(.+?\)\s*==\s*null/g, "false");
+        evalStr = evalStr.replace(/null.matchRE2JS\(.+?\)\s*==\s*null/g, "true");
 
         evalStr = evalStr.trim();
 

tests/rules.test.ts

@@ -38,7 +38,7 @@ test("Negated regex on undef var", () => {
     ];
     const variables = {};
     const rulesResult = Utils.getRulesResult({argv, cwd: "", rules, variables}, gitData);
-    expect(rulesResult).toEqual({when: "never", allowFailure: false, variables: undefined});
+    expect(rulesResult).toEqual({when: "on_success", allowFailure: false, variables: undefined});
 });
 
 test("GITLAB_CI fail and fallback", () => {