s.makhov/frostfs-testcases
1
0
Fork 0
forked from TrueCloudLab/frostfs-testcases
Code Pull requests Activity

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.
This commit is contained in:
Vladimir Domnich 2022-07-21 16:25:17 +04:00
parent 6d23a5e6e9
commit e4a3f45a10
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))
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)
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'):
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')
@allure.title('Control Operations with storage nodes')
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
file_path = generate_file()