Fix: take formattedMessage if editor is undefined (closes #77) (#78)

(ref: #77)

lib/helpers/formatters.js

@@ -27,7 +27,11 @@ export const formatMessage = (answers, argv) => {
 
   const type = combineTypeScope(combinedAnswers.type, combinedAnswers.scope);
   const formattedMessage = `${type} ${(combinedAnswers.message || '').trim()}`;
-  const result = answers.body ? combinedAnswers.editor : formattedMessage;
+  let result = formattedMessage;
+
+  if (combinedAnswers.editor) {
+    result = answers.body ? combinedAnswers.editor : formattedMessage;
+  }
 
   return result;
 };

package.json

@@ -4,8 +4,8 @@
   "description": "A CLI for semantic git commits",
   "main": "dest",
   "bin": {
-    "semantic-git-commit": "dest/index.js",
-    "sgc": "dest/index.js"
+    "semantic-git-commit": "./dest/index.js",
+    "sgc": "./dest/index.js"
   },
   "scripts": {
     "pretest": "npm run lint & npm run babel",

test/helper/formatters.js

@@ -93,3 +93,44 @@ test('FORMATMESSAGE | should format message with more argv overrides', (t) => {
 
   t.is(message, 'myTypeOverwrite (myScopeOverwrite): messageOverwrite');
 });
+
+test('FORMATMESSAGE | should format when editor is undefined but body is set to true', (t) => {
+  const message = formatMessage(
+    {
+      type: 'myType',
+      scope: 'myScope',
+      message: 'message',
+      body: true,
+    },
+  );
+
+  t.is(message, 'myType (myScope): message');
+});
+
+test('FORMATMESSAGE | should take editor when editor is not undefined and body is set to true', (t) => {
+  const message = formatMessage(
+    {
+      type: 'myType',
+      scope: 'myScope',
+      message: 'message',
+      body: true,
+      editor: 'take this',
+    },
+  );
+
+  t.is(message, 'take this');
+});
+
+test('FORMATMESSAGE | should format when editor is not undefined and body is set to false', (t) => {
+  const message = formatMessage(
+    {
+      type: 'myType',
+      scope: 'myScope',
+      message: 'message',
+      body: false,
+      editor: 'take this',
+    },
+  );
+
+  t.is(message, 'myType (myScope): message');
+});