Conversation
mogmarsh
marked this pull request as ready for review
September 9, 2026 21:39
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
Fixes #215
Description
This PR introduces a comprehensive test suite to ensure the ongoing reliability of the
configure.phpscript. It covers all functions withinconfigure.phpso that any future changes to the codebase will not inadvertently cause regressions in the setup process.Key enhancements include:
Automated Testing:
tests/ConfigureTest.php) simulates end-to-end runs ofconfigure.phpusing scripted answers. It verifies the correct file structure and content after configuration.composer test:configure) immediately after installation and prior to running the realconfigure.phpscript. This ensures the skeleton's configuration process is validated before any modifications occur.Script and Tooling Updates:
composer.jsonfile has been updated so that thetestscript now runs the configure tests alongside existing checks likephpcs,phpstan, andphpunit.test:configurescript has been added for focused execution of the configure script tests.Documentation Improvements:
CONTRIBUTING.mdnow includes a section explaining how to run and maintain the configure script’s test suite, with details on test setup and expected assertions.Cleanup Mechanism:
configure.phphas been refactored to remove test infrastructure (such asConfigureTest.php) from the initialized project, ensuring these files remain only in the skeleton/template.Use Case
With these changes, users and maintainers can confidently update the project, knowing that any unintended changes to the configuration process will be caught by automated tests. This safeguards the setup experience for all users.
Testing Instructions
composer testorcomposer test:configure) to verify that all functions inconfigure.phpare covered and functioning correctly.configure.phpand confirm that the test suite detects any regressions or failures.Additional Notes
No additional changes outside the scope described above. The added test infrastructure exists only in the skeleton and is removed from initialized projects to avoid unnecessary files.