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.

Sign up for GitHub

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

Jump to bottom

Add/tests for Data and Verify #81

Merged
merged 17 commits into from
Jun 19, 2024
Merged

Add/tests for Data and Verify #81

merged 17 commits into from
Jun 19, 2024

Conversation

BrianHenryIE
Copy link
Contributor

Proposed changes

  • Add DataTest, lint, comment, typehint Data
  • Add VerifyTest, lint, comment, typehint Verify
  • Fix HiiveConnectionTest

Type of Change

  • New feature (non-breaking change which adds functionality)
  • Documentation Update (if none of the other choices apply)

Checklist

  • I have read the CONTRIBUTING doc
  • Linting and tests pass locally with my changes
  • I have added tests that prove my fix is effective or that my feature works
  • I have added necessary documentation (if appropriate)

Further comments

BrianHenryIE added 17 commits May 14, 2024 18:06
@BrianHenryIE
Local dev 1
fce2760
@BrianHenryIE
Add tests for \NewfoldLabs\WP\Module\Data\API\Verify
5f461a3
@BrianHenryIE
Minimal wp-browser wpunit config
8e90229
@BrianHenryIE
Set phpcs php version to 7.3
b305cea
@BrianHenryIE
Set PHP platform to 7.3
f2c0e52
@BrianHenryIE
Update test config
bb5d08a
@BrianHenryIE
Add composer cs-changes to fix and lint only changed files
60d2f01
@BrianHenryIE
Add wordpress directory to autoload-dev
5ae4d1c
@BrianHenryIE
Test Data::authenticate()
5c18c04
@BrianHenryIE
Update authenticate function to match rest_authentication_errors …
4139a59 
…filter
@BrianHenryIE
Add function names which are redefined in DataTest
ed54b60
@BrianHenryIE
Fix incorrect return
eb419a5
@BrianHenryIE
Comment where the action is added
b1bacc3
@BrianHenryIE
Simplify by replacing get_items_permissions_check with __return_true
790ce4b
@BrianHenryIE
Add test for Commerce::woocommerce_hpos_enabled()
732b2e2
@BrianHenryIE
Merge branch 'main' into Add/test-dev-env
96554eb
@BrianHenryIE
Update composer.lock
22c736c
@BrianHenryIE BrianHenryIE merged commit 64ef7df into main Jun 19, 2024
3 of 8 checks passed
@BrianHenryIE BrianHenryIE deleted the Add/test-dev-env branch June 19, 2024 16:16
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@circlecube circlecube circlecube approved these changes

@wpscholar wpscholar Awaiting requested review from wpscholar

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@BrianHenryIE @circlecube