Skip to content

Commit a2d4f67

Browse files
claudedeviantintegral
authored andcommitted
test: remove redundant test/assertion in ResponseTest
Remove testSetHeadersCalculatesCorrectSize as it is redundant - the same code paths are already tested by testSetHeadersWithMultipleHeaders and testSetHeadersWithSingleCharacterValues.
1 parent e79a743 commit a2d4f67

File tree

1 file changed

+0
-12
lines changed

1 file changed

+0
-12
lines changed

tests/src/Unit/ResponseTest.php

Lines changed: 0 additions & 12 deletions
Original file line numberDiff line numberDiff line change
@@ -61,18 +61,6 @@ public function testSerialize(): void
6161
$this->assertEquals($response->getComment(), $deserialized->getComment());
6262
}
6363

64-
public function testSetHeadersCalculatesCorrectSize(): void
65-
{
66-
$response = new \Deviantintegral\Har\Response();
67-
68-
// Test with single header: "Content-Type: application/json"
69-
// Size calculation: strlen("Content-Type") + 2 + strlen("application/json") + 2 = 12 + 2 + 16 + 2 = 32
70-
// Plus final 2 for double CRLF: 32 + 2 = 34
71-
$headers = [(new Header())->setName('Content-Type')->setValue('application/json')];
72-
$response->setHeaders($headers);
73-
$this->assertSame(34, $response->getHeadersSize());
74-
}
75-
7664
public function testSetHeadersWithMultipleHeaders(): void
7765
{
7866
$response = new \Deviantintegral\Har\Response();

0 commit comments

Comments
 (0)