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.

Sign up for GitHub

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

Jump to bottom

[Coverage]: Add advanced tests for transaction subsystem configuration #97

Open
OndrejGerzicak opened this issue Jul 10, 2023 · 0 comments
Open

[Coverage]: Add advanced tests for transaction subsystem configuration #97

OndrejGerzicak opened this issue Jul 10, 2023 · 0 comments
Labels
coverage subsystem transaction

Comments

@OndrejGerzicak
Copy link
Collaborator

OndrejGerzicak commented Jul 10, 2023 •
edited by OndrejKotek
Loading

Description

The basic tests migrated from legacy TS doesn't seem to be complex enough, see #34 and #96.

We should also be testing:

  • Invalid input values
  • Invalid scenarios when some attribute cannot be written when other one is not set, etc.
  • Consider adding Reset test
  • Consider how to handle coverage for /subsystem=transactions/commit-markable-resource and /subsystem=transactions/log-store

Affects

configuration

Subsystem

transaction
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
coverage subsystem transaction
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
1 participant
@OndrejGerzicak