Skip to content

[TASK] Move/rename Rule/RuleTest.php to Property/DeclarationTest.php#1511

Merged
oliverklee merged 1 commit intomainfrom
test/property-declaration
Feb 12, 2026
Merged

[TASK] Move/rename Rule/RuleTest.php to Property/DeclarationTest.php#1511
oliverklee merged 1 commit intomainfrom
test/property-declaration

Conversation

@JakeQZ
Copy link
Collaborator

@JakeQZ JakeQZ commented Feb 12, 2026

Test the new Declaration class instead of the deprecated Rule alias.

Also rename some variables and parameters in the tests from $rule to $declaration.

Test the new `Declaration` class instead of the deprecated `Rule` alias.

Also rename some variables and parameters in the tests from `$rule` to
`$declaration`.
@JakeQZ JakeQZ requested a review from oliverklee February 12, 2026 17:07
@JakeQZ JakeQZ self-assigned this Feb 12, 2026
@JakeQZ JakeQZ added testing PRs/issues adding additional tests only, or primarily testing-focused developer-specific Issues that only affect maintainers, contributors, and people submitting PRs refactor For PRs that refactor code without changing functionality labels Feb 12, 2026
@coveralls
Copy link

Coverage Status

coverage: 72.779%. remained the same
when pulling d9a804e on test/property-declaration
into 802cebe on main.

@oliverklee oliverklee merged commit 7941744 into main Feb 12, 2026
24 checks passed
@oliverklee oliverklee deleted the test/property-declaration branch February 12, 2026 17:10
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

developer-specific Issues that only affect maintainers, contributors, and people submitting PRs refactor For PRs that refactor code without changing functionality testing PRs/issues adding additional tests only, or primarily testing-focused

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants