From: Ken Perkins <ken.perkins@rackspace.com>
Date: Thu, 26 Feb 2015 18:36:40 +0000 (-0800)
Subject: test: adds a test for undefined value in setHeader
X-Git-Tag: v1.5.0~28
X-Git-Url: http://review.tizen.org/git/?a=commitdiff_plain;h=b72fa030572175060a747bc7b95784bf5d1e163b;p=platform%2Fupstream%2Fnodejs.git

test: adds a test for undefined value in setHeader

As a result of 979d0ca8 there is a new check for undefined values on
OutgoingMessage.setHeader. This commit introduces a test for this case.

PR-URL: https://github.com/iojs/io.js/pull/970
Reviewed-By: Rod Vagg <rod@vagg.org>
Reviewed-By: Brendan Ashworth <brendan.ashworth@me.com>
---

diff --git a/test/parallel/test-http-write-head.js b/test/parallel/test-http-write-head.js
index b0cab23..acd2a60 100644
--- a/test/parallel/test-http-write-head.js
+++ b/test/parallel/test-http-write-head.js
@@ -18,6 +18,17 @@ var s = http.createServer(function(req, res) {
   }
   assert.ok(threw, 'Non-string names should throw');
 
+  // undefined value should throw, via 979d0ca8
+  threw = false;
+  try {
+    res.setHeader('foo', undefined);
+  } catch (e) {
+    assert.ok(e instanceof Error);
+    assert.equal(e.message, '`value` required in setHeader("foo", value).');
+    threw = true;
+  }
+  assert.ok(threw, 'Undefined value should throw');
+
   res.writeHead(200, { Test: '2' });
   res.end();
 });