fix(function-rule): return success if the function rule is undefined

Author: jdbruijn

src/function-rule.test.ts

@@ -29,7 +29,7 @@ test('returns a value from an implementation function', (t) => {
   t.deepEqual(returnValue, ruleOutcome);
 });
 
-test.failing('returns successful for an undefined implementation', (t) => {
+test('returns successful for an undefined implementation', (t) => {
   const returnValue = functionRule({} as Commit, 'always', undefined);
   t.deepEqual(returnValue, [true]);
 });

src/function-rule.ts

@@ -13,13 +13,16 @@ type FunctionRule = (
 const functionRule: Rule<FunctionRule> = (
   parsed: Commit,
   when: RuleConfigCondition = 'always',
-  value: FunctionRule | undefined,
+  functionRule: FunctionRule | undefined,
 ) => {
-  if (typeof value !== 'function') {
+  if (functionRule === undefined) {
+    return [true];
+  }
+  if (typeof functionRule !== 'function') {
     throw new Error('Not a valid function!');
   }
 
-  return value(parsed, when);
+  return functionRule(parsed, when);
 };
 
 export default functionRule;