opensearch-project / opensearch-project/sql
[FOLLOWUP] Reactive the explain tests in 3655.yml
Open
Nobody has claimed this yet.
enhancement
PPL
testing
- Dominant language
- Java
- Stars
- 176
- Forks
- 229
- Avg merge
- 2d 21h
- Merged PRs (30d)
- 43
Description
Is your feature request related to a problem?
The explain tests in 3655.yml is ignored as its expectation results have changed, and the new results contains a dynamic pitId
What solution would you like?
Rest test framework should support regex match.
Contributor guide
First steps
- Read the whole issue, then the project's contributing guide.
- Comment on the issue to say you are picking it up — it saves two people doing the same work.
- Fork the repository and make your change on a branch.
- Open a pull request that references the issue number.
Research direction
Start by locating 3655.yml and the REST test framework that runs its explain tests. Identify how expected results are compared and add support for matching the dynamic pitId with a regex. Done means the explain tests are no longer ignored and pass against the changed results.
Written by the indexing model from the issue text.
Assessment
- Tech stack
- java
- Domain
- testing
- Issue type
- Feature
- Difficulty
- 3/5
- Estimated time
- 1-2 days
- Activity status
- Stale
- Clarity
- Mostly clear
- Newbie friendliness
- 48/100