Skip to content

Rename package from utopia-php/test to utopia-php/tests#2

Merged
ChiragAgg5k merged 1 commit into
mainfrom
rename-to-tests
Dec 15, 2025
Merged

Rename package from utopia-php/test to utopia-php/tests#2
ChiragAgg5k merged 1 commit into
mainfrom
rename-to-tests

Conversation

@ChiragAgg5k

@ChiragAgg5k ChiragAgg5k commented Dec 15, 2025

Copy link
Copy Markdown
Member

Summary

This PR renames the package from utopia-php/test to utopia-php/tests to better reflect that it provides multiple testing utilities and extensions.

Changes

  • ✅ Updated package name in composer.json
  • ✅ Updated PSR-4 autoload namespace from Utopia\Test to Utopia\Tests
  • ✅ Moved source files from src/Test to src/Tests
  • ✅ Updated all namespace declarations and use statements
  • ✅ Updated documentation and code examples in README.md
  • ✅ Updated test files to use new namespace

Testing

All existing tests continue to work with the new namespace structure.

Summary by CodeRabbit

  • Documentation

    • Updated installation instructions and code examples to reflect package rename and new namespace paths.
  • Chores

    • Renamed package from utopia-php/test to utopia-php/tests with corresponding namespace updates.
    • Added phpstan 2.0 as a development dependency.

✏️ Tip: You can customize this high-level summary in your review settings.

- Update package name in composer.json
- Update PSR-4 autoload namespace from Utopia\Test to Utopia\Tests
- Move source files from src/Test to src/Tests
- Update all namespace declarations and use statements
- Update documentation and code examples in README.md
- Update test files to use new namespace

This change makes the package name plural to better reflect that it provides multiple testing utilities and extensions.
@coderabbitai

coderabbitai Bot commented Dec 15, 2025

Copy link
Copy Markdown

Caution

Review failed

The pull request is closed.

Walkthrough

This PR executes a comprehensive namespace migration from Utopia\Test\ to Utopia\Tests\ (plural) across the codebase. Changes include updating the package name in composer.json from utopia-php/test to utopia-php/tests, adjusting the PSR-4 autoload mapping to Utopia\Tests\, updating namespace declarations in all source files (Async.php, Eventually.php, Critical.php), updating import statements in test files, and synchronizing README.md documentation with the new namespace paths. A phpstan dependency was also added to require-dev.

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~10 minutes

  • Consistent pattern: The namespace refactoring follows a single, homogeneous pattern applied uniformly across multiple files
  • No logic changes: All modifications are straightforward namespace/naming updates with no functional logic alterations
  • Verification focus: Spot-check several files to ensure all namespace references were consistently updated and no stray references to the old namespace remain in the codebase
✨ Finishing touches
  • 📝 Generate docstrings
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment
  • Commit unit tests in branch rename-to-tests

📜 Recent review details

Configuration used: CodeRabbit UI

Review profile: CHILL

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between 143c33c and f6a6214.

📒 Files selected for processing (6)
  • README.md (3 hunks)
  • composer.json (2 hunks)
  • src/Tests/Extensions/Async.php (1 hunks)
  • src/Tests/Extensions/Async/Eventually.php (1 hunks)
  • src/Tests/Extensions/Async/Exceptions/Critical.php (1 hunks)
  • tests/AsyncTest.php (1 hunks)

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

@ChiragAgg5k ChiragAgg5k merged commit c2bca63 into main Dec 15, 2025
3 of 4 checks passed
@ChiragAgg5k ChiragAgg5k deleted the rename-to-tests branch December 15, 2025 08:28
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants