Merge pull request #1227 from focusaurus/fix-latest-mime-tests

Fix xml tests that broke with mime update
ladjs · May 24, 2017 · d0ab622 · d0ab622
2 parents ea35c57 + 0173a9d
commit d0ab622
Show file tree

Hide file tree

Showing 4 changed files with 4 additions and 4 deletions.
diff --git a/lib/client.js b/lib/client.js
@@ -169,7 +169,7 @@ request.parseString = parseString;
 request.types = {
   html: 'text/html',
   json: 'application/json',
-  xml: 'application/xml',
+  xml: 'text/xml',
   urlencoded: 'application/x-www-form-urlencoded',
   'form': 'application/x-www-form-urlencoded',
   'form-data': 'application/x-www-form-urlencoded'

diff --git a/package.json b/package.json
@@ -31,7 +31,7 @@
     "form-data": "^2.1.1",
     "formidable": "^1.1.1",
     "methods": "^1.1.1",
-    "mime": "^1.3.4",
+    "mime": "^1.3.6",
     "qs": "^6.4.0",
     "readable-stream": "^2.0.5"
   },

diff --git a/test/basic.js b/test/basic.js
@@ -360,7 +360,7 @@ describe('request', function(){
       .accept('xml')
       .end(function(err, res){
         try {
-        res.header['accept'].should.equal('application/xml');
+        res.header['accept'].should.equal('text/xml');
         done();
         } catch(e) { done(e); }
       });

diff --git a/test/request.js b/test/request.js
@@ -295,7 +295,7 @@ it('request .accept() with xml', function(next){
   .accept('xml')
   .end(function(err, res){
   try {
-    assert.equal('application/xml', res.text, res.text);
+    assert.equal('text/xml', res.text, res.text);
     next();
     } catch(e) { next(e); }
   });