Skip to content

Commit 8847193

Browse files
committed
Fix tests
1 parent 0eae036 commit 8847193

2 files changed

Lines changed: 10 additions & 10 deletions

File tree

tests/conftest.py

Lines changed: 7 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -19,9 +19,9 @@
1919

2020
import json
2121
import os
22-
import tempfile
2322
import pathlib
2423
import shutil
24+
import tempfile
2525
from contextlib import contextmanager
2626

2727
import docker
@@ -34,18 +34,17 @@
3434
GLOBAL_SKALE_CONF_FILEPATH,
3535
GLOBAL_SKALE_DIR,
3636
META_FILEPATH,
37+
NGINX_CONFIG_FILEPATH,
3738
NGINX_CONTAINER_NAME,
39+
REDIS_URI,
3840
REMOVED_CONTAINERS_FOLDER_PATH,
3941
SCHAIN_NODE_DATA_PATH,
40-
NGINX_CONFIG_FILEPATH,
41-
REDIS_URI,
4242
)
4343
from node_cli.configs.node_options import NODE_OPTIONS_FILEPATH
44-
from node_cli.configs.ssl import SSL_FOLDER_PATH
4544
from node_cli.configs.resource_allocation import RESOURCE_ALLOCATION_FILEPATH
45+
from node_cli.configs.ssl import SSL_FOLDER_PATH
4646
from node_cli.utils.docker_utils import docker_client
4747
from node_cli.utils.global_config import generate_g_config_file
48-
4948
from tests.helper import TEST_META_V1, TEST_META_V2, TEST_META_V3, TEST_SCHAINS_MNT_DIR_SINGLE_CHAIN
5049

5150

@@ -385,8 +384,9 @@ def sync_user_conf(tmp_path):
385384
yield test_env_path
386385
finally:
387386
test_env_path.unlink()
388-
389-
387+
388+
389+
@pytest.fixture
390390
def redis_client():
391391
cpool = redis.ConnectionPool.from_url(REDIS_URI)
392392
return redis.Redis(connection_pool=cpool)

tests/mirage/mirage_node_test.py

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -100,8 +100,8 @@ def test_update_mirage_boot(
100100
mock_is_alive.assert_called_once_with(node_type=NodeType.MIRAGE, is_mirage_boot=True)
101101

102102

103-
@mock.patch('node_cli.core.mirage_node.update_mirage_op')
104-
@mock.patch('node_cli.core.mirage_node.compose_node_env')
103+
@mock.patch('node_cli.mirage.mirage_node.update_mirage_op')
104+
@mock.patch('node_cli.mirage.mirage_node.compose_node_env')
105105
@mock.patch('node_cli.utils.decorators.is_user_valid', return_value=True)
106106
def test_migrate_from_boot(
107107
mock_is_user_valid,
@@ -125,7 +125,7 @@ def test_migrate_from_boot(
125125
node_type=NodeType.MIRAGE,
126126
)
127127
mock_migrate_op.assert_called_once_with(
128-
valid_env_file, mock_env, update_type=MirageUpdateType.INFRA_ONLY
128+
valid_env_file, mock_env, update_type=MirageUpdateType.FROM_BOOT
129129
)
130130

131131

0 commit comments

Comments
 (0)