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

INDY-1148 : change test for validation nodes txns #560

Merged
merged 8 commits into from
Mar 6, 2018

Conversation

Toktar
Copy link
Contributor

@Toktar Toktar commented Mar 6, 2018

-deleted test_node_adding.py
-test test_add_node_with_not_unique_alias migrate to indy sdk
-test test_add_node_with_not_unique_alias move to test_nodes_with_pool_txns.py

… - add dynamic validation of alias and test

Signed-off-by: toktar <renata.toktar@dsr-corporation.com>
… - put alias validation into PoolRequestHandler.isNodeDataConflicting

Signed-off-by: toktar <renata.toktar@dsr-corporation.com>
… - change validation in pool_req_handler.

Signed-off-by: toktar <renata.toktar@dsr-corporation.com>
-deleted test_node_adding.py
-test test_add_node_with_not_unique_alias
 move to test_nodes_with_pool_txns.py

Signed-off-by: toktar <renata.toktar@dsr-corporation.com>
@ghost
Copy link

ghost commented Mar 6, 2018

Could one of the committers please verify this patch?

-move test test_add_node_with_not_unique_alias
 to the end of test_nodes_with_pool_txns.py

Signed-off-by: toktar <renata.toktar@dsr-corporation.com>
@ashcherbakov ashcherbakov merged commit 24e74d3 into hyperledger:master Mar 6, 2018
lampkin-diet pushed a commit to lampkin-diet/indy-plenum that referenced this pull request Apr 9, 2018
* INDY-1148 - It's possible to create several nodes with the same alias - add dynamic validation of alias and test

Signed-off-by: toktar <renata.toktar@dsr-corporation.com>

* INDY-1148 - It's possible to create several nodes with the same alias - put alias validation into PoolRequestHandler.isNodeDataConflicting

Signed-off-by: toktar <renata.toktar@dsr-corporation.com>

* INDY-1148 - It's possible to create several nodes with the same alias - change validation in pool_req_handler.

Signed-off-by: toktar <renata.toktar@dsr-corporation.com>

* INDY-1148 : change test for validation nodes

-deleted test_node_adding.py
-test test_add_node_with_not_unique_alias
 move to test_nodes_with_pool_txns.py

Signed-off-by: toktar <renata.toktar@dsr-corporation.com>

* INDY-1148 : move test for validate node alias

-move test test_add_node_with_not_unique_alias
 to the end of test_nodes_with_pool_txns.py

Signed-off-by: toktar <renata.toktar@dsr-corporation.com>
lampkin-diet pushed a commit to lampkin-diet/indy-plenum that referenced this pull request Apr 10, 2018
* INDY-1148 - It's possible to create several nodes with the same alias - add dynamic validation of alias and test

Signed-off-by: toktar <renata.toktar@dsr-corporation.com>

* INDY-1148 - It's possible to create several nodes with the same alias - put alias validation into PoolRequestHandler.isNodeDataConflicting

Signed-off-by: toktar <renata.toktar@dsr-corporation.com>

* INDY-1148 - It's possible to create several nodes with the same alias - change validation in pool_req_handler.

Signed-off-by: toktar <renata.toktar@dsr-corporation.com>

* INDY-1148 : change test for validation nodes

-deleted test_node_adding.py
-test test_add_node_with_not_unique_alias
 move to test_nodes_with_pool_txns.py

Signed-off-by: toktar <renata.toktar@dsr-corporation.com>

* INDY-1148 : move test for validate node alias

-move test test_add_node_with_not_unique_alias
 to the end of test_nodes_with_pool_txns.py

Signed-off-by: toktar <renata.toktar@dsr-corporation.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants