unfulvio-godaddy commented 1 year ago

Summary

While adding https://github.com/10up/wp_mock/pull/220 I forgot one method (for @10up/godaddy-wp_mock these are basically the same as in MWC Tests, and with the addition of this PR we can drop them -- figured these are generally helpful and could be part of WP_Mock directly).

This PR also updates the docs related to AccessInaccessibleClassMembersTrait.

Fixes an unrelated flaky test.

Contributor checklist

[x] I agree to follow this project's Code of Conduct.
[x] I have updated the documentation accordingly
[x] I have added tests to cover changes introduced by this pull request
[x] All new and existing tests pass

Testing

Reviewer checklist

- [ ] Code changes review - [ ] Documentation changes review - [ ] Unit tests pass - [ ] Static analysis passes

coveralls commented 1 year ago

coverage: 65.6% (+0.05%) from 65.554% when pulling 77d399804de96dacee4cb07638c919c0561fdba7 on chore/add-set-inaccessible-property-value-helper-method into c176fd84d20636fb656d9f12f6cfff8d171c78c9 on trunk.

unfulvio-godaddy commented 1 year ago

thanks for the suggestiosn @nmolham-godaddy - I applied those changes with references to PHP.net docs

10up / wp_mock

Add setInaccessiblePropertyValue helper method and update docs #223

Summary

Contributor checklist

Testing

Reviewer checklist