TrueCloudLab/frostfs-testcases
25
0
Fork 21
Code Issues 2 Pull requests 2 Releases Activity Actions 7

Fix assert that checks presence of node in netmap

Browse source 
We should be looking for node host rather than for node name that we assigned in tests code.

Signed-off-by: Vladimir Domnich <v.domnich@yadro.com>
This commit is contained in:
Vladimir Domnich 2022-07-21 16:25:17 +04:00
parent 568b4421ce
commit 8afba7fca6
1 changed files with 5 additions and 5 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

8
pytest_tests/testsuites/network/test_node_management.py
View file

@ -80,8 +80,11 @@ def test_nodes_management(prepare_tmp_dir):
""" """
random_node = choice(list(NEOFS_NETMAP_DICT)) random_node = choice(list(NEOFS_NETMAP_DICT))
alive_node = choice([node for node in NEOFS_NETMAP_DICT if node != random_node]) alive_node = choice([node for node in NEOFS_NETMAP_DICT if node != random_node])
with allure.step('Check node {random_node} is in netmap'):
random_node_host = NEOFS_NETMAP_DICT[random_node]['rpc'].split(':')[0]
snapshot = get_netmap_snapshot(node_name=alive_node) snapshot = get_netmap_snapshot(node_name=alive_node)
assert random_node in snapshot, f'Expected node {random_node} in netmap' assert random_node_host in snapshot, f'Expected node {random_node} in netmap'
with allure.step('Run health check for all storage nodes'): with allure.step('Run health check for all storage nodes'):
for node_name in NEOFS_NETMAP_DICT.keys(): for node_name in NEOFS_NETMAP_DICT.keys():
@ -242,9 +245,6 @@ def test_drop_object(prepare_wallet_and_deposit):
@pytest.mark.skip(reason='Need to clarify scenario') @pytest.mark.skip(reason='Need to clarify scenario')
@allure.title('Control Operations with storage nodes') @allure.title('Control Operations with storage nodes')
def test_shards(prepare_wallet_and_deposit, crate_container_and_pick_node): def test_shards(prepare_wallet_and_deposit, crate_container_and_pick_node):
"""
This test checks base control operations with storage nodes (healthcheck, netmap-snapshot, set-status).
"""
wallet = prepare_wallet_and_deposit wallet = prepare_wallet_and_deposit
file_path = generate_file() file_path = generate_file()