squash: moving validate-metadata

* move validate-metadata logic to utils for easier testing

Author: lholmquist

bin/cmd.js

@@ -5,7 +5,6 @@
 const fs = require('fs')
 const nopt = require('nopt')
 const path = require('path')
-const pretty = require('../lib/format-pretty')
 const formatTap = require('../lib/format-tap')
 const Validator = require('../lib')
 const Tap = require('../lib/tap')
@@ -110,22 +109,6 @@ if (parsed.tap) {
 
 } else {
   // no --flags used,  defaults to --validate-metadata
-  v.on('commit', (c) => {
-    pretty(c.commit, c.messages, v)
-    run()
-  })
-
-  function run() {
-    if (!args.length) {
-      process.exitCode = v.errors
-      return
-    }
-    const sha = args.shift()
-    utils.load(sha, (err, data) => {
-      if (err) throw err
-      v.lint(data)
-    })
-  }
-
-  run()
+  utils.validateMetadata(v, args)
+  return
 }

lib/utils.js

@@ -4,6 +4,7 @@ const exec = require('child_process').exec
 const http = require('http')
 const https = require('https')
 const url = require('url')
+const pretty = require('../lib/format-pretty')
 const chalk = require('chalk')
 const CHECK = chalk.green('✔')
 const X = chalk.red('✖')
@@ -102,3 +103,24 @@ exports.loadPatch = function loadPatch(uri, cb) {
     })
   }).on('error', cb)
 }
+
+exports.validateMetadata = function(validator, args) {
+  validator.on('commit', (c) => {
+    pretty(c.commit, c.messages, validator)
+    run()
+  })
+
+  function run() {
+    if (!args.length) {
+      process.exitCode = validator.errors
+      return
+    }
+    const sha = args.shift()
+    exports.load(sha, (err, data) => {
+      if (err) throw err
+      validator.lint(data)
+    })
+  }
+
+  run()
+}