test: add edge case tests for LessThan mutations (#158)

Add tests for boundary conditions identified in mutation
testing:

1. HarFileRepository:
   - Test filtering of files with exactly 4 characters
   - Verify files < 4 chars are filtered out
   - Verify .har files >= 4 chars are included

2. Timings:
   - Test setConnect allows connect equal to SSL time
   - Test setConnect throws exception when connect < SSL
   - Test setSend/setWait/setReceive allow zero values
   - Test setSend/setWait/setReceive reject negative values

These tests kill the LessThan mutations identified in the report by:
- Testing the boundary value where < becomes <=
- Ensuring edge cases are properly handled

Co-authored-by: Claude <noreply@anthropic.com>

Author: deviantintegral

tests/src/Unit/Repository/HarFileRepositoryTest.php

@@ -105,4 +105,45 @@ public function testLoadJsonHandlesEmptyFile(): void
             throw $e;
         }
     }
+
+    public function testGetIdsFiltersFilesWithLessThanFourCharacters(): void
+    {
+        $tempDir = sys_get_temp_dir().'/har_test_'.uniqid();
+        mkdir($tempDir);
+
+        // Create files with various lengths
+        $files = [
+            'a' => '',           // 1 character - should be filtered
+            'ab' => '',          // 2 characters - should be filtered
+            'abc' => '',         // 3 characters - should be filtered
+            'test' => '',        // 4 characters but no .har extension - should be filtered
+            'a.bc' => '',        // 4 characters but no .har extension - should be filtered
+            'a.har' => '{}',     // 5 characters with .har extension - should be included
+            'ab.har' => '{}',    // 6 characters with .har extension - should be included
+        ];
+
+        foreach ($files as $filename => $content) {
+            file_put_contents($tempDir.'/'.$filename, $content);
+        }
+
+        $repository = new HarFileRepository($tempDir);
+        $ids = $repository->getIds();
+
+        // Only files with .har extension and length >= 4 should be included
+        $this->assertContains('a.har', $ids);
+        $this->assertContains('ab.har', $ids);
+
+        // All other files should be filtered out
+        $this->assertNotContains('a', $ids);
+        $this->assertNotContains('ab', $ids);
+        $this->assertNotContains('abc', $ids);
+        $this->assertNotContains('test', $ids);
+        $this->assertNotContains('a.bc', $ids);
+
+        // Clean up
+        foreach (array_keys($files) as $filename) {
+            unlink($tempDir.'/'.$filename);
+        }
+        rmdir($tempDir);
+    }
 }

tests/src/Unit/TimingsTest.php

@@ -99,4 +99,94 @@ public function testHasSsl(): void
         $this->assertTrue($timings->hasSsl());
         $this->assertEquals(25.7, $timings->getSsl());
     }
+
+    public function testSetConnectAllowsConnectEqualToSSL(): void
+    {
+        $timings = new Timings();
+        $timings->setSsl(100.0);
+
+        // Connect time equal to SSL time should be allowed
+        // The check is (connect < ssl), so connect >= ssl is valid
+        $timings->setConnect(100.0);
+        $this->assertEquals(100.0, $timings->getConnect());
+    }
+
+    public function testSetConnectThrowsExceptionWhenConnectLessThanSSL(): void
+    {
+        $timings = new Timings();
+        $timings->setSsl(100.0);
+
+        $this->expectException(\LogicException::class);
+        $this->expectExceptionMessage('Connect time must include SSL time');
+
+        // Connect time less than SSL time should throw exception
+        $timings->setConnect(50.0);
+    }
+
+    public function testSetConnectAllowsConnectGreaterThanSSL(): void
+    {
+        $timings = new Timings();
+        $timings->setSsl(100.0);
+
+        // Connect time greater than SSL time should be allowed
+        $timings->setConnect(150.0);
+        $this->assertEquals(150.0, $timings->getConnect());
+    }
+
+    public function testSetSendAllowsZero(): void
+    {
+        $timings = new Timings();
+
+        // Send time of 0 should be allowed (not negative)
+        $timings->setSend(0.0);
+        $this->assertEquals(0.0, $timings->getSend());
+    }
+
+    public function testSetSendThrowsExceptionForNegativeValues(): void
+    {
+        $timings = new Timings();
+
+        $this->expectException(\LogicException::class);
+        $this->expectExceptionMessage('Send must not be negative');
+
+        $timings->setSend(-1.0);
+    }
+
+    public function testSetWaitAllowsZero(): void
+    {
+        $timings = new Timings();
+
+        // Wait time of 0 should be allowed (not negative)
+        $timings->setWait(0.0);
+        $this->assertEquals(0.0, $timings->getWait());
+    }
+
+    public function testSetWaitThrowsExceptionForNegativeValues(): void
+    {
+        $timings = new Timings();
+
+        $this->expectException(\LogicException::class);
+        $this->expectExceptionMessage('Wait must not be negative');
+
+        $timings->setWait(-1.0);
+    }
+
+    public function testSetReceiveAllowsZero(): void
+    {
+        $timings = new Timings();
+
+        // Receive time of 0 should be allowed (not negative)
+        $timings->setReceive(0.0);
+        $this->assertEquals(0.0, $timings->getReceive());
+    }
+
+    public function testSetReceiveThrowsExceptionForNegativeValues(): void
+    {
+        $timings = new Timings();
+
+        $this->expectException(\LogicException::class);
+        $this->expectExceptionMessage('Receive must not be negative');
+
+        $timings->setReceive(-1.0);
+    }
 }