-
Notifications
You must be signed in to change notification settings - Fork 366
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
[INDY-2140] Fix tests in view_change directory #1366
[INDY-2140] Fix tests in view_change directory #1366
Conversation
…/indy-2223-integrate Signed-off-by: Andrew Nikitin <andrew.nikitin@dsr-corporation.com>
Signed-off-by: Andrew Nikitin <andrew.nikitin@dsr-corporation.com>
…tegrate Signed-off-by: Andrew Nikitin <andrew.nikitin@dsr-corporation.com>
…/indy-2223-integrate Signed-off-by: Andrew Nikitin <andrew.nikitin@dsr-corporation.com>
Signed-off-by: Andrew Nikitin <andrew.nikitin@dsr-corporation.com>
Signed-off-by: Andrew Nikitin <andrew.nikitin@dsr-corporation.com>
@@ -292,6 +292,7 @@ def _finish_view_change(self): | |||
view_changes=self._new_view.viewChanges, | |||
checkpoint=self._new_view.checkpoint, | |||
batches=self._new_view.batches)) | |||
self._data.last_completed_view_no = self._new_view.viewNo |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Why reference self._new_view
instead of self._data.view_no
? Can they be different?
plenum/test/view_change/test_disconnected_node_reconnects_after_view_change.py
Outdated
Show resolved
Hide resolved
…tegrate Signed-off-by: Andrew Nikitin <andrew.nikitin@dsr-corporation.com>
Signed-off-by: Andrew Nikitin <andrew.nikitin@dsr-corporation.com>
@@ -292,6 +292,7 @@ def _finish_view_change(self): | |||
view_changes=self._new_view.viewChanges, | |||
checkpoint=self._new_view.checkpoint, | |||
batches=self._new_view.batches)) | |||
self._data.last_completed_view_no = self._data.view_no |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Actually I wasn't insisting on change, just asking :)
@@ -20,7 +20,7 @@ def tconf(tconf): | |||
yield tconf | |||
|
|||
|
|||
@pytest.mark.skip(reason="INDY-2223: Temporary skipped to create build") | |||
@pytest.mark.skip(reason="All commits before vc will be cleared and we don't order during vc") |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I think this test still makes sense ans should pass with the current code. Commits coming during view change will be discarded, but we will do re-ordering in new view, and all batches we have prepared certificates for will be ordered eventually
No description provided.