Merge pull request #38 from sindresorhus/support-humanized-urls

support humanized URLs

bin/cli.js

@@ -30,9 +30,7 @@ if (process.argv.indexOf('-v') !== -1 || process.argv.indexOf('--version') !== -
   return;
 }
 
-if (query.indexOf('http') !== -1) {
-  opts.url = argv._[0];
-}
+opts.url = argv._[0];
 
 if(!opts.url){
   printHelp();

index.js

@@ -8,13 +8,20 @@
 
 'use strict';
 var pagespeed = require('gpagespeed');
+var prependHttp = require('prepend-http');
 var output = require('./output').init();
 
 module.exports = function (opts, cb) {
   opts = opts || {};
   cb = cb || function () {};
+
+  if (!opts.url) {
+    throw new Error('URL required');
+  }
+
   opts.strategy = opts.strategy || 'desktop';
   opts.nokey = opts.key === undefined;
+  opts.url = prependHttp(opts.url);
 
   pagespeed(opts, function (err, data) {
     if (err) {

package.json

@@ -38,6 +38,7 @@
     "chalk": "^0.5.1",
     "gpagespeed": "0.0.8",
     "minimist": "^1.1.0",
+    "prepend-http": "^1.0.0",
     "pretty-bytes": "^1.0.1"
   },
   "devDependencies": {

test/test.js

@@ -34,10 +34,9 @@ describe('PSI formatting', function () {
 });
 
 describe('CLI Error handling', function() {
-  it('should throw if no valid URL is provided', function (done) {
-    insights({}, function(err) {
-      assert.equal(err.message, 'Missing required param: url');
-      done();
+  it('should throw if no valid URL is provided', function () {
+    assert.throws(function () {
+      insights({}, function() {});
     });
   });
 });