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

Initial checkin of prototype of merged tests #583

Closed
wants to merge 1 commit into from

Conversation

mindhog
Copy link
Member

@mindhog mindhog commented May 12, 2020

XXX DO NOT MERGE XXX

This currently fails because JpaUnitTestRule doesn't implement the extension
interface yet.


This change is Reviewable

This currently fails because JpaUnitTestRule doesn't implement the extension
interface yet.
@sarahcaseybot
Copy link
Collaborator

Not sure if it's what you need, but I submitted a PR this morning that implements some of the extension interface callbacks for JpaUnitTestRule

@hstonec
Copy link
Contributor

hstonec commented May 12, 2020

Thanks Sara, I will take a look. This PR is for the purpose to merge the unit test of DatastoreTransacntionManagerTest and JpaTransactionManagerImplTest into a single code path.

@jianglai jianglai closed this May 14, 2021
@jianglai
Copy link
Collaborator

Closing due to inactivity.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

5 participants