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

Refactor spec tester to improve reusability of classes #302

Merged
merged 10 commits into from
Jun 6, 2024

Conversation

Xtansia
Copy link
Collaborator

@Xtansia Xtansia commented May 31, 2024

Description

Refactors sections of the spec tester to allow better reusability of the classes and remove the SharedResources pattern. Additionally pulls out a common set of parameters for connecting to OpenSearch.

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.
For more information on following Developer Certificate of Origin and signing off your commits, please check here.

Copy link
Contributor

github-actions bot commented May 31, 2024

Changes Analysis

Commit SHA: c2a59e7
Comparing To SHA: 6711e8f

API Changes

Summary

NO CHANGES

Report

The full API changes report is available at: https://github.com/opensearch-project/opensearch-api-specification/actions/runs/9392314713/artifacts/1573036448

API Coverage

Before After Δ
Covered (%) 476 (46.62 %) 476 (46.62 %) 0 (0 %)
Uncovered (%) 545 (53.38 %) 545 (53.38 %) 0 (0 %)
Unknown 24 24 0

dblock
dblock previously approved these changes May 31, 2024
Copy link
Member

@dblock dblock left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I like it! Leaving for @nhtruong.

@nhtruong
Copy link
Collaborator

nhtruong commented Jun 4, 2024

@Xtansia If you dont mind updating this later, let's wait for this PR to be merged first before we do a refactoring.

@Xtansia Xtansia marked this pull request as draft June 5, 2024 00:37
@Xtansia Xtansia force-pushed the refactor/reusability branch 2 times, most recently from 47af341 to 783cfa0 Compare June 5, 2024 05:17
@nhtruong nhtruong mentioned this pull request Jun 5, 2024
@Xtansia Xtansia force-pushed the refactor/reusability branch from 7790972 to 7695759 Compare June 5, 2024 22:33
@Xtansia Xtansia marked this pull request as ready for review June 5, 2024 22:47
@Xtansia
Copy link
Collaborator Author

Xtansia commented Jun 5, 2024

@dblock @nhtruong This is ready for you both to have a look at. Can also break it into a couple separate PRs if you like

@Xtansia Xtansia requested a review from dblock June 5, 2024 22:48
Signed-off-by: Thomas Farr <[email protected]>
Copy link
Member

@dblock dblock left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Let's merge this. I'll do some more simplifying in #316

@dblock dblock merged commit 698266e into opensearch-project:main Jun 6, 2024
7 checks passed
@Xtansia Xtansia deleted the refactor/reusability branch June 6, 2024 00:39
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.

3 participants