Revert "Return empty .json() object on 204. Fix #165. (#166)" (#201)

This reverts commit 95b5893. Fixes #165.
node-fetch · Nov 26, 2016 · d3071fa · d3071fa
1 parent 7f0e502
commit d3071fa
Show file tree

Hide file tree

Showing 2 changed files with 0 additions and 15 deletions.
diff --git a/src/body.js b/src/body.js
@@ -51,11 +51,6 @@ export default class Body {
 	 * @return  Promise
 	 */
 	json() {
-		// for 204 No Content response, buffer will be empty, parsing it will throw error
-		if (this.status === 204) {
-			return Body.Promise.resolve({});
-		}
-
 		return this[CONSUME_BODY]().then(buffer => JSON.parse(buffer.toString()));
 	}
 

diff --git a/test/test.js b/test/test.js
@@ -471,16 +471,6 @@ describe(`node-fetch with FOLLOW_SPEC = ${defaultFollowSpec}`, () => {
 		});
 	});
 
-	it('should return empty object on no-content response', function() {
-		url = `${base}no-content`;
-		return fetch(url).then(res => {
-			return res.json().then(result => {
-				expect(result).to.be.an('object');
-				expect(result).to.be.empty;
-			});
-		});
-	});
-
 	it('should handle no content response with gzip encoding', function() {
 		url = `${base}no-content/gzip`;
 		return fetch(url).then(res => {