Merge pull request #317 from jolicode/more-tests

more tests

bin/generate-tests.php

@@ -5,11 +5,12 @@
 
 use Castor\Console\ApplicationFactory;
 use Castor\PlatformUtil;
+use Castor\Tests\Helper\OutputCleaner;
 use Castor\Tests\Helper\WebServerHelper;
-use Castor\Tests\OutputCleaner;
 use Symfony\Component\Console\Input\ArrayInput;
 use Symfony\Component\Console\Output\BufferedOutput;
 use Symfony\Component\Filesystem\Filesystem;
+use Symfony\Component\Finder\Finder;
 use Symfony\Component\Process\Process;
 
 use function Symfony\Component\String\u;
@@ -60,7 +61,6 @@
     'castor:watcher:linux',
     'castor:watcher:windows',
     // Customized tests
-    'cd:directory',
     'fingerprint:task-with-a-fingerprint-and-force',
     'fingerprint:task-with-a-fingerprint',
     'fingerprint:task-with-complete-fingerprint-check',
@@ -107,6 +107,15 @@
     add_test($args, $class);
 }
 
+$dirs = (new Finder())
+    ->in($basePath = __DIR__ . '/../tests/Examples/fixtures/broken')
+    ->depth(1)
+;
+foreach ($dirs as $dir) {
+    $class = u($dir->getRelativePath())->camel()->title()->append('Test')->toString();
+    add_test([], $class, '{{ base }}/tests/Examples/fixtures/broken/' . $dir->getRelativePath());
+}
+
 add_test(['parallel:sleep', '--sleep5', '0', '--sleep7', '0', '--sleep10', '0'], 'ParallelSleepTest');
 add_test(['context:context', '--context', 'run'], 'ContextContextRunTest');
 add_test(['context:context', '--context', 'my_default', '-v'], 'ContextContextMyDefaultTest');
@@ -129,7 +138,7 @@ function add_test(array $args, string $class, ?string $cwd = null)
 
     $process = new Process(
         [\PHP_BINARY,  __DIR__ . '/castor', '--no-ansi', ...$args],
-        cwd: $cwd ?: __DIR__ . '/../',
+        cwd: $cwd ? str_replace('{{ base }}', __DIR__ . '/..', $cwd) : __DIR__ . '/..',
         env: [
             'COLUMNS' => 120,
             'ENDPOINT' => $_SERVER['ENDPOINT'],

tests/Examples/CdDirectoryTest.php → tests/Examples/Generated/CdDirectoryTest.php

@@ -1,21 +1,20 @@
 <?php
 
-namespace Castor\Tests\Examples;
+namespace Castor\Tests\Examples\Generated;
 
-use Castor\Tests\OutputCleaner;
 use Castor\Tests\TaskTestCase;
 
 class CdDirectoryTest extends TaskTestCase
 {
     // cd:directory
-    public function testCdDirectory(): void
+    public function test(): void
     {
         $process = $this->runTask(['cd:directory']);
+
         $this->assertSame(0, $process->getExitCode());
-        $output = OutputCleaner::cleanOutput($process->getOutput());
-        $this->assertStringEqualsFile(__FILE__ . '.output.txt', $output);
+        $this->assertStringEqualsFile(__FILE__ . '.output.txt', $process->getOutput());
         if (file_exists(__FILE__ . '.err.txt')) {
-            $this->assertStringEqualsFile(__FILE__ . '.err.txt', OutputCleaner::cleanOutput($process->getErrorOutput()));
+            $this->assertStringEqualsFile(__FILE__ . '.err.txt', $process->getErrorOutput());
         } else {
             $this->assertSame('', $process->getErrorOutput());
         }

tests/Examples/Generated/ContextGeneratorArg2Test.php

@@ -0,0 +1,22 @@
+<?php
+
+namespace Castor\Tests\Examples\Generated;
+
+use Castor\Tests\TaskTestCase;
+
+class ContextGeneratorArg2Test extends TaskTestCase
+{
+    // no task
+    public function test(): void
+    {
+        $process = $this->runTask([], '{{ base }}/tests/Examples/fixtures/broken/context-generator-arg-2');
+
+        $this->assertSame(1, $process->getExitCode());
+        $this->assertStringEqualsFile(__FILE__ . '.output.txt', $process->getOutput());
+        if (file_exists(__FILE__ . '.err.txt')) {
+            $this->assertStringEqualsFile(__FILE__ . '.err.txt', $process->getErrorOutput());
+        } else {
+            $this->assertSame('', $process->getErrorOutput());
+        }
+    }
+}

tests/Examples/Generated/ContextGeneratorArg2Test.php.err.txt

@@ -0,0 +1,8 @@
+
+In FunctionFinder.php line 218:
+
+  Function "gen()" is not properly configured:          
+  The context generator "foo" must not have arguments.  
+  Defined in "castor.php" line 7.                       
+
+

tests/Examples/Generated/ContextGeneratorArgTest.php

@@ -0,0 +1,22 @@
+<?php
+
+namespace Castor\Tests\Examples\Generated;
+
+use Castor\Tests\TaskTestCase;
+
+class ContextGeneratorArgTest extends TaskTestCase
+{
+    // no task
+    public function test(): void
+    {
+        $process = $this->runTask([], '{{ base }}/tests/Examples/fixtures/broken/context-generator-arg');
+
+        $this->assertSame(1, $process->getExitCode());
+        $this->assertStringEqualsFile(__FILE__ . '.output.txt', $process->getOutput());
+        if (file_exists(__FILE__ . '.err.txt')) {
+            $this->assertStringEqualsFile(__FILE__ . '.err.txt', $process->getErrorOutput());
+        } else {
+            $this->assertSame('', $process->getErrorOutput());
+        }
+    }
+}

tests/Examples/Generated/ContextGeneratorArgTest.php.err.txt

@@ -0,0 +1,8 @@
+
+In FunctionFinder.php line 209:
+
+  Function "gen()" is not properly configured:     
+  The contexts generator must not have arguments.  
+  Defined in "castor.php" line 7.                  
+
+

tests/Examples/Generated/ContextGeneratorNotCallableTest.php

@@ -0,0 +1,22 @@
+<?php
+
+namespace Castor\Tests\Examples\Generated;
+
+use Castor\Tests\TaskTestCase;
+
+class ContextGeneratorNotCallableTest extends TaskTestCase
+{
+    // no task
+    public function test(): void
+    {
+        $process = $this->runTask([], '{{ base }}/tests/Examples/fixtures/broken/context-generator-not-callable');
+
+        $this->assertSame(1, $process->getExitCode());
+        $this->assertStringEqualsFile(__FILE__ . '.output.txt', $process->getOutput());
+        if (file_exists(__FILE__ . '.err.txt')) {
+            $this->assertStringEqualsFile(__FILE__ . '.err.txt', $process->getErrorOutput());
+        } else {
+            $this->assertSame('', $process->getErrorOutput());
+        }
+    }
+}

tests/Examples/Generated/ContextGeneratorNotCallableTest.php.err.txt

@@ -0,0 +1,8 @@
+
+In FunctionFinder.php line 214:
+
+  Function "gen()" is not properly configured:  
+  The context generator "foo" is not callable.  
+  Defined in "castor.php" line 6.               
+
+

tests/Examples/Generated/NoDefaultContextTest.php

@@ -0,0 +1,22 @@
+<?php
+
+namespace Castor\Tests\Examples\Generated;
+
+use Castor\Tests\TaskTestCase;
+
+class NoDefaultContextTest extends TaskTestCase
+{
+    // no task
+    public function test(): void
+    {
+        $process = $this->runTask([], '{{ base }}/tests/Examples/fixtures/broken/no-default-context');
+
+        $this->assertSame(1, $process->getExitCode());
+        $this->assertStringEqualsFile(__FILE__ . '.output.txt', $process->getOutput());
+        if (file_exists(__FILE__ . '.err.txt')) {
+            $this->assertStringEqualsFile(__FILE__ . '.err.txt', $process->getErrorOutput());
+        } else {
+            $this->assertSame('', $process->getErrorOutput());
+        }
+    }
+}

tests/Examples/Generated/NoDefaultContextTest.php.err.txt

@@ -0,0 +1,6 @@
+
+In ContextRegistry.php line XXXX:
+
+  Since there are multiple contexts "one", "two", you must set a "default: true" context.  
+
+

tests/Examples/Generated/TwoDefaultContextTest.php

@@ -0,0 +1,22 @@
+<?php
+
+namespace Castor\Tests\Examples\Generated;
+
+use Castor\Tests\TaskTestCase;
+
+class TwoDefaultContextTest extends TaskTestCase
+{
+    // no task
+    public function test(): void
+    {
+        $process = $this->runTask([], '{{ base }}/tests/Examples/fixtures/broken/two-default-context');
+
+        $this->assertSame(1, $process->getExitCode());
+        $this->assertStringEqualsFile(__FILE__ . '.output.txt', $process->getOutput());
+        if (file_exists(__FILE__ . '.err.txt')) {
+            $this->assertStringEqualsFile(__FILE__ . '.err.txt', $process->getErrorOutput());
+        } else {
+            $this->assertSame('', $process->getErrorOutput());
+        }
+    }
+}

tests/Examples/Generated/TwoDefaultContextTest.php.err.txt

@@ -0,0 +1,8 @@
+
+In ContextRegistry.php line XXXX:
+
+  Function "two()" is not properly configured:                                                                  
+  You cannot define two contexts with the same name "default". There is one already defined in "castor.php:7".  
+  Defined in "castor.php" line 13.                                                                              
+
+

tests/Examples/fixtures/broken/context-generator-arg-2/castor.php

@@ -0,0 +1,10 @@
+<?php
+
+use Castor\Attribute\AsContextGenerator;
+use Castor\Context;
+
+#[AsContextGenerator()]
+function gen(): iterable
+{
+    yield 'foo' => fn ($a) => new Context();
+}

tests/Examples/fixtures/broken/context-generator-arg/castor.php

@@ -0,0 +1,10 @@
+<?php
+
+use Castor\Attribute\AsContextGenerator;
+use Castor\Context;
+
+#[AsContextGenerator()]
+function gen($a): iterable
+{
+    return new Context();
+}

tests/Examples/fixtures/broken/context-generator-not-callable/castor.php

@@ -0,0 +1,9 @@
+<?php
+
+use Castor\Attribute\AsContextGenerator;
+
+#[AsContextGenerator()]
+function gen(): iterable
+{
+    yield 'foo' => 'not a callable';
+}

tests/Examples/fixtures/broken/no-default-context/castor.php

@@ -0,0 +1,16 @@
+<?php
+
+use Castor\Attribute\AsContext;
+use Castor\Context;
+
+#[AsContext()]
+function one(): Context
+{
+    return new Context();
+}
+
+#[AsContext()]
+function two(): Context
+{
+    return new Context();
+}

tests/Examples/fixtures/broken/two-default-context/castor.php

@@ -0,0 +1,16 @@
+<?php
+
+use Castor\Attribute\AsContext;
+use Castor\Context;
+
+#[AsContext(default: true)]
+function one(): Context
+{
+    return new Context();
+}
+
+#[AsContext(default: true)]
+function two(): Context
+{
+    return new Context();
+}

tests/OutputCleaner.php → tests/Helper/OutputCleaner.php

@@ -1,6 +1,6 @@
 <?php
 
-namespace Castor\Tests;
+namespace Castor\Tests\Helper;
 
 final class OutputCleaner
 {
@@ -16,6 +16,6 @@ public static function cleanOutput(string $string): string
         $string = preg_replace('{you are using v\d+.\d+.\d+.}m', 'you are using vX.Y.Z.', $string);
         $string = preg_replace('{you are using v\d+.\d+.\d+.}m', 'you are using vX.Y.Z.', $string);
 
-        return str_replace(\dirname(__DIR__, 1), '...', $string);
+        return str_replace(\dirname(__DIR__, 2), '...', $string);
     }
 }

tests/CompileCommandTest.php → tests/Slow/CompileCommandTest.php

@@ -1,6 +1,6 @@
 <?php
 
-namespace Castor\Tests;
+namespace Castor\Tests\Slow;
 
 use PHPUnit\Framework\TestCase;
 use Symfony\Component\Process\Process;

tests/RepackCommandTest.php → tests/Slow/RepackCommandTest.php

@@ -1,6 +1,6 @@
 <?php
 
-namespace Castor\Tests;
+namespace Castor\Tests\Slow;
 
 use PHPUnit\Framework\TestCase;
 use Symfony\Component\Filesystem\Filesystem;

tests/TaskTestCase.php

@@ -2,6 +2,7 @@
 
 namespace Castor\Tests;
 
+use Castor\Tests\Helper\OutputCleaner;
 use Castor\Tests\Helper\WebServerHelper;
 use PHPUnit\Framework\TestCase;
 use Symfony\Component\Process\Process;
@@ -35,7 +36,7 @@ public function runTask(array $args, ?string $cwd = null): Process
 
         $process = new Process(
             [$castorBin, '--no-ansi', ...$args],
-            cwd: $cwd ?? __DIR__ . '/..',
+            cwd: $cwd ? str_replace('{{ base }}', __DIR__ . '/..', $cwd) : __DIR__ . '/..',
             env: [
                 'COLUMNS' => 120,
                 ...$extraEnv,