feat: Add `init` command by burn1ngbear · Pull Request #182 · php-testo/testo

burn1ngbear · 2026-05-15T07:44:21Z

What was changed

See issue 174

Checklist

Closes #
Tested
- Tested manually
- Unit tests added
Documentation

burn1ngbear · 2026-05-15T07:45:50Z

#174

burn1ngbear · 2026-05-15T09:34:21Z

@roxblnfk review pls

Copilot

Pull request overview

Adds an init command to the Symfony Console bridge so projects can bootstrap Testo configuration, test directories, and a Composer script.

Changes:

Added a new Init console command.
Added a testo.php configuration stub.
Registered the init command in the CLI entrypoint.

Reviewed changes

Copilot reviewed 2 out of 3 changed files in this pull request and generated 2 comments.

File	Description
`bridge/symfony-console/src/Command/Init.php`	Implements the new project initialization flow.
`bridge/symfony-console/resources/stubs/testo.php`	Provides the generated configuration template.
`bridge/symfony-console/bin/testo`	Registers the new `init` command with the console application.

Comments suppressed due to low confidence (2)

bridge/symfony-console/src/Command/Init.php:136

When testo.php does not already exist, this block is skipped entirely, so the init command never writes the new configuration file on a fresh project. The stub generation should also run for the missing-file case; otherwise the final message points to a configuration file that was not created.

        if (\is_file(self::DESTINATION)) {
            if (!$input->isInteractive()) {
                $io->warning('testo.php already exists. Skipping (non-interactive mode).');
                return Command::SUCCESS;
            }

            if ($io->confirm('testo.php already exists. Overwrite?', false)) {

bridge/symfony-console/src/Command/Init.php:140

The raw path values are interpolated into PHP single-quoted strings in the stub. A user-supplied path containing an apostrophe (or other PHP string-sensitive characters) will generate an invalid or incorrect testo.php; the generated values should be escaped/serialized before being written into PHP code.

                $stubContent = \str_replace(
                    ['__SRC_PATH__', '__TESTS_UNIT_PATH__'],
                    [(string) $srcPath, (string) $testsUnitPath],
                    \file_get_contents(self::STUB),

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com>

Copilot

Pull request overview

Copilot reviewed 2 out of 3 changed files in this pull request and generated 6 comments.

Comments suppressed due to low confidence (2)

bridge/symfony-console/src/Command/Init.php:153

file_get_contents(self::STUB) isn’t checked for failure. If the stub path is missing/unreadable, this will produce an invalid/empty testo.php. Consider validating the stub read and failing the command with a clear error message instead of proceeding.

                ['__SRC_PATH__', '__SUITES__'],
                [(string) $srcPath, $suitesCode],
                \file_get_contents(self::STUB),
            );
            \file_put_contents(self::DESTINATION, $stubContent);

bridge/symfony-console/src/Command/Init.php:155

file_put_contents(self::DESTINATION, ...) isn’t checked. If writing testo.php fails (permissions, existing file locked), the command will still report success. Consider checking for false and returning Command::FAILURE with an actionable error message.

                \file_get_contents(self::STUB),
            );
            \file_put_contents(self::DESTINATION, $stubContent);
            $io->success('Created testo.php');
        }

Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com>

Copilot

Pull request overview

Copilot reviewed 7 out of 8 changed files in this pull request and generated 12 comments.

+        if ($path->isDir()) {
+            return;
+        }
+
+        \mkdir((string) $path, 0755, true);


+            $basePath = Path::create((string) $input->getOption('path'));
+            self::ensureDirectory($basePath, $io);
+
+            $srcPath = self::discoverSourceDirectory($input, $io);
+
+            $testsPath = $basePath->join('tests');
+            self::ensureDirectory($testsPath, $io);
+
+            $suites = self::discoverSuites($testsPath, $io);
+            $composerKeys = self::updateComposerScripts($suites, $basePath, $io);
+


+        }
+
+        if (!$input->isInteractive()) {
+            throw new \RuntimeException('src/ directory not found. Skipping (non-interactive mode).');


+        /** @var array{scripts?: array<string, string>}&array<string, mixed> $composer */
+        $composer = \json_decode(
+            \file_get_contents((string) $composerJsonPath),
+            associative: true,
+            flags: \JSON_THROW_ON_ERROR,
+        );
+
+        if (!isset($composer['scripts'][self::SCRIPT_ALL_KEY])) {
+            $composer['scripts'][self::SCRIPT_ALL_KEY] = self::SCRIPT_ALL_COMMAND;
+            $io->success(\sprintf('Added "%s" script to composer.json', self::SCRIPT_ALL_KEY));
+        }
+
+        foreach ($suites as $suite) {
+            $key = \sprintf(self::SCRIPT_SUITE_KEY_TEMPLATE, \strtolower($suite));
+            $keys[] = $key;
+
+            if (!isset($composer['scripts'][$key])) {
+                $composer['scripts'][$key] = \sprintf(self::SCRIPT_SUITE_COMMAND_TEMPLATE, $suite);
+                $io->success(\sprintf('Added "%s" script to composer.json', $key));
+            }


+        \file_put_contents(
+            (string) $composerJsonPath,
+            \json_encode($composer, \JSON_PRETTY_PRINT | \JSON_UNESCAPED_SLASHES | \JSON_UNESCAPED_UNICODE) . "\n",
+        );
+


+            \file_get_contents(self::STUB),
+        );
+
+        \file_put_contents((string) $configPath, $stub);


+        $keys = [self::SCRIPT_ALL_KEY];
+
+        $composerJsonPath = $basePath->join('composer.json');
+        if (!$composerJsonPath->isFile()) {
+            return $keys;
+        }
+


+    public function customPathPlacesTestsAndConfigUnderSubdirectory(): void
+    {
+        $this->sandbox->makeDir('src');
+        $this->sandbox->writeFile(
+            'app/composer.json',
+            \json_encode(['name' => 'acme/sub-app'], \JSON_PRETTY_PRINT | \JSON_UNESCAPED_SLASHES) . "\n",
+        );
+        $rootComposer = \json_encode(['name' => 'acme/root'], \JSON_PRETTY_PRINT | \JSON_UNESCAPED_SLASHES) . "\n";
+        $this->sandbox->writeFile('composer.json', $rootComposer);
+
+        $tester = $this->run(['--path' => 'app']);
+
+        Assert::same(
+            $tester->getStatusCode(),
+            Command::SUCCESS,
+            'init must succeed when --path points at a subdirectory; output: ' . $tester->getDisplay(),
+        );
+
+        Assert::true(\is_dir($this->sandbox->path('app/tests/Unit')), 'tests/Unit/ must be created under --path');
+        Assert::true(\is_file($this->sandbox->path('app/testo.php')), 'testo.php must be created under --path');
+
+        Assert::false(
+            \is_file($this->sandbox->path('testo.php')),
+            'no testo.php must be created at the project root when --path is set',
+        );
+        Assert::false(
+            \is_dir($this->sandbox->path('tests')),
+            'no tests/ must be created at the project root when --path is set',
+        );
+
+        $subScripts = $this->readComposerScripts('app/composer.json');
+        Assert::same(
+            $subScripts['test:unit'] ?? null,
+            'vendor/bin/testo --suite=Unit',
+            'composer scripts must be written to the composer.json colocated with --path',
+        );
+
+        Assert::same(
+            \file_get_contents($this->sandbox->path('composer.json')),
+            $rootComposer,
+            'the project-root composer.json must be left untouched when --path is set',
+        );
+    }


+    {
+        $target = $this->path . \DIRECTORY_SEPARATOR . \ltrim($relative, '/\\');
+        $dir = \dirname($target);
+        \is_dir($dir) or \mkdir($dir, 0o755, true);


+
+    public function destroy(): void
+    {
+        \chdir($this->originalCwd);


Sergey Syomkin added 5 commits May 14, 2026 22:44

Init command

eac9396

Init command

56e3b1c

Init command

d98b29b

Init command

2291340

Init command

a0cc2f0

burn1ngbear requested a review from a team as a code owner May 15, 2026 07:44

Init command

d21108a

roxblnfk and others added 2 commits May 15, 2026 13:34

Merge branch '1.x' into 1.x

00082a8

fix

3046109

roxblnfk requested a review from Copilot May 15, 2026 11:37

Copilot started reviewing on behalf of roxblnfk May 15, 2026 11:37 View session

Copilot AI reviewed May 15, 2026

View reviewed changes

Comment thread bridge/symfony-console/src/Command/Init.php Outdated

Comment thread bridge/symfony-console/resources/stubs/testo.php Outdated

Potential fix for pull request finding

e04ad34

Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com>

roxblnfk reviewed May 15, 2026

View reviewed changes

Comment thread bridge/symfony-console/resources/stubs/testo.php

roxblnfk changed the title ~~Add init command~~ feat: Add init command May 15, 2026

Sergey Syomkin added 3 commits May 20, 2026 12:08

search suite

9dee050

Merge remote-tracking branch 'origin/1.x' into 1.x

1568cbe

create testo.php is dont exist

e7494cb

roxblnfk requested a review from Copilot May 20, 2026 09:47

Copilot started reviewing on behalf of roxblnfk May 20, 2026 09:48 View session

Copilot AI reviewed May 20, 2026

View reviewed changes

burn1ngbear and others added 3 commits May 20, 2026 13:25

Potential fix for pull request finding

0a071c4

Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com>

refactor: Update Init command; add acceptance tests

5d5a0d0

fix: Security fixes

4ba0e1c

roxblnfk requested a review from Copilot May 20, 2026 11:25

Copilot started reviewing on behalf of roxblnfk May 20, 2026 11:26 View session

roxblnfk added 2 commits May 20, 2026 15:30

fix: Add missing test directory to FinderConfig

7930724

Merge branch '1.x' into 1.x

d054881

Copilot AI reviewed May 20, 2026

View reviewed changes

roxblnfk approved these changes May 20, 2026

View reviewed changes

roxblnfk merged commit cd940d6 into php-testo:1.x May 20, 2026
10 checks passed

This was referenced May 20, 2026

chore(1.x): release bridge-symfony-console 0.1.3 #186

Merged

chore(1.x): release 0.10.12 #187

Merged

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Uh oh!

feat: Add `init` command#182

feat: Add `init` command#182
roxblnfk merged 17 commits into
php-testo:1.xfrom
burn1ngbear:1.x

burn1ngbear commented May 15, 2026 •

edited

Loading

Uh oh!

burn1ngbear commented May 15, 2026

Uh oh!

burn1ngbear commented May 15, 2026

Uh oh!

Copilot AI left a comment

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Copilot AI left a comment

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Copilot AI left a comment

Uh oh!

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

3 participants

Uh oh!

Conversation

burn1ngbear commented May 15, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

What was changed

Checklist

Uh oh!

burn1ngbear commented May 15, 2026

Uh oh!

burn1ngbear commented May 15, 2026

Uh oh!

Copilot AI left a comment

Choose a reason for hiding this comment

Pull request overview

Reviewed changes

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Copilot AI left a comment

Choose a reason for hiding this comment

Pull request overview

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Copilot AI left a comment

Choose a reason for hiding this comment

Pull request overview

Uh oh!

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

3 participants

burn1ngbear commented May 15, 2026 •

edited

Loading