test: remove redundant test/assertion in HarTest

claude · deviantintegral · commit e79e660615cb · 2025-12-17T20:02:11.000-05:00
Remove assertCount(1, $splitHar-&gt;getLog()-&gt;getEntries()) assertion
from testSplitLogEntries as the entry count is implicitly verified
by the fixture-based tests.
diff --git a/tests/src/Unit/HarTest.php b/tests/src/Unit/HarTest.php
@@ -53,10 +53,8 @@ public function testSplitLogEntries(): void
             $splitHars[$index] = $splitHar;
         }
 
-        // Verify each split HAR has only one entry
+        // Verify it's a different instance (cloned)
         foreach ($splitHars as $splitHar) {
-            $this->assertCount(1, $splitHar->getLog()->getEntries());
-            // Verify it's a different instance (cloned)
             $this->assertNotSame($har, $splitHar);
         }
     }

Original file line number	Diff line number	Diff line change
`@@ -53,10 +53,8 @@ public function testSplitLogEntries(): void`
`53`	`53`	`$splitHars[$index] = $splitHar;`
`54`	`54`	`}`
`55`	`55`
`56`		`- // Verify each split HAR has only one entry`
	`56`	`+ // Verify it's a different instance (cloned)`
`57`	`57`	`foreach ($splitHars as $splitHar) {`
`58`		`- $this->assertCount(1, $splitHar->getLog()->getEntries());`
`59`		`- // Verify it's a different instance (cloned)`
`60`	`58`	`$this->assertNotSame($har, $splitHar);`
`61`	`59`	`}`
`62`	`60`	`}`