fix: only throw error responses (nodejs#1076)

Fixes: nodejs#1072

Author: ronag

lib/fetch/index.js

@@ -171,7 +171,7 @@ async function fetch (...args) {
 
     // 3. If response is a network error, then reject p with a TypeError
     // and terminate these substeps.
-    if (response.status === 0) {
+    if (response.type === 'error') {
       p.reject(
         Object.assign(new TypeError('fetch failed'), { cause: response.error })
       )

test/node-fetch/main.js

@@ -409,14 +409,14 @@ describe('node-fetch', () => {
     })
   })
 
-  xit('should treat broken redirect as ordinary response (manual)', () => {
+  it('should treat broken redirect as ordinary response (manual)', () => {
     const url = `${base}redirect/no-location`
     const options = {
       redirect: 'manual'
     }
     return fetch(url, options).then(res => {
       expect(res.url).to.equal(url)
-      expect(res.status).to.equal(301)
+      expect(res.status).to.equal(0)
       expect(res.headers.get('location')).to.be.null
     })
   })