Refactor: change function behavior

BREAKING CHANGE: rules are now functions where input and config are passed immediatelly

Author: JPeer264

lib/questions.js

@@ -41,7 +41,7 @@ const questions = (config) => {
       name: 'description',
       message: 'Enter your commit message:',
       validate: (input) => {
-        const warnings = ruleWarningMessages(input);
+        const warnings = ruleWarningMessages(input, config);
 
         return warnings || true;
       },

lib/rules/ruleWarningMessages.js

@@ -1,11 +1,9 @@
 import entries from 'object.entries';
-import getConfig from '../getConfig';
 import rules from './availableRules';
 
-const config = getConfig();
-
-const ruleWarningMessages = (input) => {
+const ruleWarningMessages = (input, config) => {
   let warningMessage = '';
+
   const configRuleEntries = entries(config.rules);
 
   configRuleEntries.forEach((rule) => {

test/rules/ruleWarningMessages.js

@@ -1,7 +1,15 @@
 import test from 'ava';
+
 import ruleWaringMessages from '../../lib/rules/ruleWarningMessages';
 
 test('ruleWarningMessages', (t) => {
-  const messages = ruleWaringMessages('input.');
+  const config = {
+    rules: {
+      'max-char': 72,
+      'min-char': 10,
+      'end-with-dot': false,
+    },
+  };
+  const messages = ruleWaringMessages('input.', config);
   t.deepEqual(messages, 'The commit message has to be at least 10 character.\nThe commit message can not end with a dot\n');
 });