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

Transaction - Add test on Transaction #16

Open
3 tasks
stombre opened this issue Sep 15, 2020 · 0 comments
Open
3 tasks

Transaction - Add test on Transaction #16

stombre opened this issue Sep 15, 2020 · 0 comments
Labels
available for contribution If you search issue, you could contribute in component - transaction Subject related to Transaction enhancement New feature or request good first issue Good for newcomers

Comments

@stombre
Copy link
Contributor

stombre commented Sep 15, 2020

Currently, the test scenario on Transaction include the three ways to do transaction with ilorm with findOne and save operation.

Some scenarios cases for Transaction are missing, this issue could eventually be done by different PR and different test;

  • Test instance.remove operator on transaction
  • Test throw an error in a Transaction.run (to trigger rollback).
  • Test rollback by calling the transaction.rollback operator

Any other cases that seems missing could be added.

@stombre stombre added enhancement New feature or request good first issue Good for newcomers available for contribution If you search issue, you could contribute in component - transaction Subject related to Transaction labels Sep 15, 2020
@stombre stombre mentioned this issue Sep 15, 2020
7 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
available for contribution If you search issue, you could contribute in component - transaction Subject related to Transaction enhancement New feature or request good first issue Good for newcomers
Projects
None yet
Development

No branches or pull requests

1 participant