build: supertest@3.3.0

Author: dougwilson

.travis.yml

@@ -32,6 +32,15 @@ before_install:
     elif [[ "$(cut -d. -f1 <<< "$TRAVIS_NODE_VERSION")" -lt 6 ]]; then
       npm install --save-dev mocha@3.5.3
     fi
+  - |
+    # supertest for http calls
+    # - use 1.1.0 for Node.js < 0.10
+    # - use 2.0.0 for Node.js < 4
+    if [[ "$(cut -d. -f1 <<< "$TRAVIS_NODE_VERSION")" -eq 0 && "$(cut -d. -f2 <<< "$TRAVIS_NODE_VERSION")" -lt 10 ]]; then
+      npm install --save-dev supertest@1.1.0
+    elif [[ "$(cut -d. -f1 <<< "$TRAVIS_NODE_VERSION")" -lt 4 ]]; then
+      npm install --save-dev supertest@2.0.0
+    fi
   - |
     # istanbul for coverage
     # - remove for Node.js < 0.10

appveyor.yml

@@ -46,6 +46,15 @@ install:
       } elseif ([int]$env:nodejs_version.split(".")[0] -lt 6) {
         npm install --silent --save-dev mocha@3.5.3
       }
+  - ps: |
+      # supertest for http calls
+      # - use 1.1.0 for Node.js < 0.10
+      # - use 2.0.0 for Node.js < 4
+      if ([int]$env:nodejs_version.split(".")[0] -eq 0 -and [int]$env:nodejs_version.split(".")[1] -lt 10) {
+        npm install --silent --save-dev supertest@1.1.0
+      } elseif ([int]$env:nodejs_version.split(".")[0] -lt 4) {
+        npm install --silent --save-dev supertest@2.0.0
+      }
   # Update Node.js modules
   - ps: |
       # Prune & rebuild node_modules

package.json

@@ -41,7 +41,7 @@
     "eslint-plugin-standard": "3.1.0",
     "istanbul": "0.4.5",
     "mocha": "5.2.0",
-    "supertest": "1.1.0"
+    "supertest": "3.3.0"
   },
   "files": [
     "HISTORY.md",

test/send.js

@@ -85,8 +85,10 @@ describe('send(file).pipe(res)', function () {
   it('should support HEAD', function (done) {
     request(app)
       .head('/name.txt')
+      .expect(200)
       .expect('Content-Length', '4')
-      .expect(200, '', done)
+      .expect(shouldNotHaveBody())
+      .end(done)
   })
 
   it('should add an ETag header field', function (done) {
@@ -1464,6 +1466,12 @@ function createServer (opts, fn) {
   })
 }
 
+function shouldNotHaveBody () {
+  return function (res) {
+    assert.ok(res.text === '' || res.text === undefined)
+  }
+}
+
 function shouldNotHaveHeader (header) {
   return function (res) {
     assert.ok(!(header.toLowerCase() in res.headers), 'should not have header ' + header)