Merge pull request #483 from squarejaw/test-coverage

Increase test coverage

test/request/ip.js

@@ -18,5 +18,12 @@ describe('req.ip', function(){
       req.socket.remoteAddress = '127.0.0.2';
       req.ip.should.equal('127.0.0.2');
     })
+    describe('with req.socket.remoteAddress not present', function() {
+      it('should return an empty string', function(){
+        var req = request();
+        req.socket.remoteAddress = null;
+        req.ip.should.equal('');
+      })
+    })
   })
 })

test/request/querystring.js

@@ -1,6 +1,20 @@
 
 var context = require('../context');
 
+describe('ctx.querystring', function(){
+  it('should return the querystring', function () {
+    var ctx = context({ url: '/store/shoes?page=2&color=blue' });
+    ctx.querystring.should.equal('page=2&color=blue');
+  })
+  describe('when ctx.req not present', function() {
+    it('should return an empty string', function (){
+      var ctx = context();
+      ctx.request.req = null;
+      ctx.querystring.should.equal('');
+    })
+  })
+})
+
 describe('ctx.querystring=', function(){
   it('should replace the querystring', function(){
     var ctx = context({ url: '/store/shoes' });