Merge pull request #2 from omaralalwi/add-tests

add tests

Author: omaralalwi

composer.json

@@ -29,7 +29,7 @@
             "role": "Developer"
         }
     ],
-    "version": "1.0.1",
+    "version": "1.0.2",
     "require": {
         "php": "^8.1",
         "omaralalwi/php-builders": "^1.0"

tests/PhpPyTest.php

@@ -1,10 +1,66 @@
 <?php
 
-use PHPUnit\Framework\TestCase;
-use Omaralalwi\PhpPy\PhpPy;
 use Omaralalwi\PhpPy\Managers\ConfigManager;
+use Omaralalwi\PhpPy\PhpPy;
+
+test('PhpPy runs the script correctly', function () {
+
+    $configManager = new ConfigManager([
+        'scripts_directory' => __DIR__ . '/../example-scripts',
+        'python_executable' => '/usr/bin/python3',
+        'max_timeout' => 30,
+    ]);
+
+    $result = PhpPy::build()
+        ->setConfig($configManager)
+        ->loadScript('sum_calculator.py')
+        ->withArguments([10, 20, 30])
+        ->run();
+
+    expect(json_decode($result))->toBe(60.0);
+});
+
+test('PhpPy throws exception when try to run script outside allowed path', function () {
+
+    $configManager = new ConfigManager([
+        'scripts_directory' => __DIR__ . '/../example-scripts',
+        'python_executable' => '/usr/bin/python3',
+        'max_timeout' => 30,
+    ]);
+
+    $this->expectException(\InvalidArgumentException::class);
+    $this->expectExceptionMessage('Script path does not exist.');
+
+    $result = PhpPy::build()
+        ->setConfig($configManager)
+        ->loadScript('/fake-path/sum_calculator.py')
+        ->withArguments([10, 20, 30])
+        ->run();
+
+    expect(json_decode($result))->toBe('Script path does not exist.');
+});
+
+test('withEnvironment throws exception for blacklisted environment variables', function () {
+
+    $configManager = new ConfigManager([
+        'scripts_directory' => __DIR__ . '/../example-scripts',
+        'python_executable' => '/usr/bin/python3',
+        'max_timeout' => 30,
+    ]);
+
+    $env = [
+        'LD_LIBRARY_PATH' => 'fake value',
+    ];
+
+    $this->expectException(\InvalidArgumentException::class);
+    $this->expectExceptionMessage("Environment variable 'LD_LIBRARY_PATH' is not allowed.");
+
+    $result = PhpPy::build()
+        ->setConfig($configManager)
+        ->loadScript('sum_calculator.py')
+        ->withArguments([10, 20, 30])
+        ->withEnvironment($env)
+        ->run();
 
-class PhpPyTest extends TestCase
-{
-    // will be ready in coming version
-}
+    expect(json_decode($result))->toBe("Environment variable 'LD_LIBRARY_PATH' is not allowed.");
+});

tests/TestCase.php

@@ -0,0 +1,20 @@
+#!/usr/bin/env python3
+import sys
+import json
+import random
+
+def main():
+    # Parse command-line arguments into numbers
+    try:
+        numbers = [float(arg) for arg in sys.argv[1:]]
+    except ValueError:
+        print(json.dumps({'error': 'All arguments must be numeric.'}))
+        sys.exit(1)
+
+    # Calculate the sum of the numbers
+    result = sum(numbers)
+    # Output the result as JSON
+    print(json.dumps(result))
+
+if __name__ == '__main__':
+    main()