rejoinder - npm Package Compare versions

Comparing version 0.3.7 to 0.3.8

lib/Echo/Echo.js

 #!/usr/bin/nodejs
-/*jshint esnext:true */
 
@@ -106,3 +105,3 @@ /*
                 if(fname)
-                    return this.withPostfix(postfix, msg.join(this.separator)[fname]());
+                    return this.withPostfix(postfix, msg.join(' ')[fname]());
                 else
@@ -193,8 +192,9 @@ return this.withPostfix(postfix, ...msg);
      * 
-     * @param {...String} msg   One or more strings to emit to stdout
-     * @returns {*}             The result from calling shelljs.echo
+     * @param {...String} msg   One or more strings to emit to stdout, separated
+     * by spaces
+     * @returns {*}
      */
     now(...msg)
     {
-        let message = msg.join(this.separator);
+        let message = msg.join(' ');
         let prefix  = fixPrefix(this.prefix);
@@ -280,3 +280,3 @@ let separator = prefix.length ? this.separator : '';
             fixPrefix(this.prefix, true) + 'DEBUG',
-            () => { return this.if(...msg); }
+            () => this.if(...msg)
         );
@@ -322,3 +322,3 @@ }
 
-        return fs.appendFileSync(this.logFilePath, msg.join(this.separator) + '\n', 'utf8');
+        return fs.appendFileSync(this.logFilePath, msg.join(' ') + '\n', 'utf8');
     }
@@ -337,3 +337,3 @@
             fixPrefix(this.prefix, true) + postfix,
-            () => { return this.now(...msg); }
+            () => this.now(...msg)
         );
@@ -340,0 +340,0 @@ }

lib/Echo/EmitStrategy/ConsoleLogEmitStrategy.js

 #!/usr/bin/nodejs
-/*jshint esnext:true */
 
@@ -4,0 +3,0 @@ /*

lib/Echo/EmitStrategy/EmitStrategy.js

 #!/usr/bin/nodejs
-/*jshint esnext:true */
 
@@ -4,0 +3,0 @@ /*

lib/Echo/EmitStrategy/ShellEchoEmitStrategy.js

 #!/usr/bin/nodejs
-/*jshint esnext:true */
 
@@ -4,0 +3,0 @@ /*

lib/Execute/Execute.js

 #!/usr/bin/nodejs
-/*jshint esnext:true */
 
@@ -13,2 +12,5 @@ /*
 
+const failmsg = 'FAILED!';
+const successmsg = 'OK';
+
 let noop = function(arg){ return arg; };
@@ -70,5 +72,2 @@ let colors = { red: noop, green: noop, yellow: noop, blue: noop };
     {
-        let failmsg = 'FAILED!';
-        let successmsg = 'OK';
-
         // Echo message
@@ -75,0 +74,0 @@ if(description)

lib/rejoinder.js

 #!/usr/bin/nodejs
-/*jshint esnext:true */
 
@@ -4,0 +3,0 @@ /*

lib/Utils/string.prototype.js

 #!/usr/bin/nodejs
-/*jshint esnext:true */
 
@@ -4,0 +3,0 @@ /*

package.json

 {
   "name": "rejoinder",
-  "version": "0.3.7",
+  "version": "0.3.8",
   "description": "A generator of smart, pretty, and organized output for Node/JS scripts in the terminal",
@@ -5,0 +5,0 @@ "repository": {

README.md

@@ -76,4 +76,6 @@ [![npm version](https://badge.fury.io/js/rejoinder.svg)](https://badge.fury.io/js/rejoinder)
 
+```
 npm install
 npm test
+```
 
@@ -80,0 +82,0 @@ ## Contributing

tests/Echo/Echo.js

 #!/usr/bin/nodejs
-/*jshint esnext:true */
 /*global describe, context, it, beforeEach */
@@ -231,12 +230,29 @@
                 echo.now('t', 'e', 's', 't');
-                expects(emission).to.equal('test');
+                expects(emission).to.equal('t e s t');
 
                 echo.separator = false;
                 echo.now('t', 'e', 's', 't');
-                expects(emission).to.equal('test');
+                expects(emission).to.equal('t e s t');
 
                 echo.separator = '-';
                 echo.now('t', 'e', 's', 't');
-                expects(emission).to.equal('t-e-s-t');
+                expects(emission).to.equal('t e s t');
             });
+
+            it('should emit properly with a set prefix when a different separator is set', function()
+            {
+                echo.prefix = '>>';
+
+                echo.separator = '';
+                echo.now('t', 'e', 's', 't');
+                expects(emission).to.equal('>>t e s t');
+
+                echo.separator = false;
+                echo.now('t', 'e', 's', 't');
+                expects(emission).to.equal('>>t e s t');
+
+                echo.separator = '-';
+                echo.now('t', 'e', 's', 't');
+                expects(emission).to.equal('>>-t e s t');
+            });
         });
@@ -250,3 +266,3 @@
                 echo.usingPredicate(function(s){ return `-${s}-`; }, 't', 'e', 's', 't');
-                expects(emission).to.equal('-t--e--s--t-');
+                expects(emission).to.equal('-t- -e- -s- -t-');
             });
@@ -253,0 +269,0 @@ });

tests/Execute/Execute.js

 #!/usr/bin/nodejs
-/*jshint esnext:true */
 /*global describe, context, beforeEach, it */
@@ -4,0 +3,0 @@

tests/rejoinder.js

 #!/usr/bin/nodejs
-/*jshint esnext:true */
 
@@ -4,0 +3,0 @@ /*

No alert changes

Improved metrics

Total package byte prevSize

48713

increased by0.59%

Lines of code

1024

increased by0.39%

Number of lines in readme file

91

increased by2.25%

No dependency changes