Feat: throw on not valid route

JPeer264 · Sep 6, 2018 · 08ea1ed · 08ea1ed
1 parent c8361b5
commit 08ea1ed
Show file tree

Hide file tree

Showing 2 changed files with 17 additions and 0 deletions.
diff --git a/lib/rules/ruleWarningMessages.js b/lib/rules/ruleWarningMessages.js
@@ -8,6 +8,11 @@ const ruleWarningMessages = (input, config) => {
 
   configRuleEntries.forEach((rule) => {
     const ruleName = rule[0];
+
+    if (!(rules[ruleName] instanceof Function)) {
+      throw new Error(`The rule '${ruleName}' does not exist, look at the documentation to check valid rules.`);
+    }
+
     const ruleIs = rules[ruleName](input, config).check();
 
     if (!ruleIs) {

diff --git a/test/rules/ruleWarningMessages.js b/test/rules/ruleWarningMessages.js
@@ -13,3 +13,15 @@ test('ruleWarningMessages', (t) => {
   const messages = ruleWaringMessages('input.', config);
   t.deepEqual(messages, 'The commit message has to be at least 10 character, but is only 6 character long.\nThe commit message can not end with a dot\n');
 });
+
+test('should throw an error', (t) => {
+  const config = {
+    rules: {
+      notExisting: 10,
+    },
+  };
+
+  const error = t.throws(() => ruleWaringMessages('input.', config));
+
+  t.true(error instanceof Error);
+});