Skip to content
This repository has been archived by the owner on Apr 17, 2019. It is now read-only.

Fix/mst batches tx processor #1650

Merged
merged 2 commits into from
Aug 27, 2018
Merged

Fix/mst batches tx processor #1650

merged 2 commits into from
Aug 27, 2018

Conversation

muratovv
Copy link
Contributor

Description of the Change

Fix tests for tx processor with batches

Benefits

Regressions on tx processor can be checked

Note for reviewers

After feature/mst_batches will be merged into trunk, trunk should be a base branch for this pull request.

@igor-egorov
Copy link
Contributor

Please check failure of 50 - module_ordering_service_test (Failed) on Linux.

Copy link
Contributor

@igor-egorov igor-egorov left a comment

Choose a reason for hiding this comment

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

LGTM, but we have to update the branch and wait for CI.

@l4l l4l added the tests pr aimed at the code coverage increase or test refactorings label Aug 27, 2018
@muratovv muratovv changed the base branch from feature/mst_batches to dev August 27, 2018 16:28
Signed-off-by: Fedor Muratov <[email protected]>
Add pcs deprecated to propagate transaction

Signed-off-by: Fedor Muratov <[email protected]>
@muratovv muratovv force-pushed the fix/mst_batches_tx_processor branch from d5c885f to 1f7f574 Compare August 27, 2018 17:17
@muratovv muratovv merged commit b801664 into dev Aug 27, 2018
@muratovv muratovv deleted the fix/mst_batches_tx_processor branch August 27, 2018 17:20
nickaleks pushed a commit that referenced this pull request Sep 10, 2018
* Fix rest of test for tx processor
Add pcs deprecated to propagate transaction

Signed-off-by: Fedor Muratov <[email protected]>
bakhtin pushed a commit that referenced this pull request Nov 2, 2018
* Fix rest of test for tx processor
Add pcs deprecated to propagate transaction

Signed-off-by: Fedor Muratov <[email protected]>
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
tests pr aimed at the code coverage increase or test refactorings
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants