fix: set protocol correctly (#1027)

fixes #976

lib/request_overrider.js

@@ -238,6 +238,9 @@ function RequestOverrider(req, options, interceptors, remove, cb) {
     /// like to change request.path in mid-flight.
     options.path = req.path;
 
+    // fixes #976
+    options.protocol = options.proto + ':';
+
     interceptors.forEach(function(interceptor) {
       //  For correct matching we need to have correct request headers - if these were specified.
       setRequestHeaders(req, options, interceptor);

tests/test_https_allowunmocked.js

@@ -2,10 +2,11 @@
 
 var test          = require('tap').test;
 var mikealRequest = require('request');
+var nock          = require('../');
+
+nock.enableNetConnect();
 
 test('allowUnmocked for https', {skip: process.env.AIRPLANE}, function(t) {
-  var nock = require('../');
-  nock.enableNetConnect();
   nock('https://www.google.com/', {allowUnmocked: true})
   .get('/pathneverhit')
   .reply(200, {foo: 'bar'});
@@ -22,3 +23,23 @@ test('allowUnmocked for https', {skip: process.env.AIRPLANE}, function(t) {
     t.end();
   });
 });
+
+test('allowUnmocked for https with query test miss', {skip: process.env.AIRPLANE}, function(t) {
+  nock.cleanAll();
+  nock('https://www.google.com', {allowUnmocked: true})
+    .get('/search')
+    .query(function() {return false;})
+    .reply(500);
+
+  var options = {
+    method: 'GET',
+    uri: 'https://www.google.com/search'
+  };
+
+  mikealRequest(options, function(err, resp, body) {
+    t.notOk(err, 'should be no error');
+    t.true(typeof body !== 'undefined', 'body should not be undefined');
+    t.true(body.length !== 0, 'body should not be empty');
+    t.end();
+  });
+});