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

Rearrange integration tests #558

Merged
merged 1 commit into from
Apr 20, 2022
Merged

Conversation

brfrn169
Copy link
Collaborator

@brfrn169 brfrn169 commented Apr 19, 2022

This PR rearranges the integration tests.

What I did mainly in this PR is as follows:

  • Deleted integration-test-for-two-phase-consensus-commit and run the two-phase commit transaction integration tests in each database integration test
  • Moved and rearranged the integration test classes

I basically didn't change the integration tests themselves behaviors.

Please take a look!

@brfrn169 brfrn169 self-assigned this Apr 19, 2022
Copy link
Contributor

@Torch3333 Torch3333 left a comment

Choose a reason for hiding this comment

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

LGTM, thank you!

Copy link
Contributor

@feeblefakie feeblefakie left a comment

Choose a reason for hiding this comment

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

LGTM! Thank you!

@feeblefakie feeblefakie merged commit a740cb1 into master Apr 20, 2022
@feeblefakie feeblefakie deleted the rearrange-integration-tests branch April 20, 2022 06:50
brfrn169 added a commit that referenced this pull request Apr 27, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants