✅ Test: add test for new rules

Author: aichbauer

test/fixtures/.sgcrc

@@ -5,5 +5,13 @@
       "type": "Add:",
       "description": "Files added"
     }
+  ],
+  "rules": [
+    {
+    "max-char-commit-message": true,
+    "max-char": 72,
+    "min-char": 10,
+    "end-with-dot": false
+    }
   ]
 }

test/prompConfig.js

@@ -11,41 +11,38 @@ test('get configuration file equals .sgcrc_default', (t) => {
   t.deepEqual(getConfig(), json.readToObjSync(path.join(cwd, '.sgcrc_default')));
 });
 
-test('choices are the same as choices generated from .sgcrc_default', (t) => {
-  const sgc = getConfig();
-  const configuration = getConfig();
-  const choicesList = choices(configuration);
+test('choices are the same as choices generated from .sgcrc_default', async (t) => {
+  const sgc = await getConfig();
+  const choicesList = await choices(sgc);
   const choicesArray = [];
 
-  Promise.resolve(() => {
-    sgc.types.forEach((type) => {
-      const emoji = `${type.emoji} ` || '';
-      const configType = type.type;
-      const description = type.description || '';
-
-      choicesArray.push({
-        value: emoji + configType,
-        name: `${chalk.bold(configType)} ${description}`,
-      });
-    }).then(() => {
-      t.deepEqual(choicesList, choicesArray);
+  sgc.types.forEach((type) => {
+    const emoji = `${type.emoji} ` || '';
+    const configType = type.type;
+    const description = type.description || '';
+
+    choicesArray.push({
+      value: emoji + configType,
+      name: `${chalk.bold(configType)} ${description}`,
     });
   });
+
+  t.deepEqual(choicesList, await choicesArray);
 });
 
-test('check the values of the question object', (t) => {
-  const configuration = getConfig();
-  const choicesList = choices(configuration);
-  const questionsList = questions(choicesList);
+test('check the values of the question object', async (t) => {
+  const configuration = await getConfig();
+  const choicesList = await choices(configuration);
+  const questionsList = await questions(choicesList, configuration);
 
   t.deepEqual(typeof questionsList, 'object');
 });
 
-test('validate functions in questions', (t) => {
-  const configuration = getConfig();
-  const choicesList = choices(configuration);
-  const questionsList = questions(choicesList);
+test('validate functions in questions', async (t) => {
+  const configuration = await getConfig();
+  const choicesList = await choices(configuration);
+  const questionsList = await questions(choicesList, configuration);
 
   t.deepEqual(questionsList[1].validate('input text'), true);
-  t.deepEqual(questionsList[1].validate(), 'A commit message is mandatory!');
+  t.deepEqual(questionsList[1].validate('This message has over 72 characters so test will fails OMG really, it is so looooooooooooooong'), 'The commit message is not allowed to be longer as 72. Consider writing a body.');
 });

test/rulesConfig.js

@@ -0,0 +1,17 @@
+import test from 'ava';
+import { checkRulesMaxLength, checkRulesMinLength, checkRulesEndWithDot } from '../lib/rulesConfig';
+
+test('checkRulesMaxLength', (t) => {
+  t.deepEqual(checkRulesMaxLength('1234'), true);
+  t.deepEqual(checkRulesMaxLength('This is a very loooooooooooooooooooooooooong commit message with over 72 characters'), false);
+});
+
+test('checkRulesMinLength', (t) => {
+  t.deepEqual(checkRulesMinLength('1234'), false);
+  t.deepEqual(checkRulesMinLength('This is a a commit message'), true);
+});
+
+test('checkRulesEndWithDot', (t) => {
+  t.deepEqual(checkRulesEndWithDot('1234'), true);
+  t.deepEqual(checkRulesEndWithDot('This is a very a commit message with a dot.'), false);
+});