remove console.log from prettyPrint

Author: jeremydaly

lib/prettyPrint.js

@@ -7,17 +7,27 @@
  */
 
 module.exports = routes => {
+
+  let out = ''
+
   // Calculate column widths
   let widths = routes.reduce((acc,row) => {
-    return [Math.max(acc[0],row[0].length),Math.max(acc[1],row[1].length)]
+    return [
+      Math.max(acc[0],Math.max(6,row[0].length)),
+      Math.max(acc[1],Math.max(5,row[1].length))
+    ]
   },[0,0])
 
-  console.log('╔══' + ''.padEnd(widths[0],'═') + '══╤══' + ''.padEnd(widths[1],'═') + '══╗')
-  console.log('║  ' + "\u001b[1m" + 'METHOD'.padEnd(widths[0]) + "\u001b[0m" + '  │  ' + "\u001b[1m" + 'ROUTE'.padEnd(widths[1]) + "\u001b[0m" + '  ║')
-  console.log('╟──' + ''.padEnd(widths[0],'─') + '──┼──' + ''.padEnd(widths[1],'─') + '──╢')
+  out += '╔══' + ''.padEnd(widths[0],'═') + '══╤══' + ''.padEnd(widths[1],'═') + '══╗\n'
+  out += '║  ' + '\u001b[1m' + 'METHOD'.padEnd(widths[0]) + '\u001b[0m' + '  │  ' + '\u001b[1m' + 'ROUTE'.padEnd(widths[1]) + '\u001b[0m' + '  ║\n'
+  out += '╟──' + ''.padEnd(widths[0],'─') + '──┼──' + ''.padEnd(widths[1],'─') + '──╢\n'
   routes.forEach((route,i) => {
-    console.log('║  ' + route[0].padEnd(widths[0]) + '  │  ' + route[1].padEnd(widths[1]) + '  ║')
-    if (i < routes.length-1) { console.log('╟──' + ''.padEnd(widths[0],'─') + '──┼──' + ''.padEnd(widths[1],'─') + '──╢') }
+    out += '║  ' + route[0].padEnd(widths[0]) + '  │  ' + route[1].padEnd(widths[1]) + '  ║\n'
+    if (i < routes.length-1) {
+      out += '╟──' + ''.padEnd(widths[0],'─') + '──┼──' + ''.padEnd(widths[1],'─') + '──╢\n'
+    } // end if
   })
-  console.log('╚══' + ''.padEnd(widths[0],'═') + '══╧══' + ''.padEnd(widths[1],'═') + '══╝')
+  out += '╚══' + ''.padEnd(widths[0],'═') + '══╧══' + ''.padEnd(widths[1],'═') + '══╝'
+
+  return out
 }

test/prettyPrint.js

@@ -9,8 +9,12 @@ const prettyPrint = require('../lib/prettyPrint')
 /******************************************************************************/
 
 describe('PrettyPrint Tests:', function() {
-  it('Execute', function() {
-    expect(prettyPrint([['GET','/'],['POST','/'],['DELETE','/']])).to.equal(undefined)
+  it('Minimum header widths', function() {
+    expect(prettyPrint([['GET','/'],['POST','/'],['DELETE','/']])).to.equal('╔══════════╤═════════╗\n║  \u001b[1mMETHOD\u001b[0m  │  \u001b[1mROUTE\u001b[0m  ║\n╟──────────┼─────────╢\n║  GET     │  /      ║\n╟──────────┼─────────╢\n║  POST    │  /      ║\n╟──────────┼─────────╢\n║  DELETE  │  /      ║\n╚══════════╧═════════╝')
+  }) // end it
+
+  it('Adjusted header widths', function() {
+    expect(prettyPrint([['GET','/'],['POST','/testing'],['DELETE','/long-url-path-name']])).to.equal('╔══════════╤═══════════════════════╗\n║  \u001b[1mMETHOD\u001b[0m  │  \u001b[1mROUTE              \u001b[0m  ║\n╟──────────┼───────────────────────╢\n║  GET     │  /                    ║\n╟──────────┼───────────────────────╢\n║  POST    │  /testing             ║\n╟──────────┼───────────────────────╢\n║  DELETE  │  /long-url-path-name  ║\n╚══════════╧═══════════════════════╝')
   }) // end it
 
 }) // end UTILITY tests