From 2d5bab8c0c3e83e4e19636078eec22241f8f2e14 Mon Sep 17 00:00:00 2001
From: James Talmage <james@talmage.io>
Date: Wed, 1 Jul 2015 22:56:47 -0400
Subject: [PATCH] add test with query params in the redirect

---
 test/test-with-server.js | 19 ++++++++++++++++++-
 1 file changed, 18 insertions(+), 1 deletion(-)

diff --git a/test/test-with-server.js b/test/test-with-server.js
index d81c0f3..a33866b 100644
--- a/test/test-with-server.js
+++ b/test/test-with-server.js
@@ -184,6 +184,23 @@ describe('follow-redirects', function() {
       .nodeify(done);
   });
 
+  it('redirect to query params', function(done) {
+    app.get('/a', redirectsTo('/b?greeting=hello'));
+    app.get('/b', function(req, res) {
+      res.json({greeting:req.query.greeting});
+    });
+
+    server.start(app)
+      .then(asPromise(function(resolve, reject){
+        http.get('http://localhost:3600/a', resolve).on('error', reject)
+      }))
+      .then(concatJson)
+      .then(function(json) {
+        assert.deepEqual(json, {greeting:'hello'})
+      })
+      .nodeify(done);
+  });
+
   function redirectsTo(opt_status, path) {
     var args = Array.prototype.slice.call(arguments);
     return function(req, res) {
@@ -203,7 +220,7 @@ describe('follow-redirects', function() {
         try {
           resolve(JSON.parse(string));
         } catch (e) {
-          reject(e);
+          reject(new Error('error parsing ' + JSON.stringify(string) + '\n caused by: ' + e.message));
         }
       })).on('error', reject);
     });