Fixed an inconsistency in behavior between Node and browsers

Author: JamesMessinger

lib/read.js

@@ -221,7 +221,7 @@ function download(protocol, u, options) {
         if (res.statusCode >= 400) {
           reject(ono('GET %s \nHTTP ERROR %d \n%s', u.href, res.statusCode, body));
         }
-        else if ((res.statusCode === 204 || !body || !body.length) && !options.allow.empty) {
+        else if (res.statusCode === 204 && !options.allow.empty) {
           reject(ono('GET %s \nHTTP 204: No Content', u.href));
         }
         else {

tests/specs/blank/blank.spec.js

@@ -47,16 +47,10 @@ describe('Blank files', function() {
       .parse(path.rel('specs/blank/blank.yaml'), {allow: {empty: false}})
       .then(helper.shouldNotGetCalled)
       .catch(function(err) {
-        if (userAgent.isNode) {
-          expect(err).to.be.an.instanceOf(SyntaxError);
-          expect(err.message).to.contain('Error parsing "');
-          expect(err.message).to.contain('blank/blank.yaml"');
-          expect(err.message).to.contain('Parsed value is empty');
-        }
-        else {
-          expect(err).to.be.an.instanceOf(Error);
-          expect(err.message).to.contain('HTTP 204: No Content');
-        }
+        expect(err).to.be.an.instanceOf(SyntaxError);
+        expect(err.message).to.contain('Error parsing "');
+        expect(err.message).to.contain('blank/blank.yaml"');
+        expect(err.message).to.contain('Parsed value is empty');
         done();
       });
   });