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

Add some unit tests to cover changes in PR 46972 #47098

Merged
merged 3 commits into from May 14, 2024
Merged

Conversation

jorgeatorres
Copy link
Member

Submission Review Guidelines:

Changes proposed in this Pull Request:

In #46972 we fixed an issue with HPOS searches and terms including possibly url-encoded characters. This PR adds some unit tests to cover those changes.
It's basically a failing test for the prior behavior.

How to test the changes in this Pull Request:

Using the WooCommerce Testing Instructions Guide, include your detailed testing instructions:

From within the plugins/woocommerce directory:

  1. Run the new tests and make sure they pass:
    vendor/bin/phpunit --filter=test_query_s_argument
    or (if using wpenv)
    pnpm test:php:env --filter=test_query_s_argument
  2. (Optional) Confirm the tests don't pass in 8.8.3 (which has the bug):
    git co 8.8.3
    git co tests/test-46972 -- tests/php/src/Internal/DataStores/Orders/OrdersTableQueryTests.php
    vendor/bin/phpunit --filter=test_query_s_argument
    
    If you do this, reset the local changes by running git reset --hard.

Changelog entry

  • Automatically create a changelog entry from the details below.

Significance

  • Patch
  • Minor
  • Major

Type

  • Fix - Fixes an existing bug
  • Add - Adds functionality
  • Update - Update existing functionality
  • Dev - Development related task
  • Tweak - A minor adjustment to the codebase
  • Performance - Address performance issues
  • Enhancement - Improvement to existing functionality

Message

Comment

@jorgeatorres jorgeatorres requested review from a team and vedanshujain and removed request for a team May 1, 2024 16:43
@github-actions github-actions bot added the plugin: woocommerce Issues related to the WooCommerce Core plugin. label May 1, 2024
Copy link
Contributor

github-actions bot commented May 1, 2024

Hi @vedanshujain,

Apart from reviewing the code changes, please make sure to review the testing instructions as well.

You can follow this guide to find out what good testing instructions should look like:
https://github.com/woocommerce/woocommerce/wiki/Writing-high-quality-testing-instructions

1 similar comment
Copy link
Contributor

github-actions bot commented May 1, 2024

Hi @vedanshujain,

Apart from reviewing the code changes, please make sure to review the testing instructions as well.

You can follow this guide to find out what good testing instructions should look like:
https://github.com/woocommerce/woocommerce/wiki/Writing-high-quality-testing-instructions

@vedanshujain vedanshujain merged commit 089be92 into trunk May 14, 2024
21 checks passed
@vedanshujain vedanshujain deleted the tests/test-46972 branch May 14, 2024 15:04
@github-actions github-actions bot added this to the 9.0.0 milestone May 14, 2024
@github-actions github-actions bot added the needs: analysis Indicates if the PR requires a PR testing scrub session. label May 14, 2024
@nigeljamesstevenson nigeljamesstevenson added status: analysis complete Indicates if a PR has been analysed by Solaris and removed needs: analysis Indicates if the PR requires a PR testing scrub session. labels May 14, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
plugin: woocommerce Issues related to the WooCommerce Core plugin. status: analysis complete Indicates if a PR has been analysed by Solaris
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants