Merge pull request #101 from pdscopes/master

Fixing issue #100 withHeader does not remove old header value

Author: weierophinney

src/MessageTrait.php

@@ -204,6 +204,9 @@ public function withHeader($header, $value)
         $normalized = strtolower($header);
 
         $new = clone $this;
+        if ($new->hasHeader($header)) {
+            unset($new->headers[$new->headerNames[$normalized]]);
+        }
         $new->headerNames[$normalized] = $header;
         $new->headers[$header]         = $value;
 

test/MessageTraitTest.php

@@ -175,6 +175,14 @@ public function testWithHeaderRaisesExceptionForInvalidValueType($value)
         $message = $this->message->withHeader('X-Foo', $value);
     }
 
+    public function testWithHeaderReplacesDifferentCapitalization()
+    {
+        $this->message = $this->message->withHeader('X-Foo', ['foo']);
+        $new = $this->message->withHeader('X-foo', ['bar']);
+        $this->assertEquals(['bar'], $new->getHeader('x-foo'));
+        $this->assertEquals(['X-foo' => ['bar']], $new->getHeaders());
+    }
+
     /**
      * @dataProvider invalidGeneralHeaderValues
      */