ci

Author: elwafa

src/Platform.php

@@ -214,4 +214,4 @@ public static function getBinaryPath(): string
 
         return $binaryPath;
     }
-}
+}

src/ProcessManager.php

@@ -170,4 +170,4 @@ protected function closeProcess($process): int
 
         return proc_close($process);
     }
-}
+}

tests/VoltTestTest.php

@@ -21,22 +21,55 @@ protected function tearDown(): void
         ErrorHandler::unregister();
         parent::tearDown();
     }
+
     public function testVoltTest()
     {
         $voltTest = new VoltTest("test");
         $voltTest->setVirtualUsers(10);
+
+        // Create first scenario
         $scenario1 = $voltTest->scenario("Test Scenario")->setWeight(10);
-        $scenario1->step("Step 1")->get('https://www.google.com');
+        $scenario1->step("Step 1")
+            ->get('https://www.google.com')
+            ->validateStatus('success', 200);
+
+        // Create second scenario
         $scenario2 = $voltTest->scenario("Test Scenario 2")->setWeight(90);
-        $scenario2->step("Step 1")->get('https://www.google.com');
+        $scenario2->step("Step 1")
+            ->get('https://www.google.com')
+            ->validateStatus('success', 200);
+
+        // Run test and get results
         $result = $voltTest->run(true);
-        var_dump($result->getDuration());
-        $this->assertNotEquals('0', $result->getDuration(), "Duration should not be 0");
-        $this->assertNotNull($result->getAvgResponseTime(), "Average response time should not be null");
-
-        $this->assertIsFloat($result->getSuccessRate(), "Success rate should be a float");
-        $this->assertIsInt($result->getTotalRequests(), "Total requests should be an integer");
-        $this->assertIsInt($result->getSuccessRequests(), "Success requests should be an integer");
-        $this->assertIsInt($result->getFailedRequests(), "Failed requests should be an integer");
+
+        // Basic assertions about test execution
+        $this->assertNotEmpty($result->getRawOutput(), "Raw output should not be empty");
+
+        // Get duration and remove any 's' suffix if present
+        $duration = str_replace('s', '', $result->getDuration());
+        $this->assertNotEquals(0, (float)$duration, "Duration should not be 0");
+
+        // Verify response time metrics
+        $avgResponseTime = $result->getAvgResponseTime();
+        if ($avgResponseTime !== null) {
+            $this->assertStringContainsString('ms', $avgResponseTime, "Average response time should contain 'ms'");
+        }
+
+        // Test rate metrics
+        $this->assertGreaterThan(0, $result->getSuccessRate(), "Success rate should be greater than 0");
+        $this->assertGreaterThanOrEqual(0, $result->getRequestsPerSecond(), "Requests per second should be non-negative");
+
+        // Request counts
+        $this->assertGreaterThan(0, $result->getTotalRequests(), "Total requests should be greater than 0");
+        $this->assertGreaterThanOrEqual(0, $result->getSuccessRequests(), "Success requests should be non-negative");
+        $this->assertGreaterThanOrEqual(0, $result->getFailedRequests(), "Failed requests should be non-negative");
+
+        // Total requests should equal success + failed requests
+        $this->assertEquals(
+            $result->getTotalRequests(),
+            $result->getSuccessRequests() + $result->getFailedRequests(),
+            "Total requests should equal sum of success and failed requests"
+        );
+
     }
 }