Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Unit Test: Ensure that apply_filters is extracted, and that the parameters to it are as well. #239

Merged
merged 3 commits into from
Jan 18, 2024

Conversation

dd32
Copy link
Member

@dd32 dd32 commented Jan 18, 2024

Pretty self explanatory, there's a lack of proper unit tests for some aspects of this parser, which makes upgrading the libraries difficult if we've got no baseline.

@dd32 dd32 merged commit e30c309 into WordPress:master Jan 18, 2024
1 check passed
dd32 added a commit to dd32/phpdoc-parser that referenced this pull request Jan 18, 2024
…rameters to it are as well. (WordPress#239)

* Add a apply_filters() unit test.

* Add a way to test that the arguments are extracted.

* Add test groups
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.

1 participant