Skip to content

Commit 78e9797

Browse files
Add SMASH service support for testing
1 parent 8b0442e commit 78e9797

File tree

11 files changed

+523
-3
lines changed

11 files changed

+523
-3
lines changed
Lines changed: 16 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,16 @@
1+
#!/usr/bin/env bash
2+
3+
set -uo pipefail
4+
5+
SOCKET_PATH="$(readlink -m "$CARDANO_NODE_SOCKET_PATH")"
6+
STATE_CLUSTER="${SOCKET_PATH%/*}"
7+
STATE_CLUSTER_NAME="${STATE_CLUSTER##*/}"
8+
9+
export PGPASSFILE="$STATE_CLUSTER/pgpass"
10+
export SMASH_ADMIN="${SMASH_ADMIN:-admin}"
11+
export SMASH_PASSWORD="${SMASH_PASSWORD:-password}"
12+
export SMASH_ADMINS_FILE="$STATE_CLUSTER/admins.txt"
13+
14+
echo "${SMASH_ADMIN}, ${SMASH_PASSWORD}" > "$SMASH_ADMINS_FILE"
15+
16+
exec "$DBSYNC_REPO/smash-server/bin/cardano-smash-server" --config "./$STATE_CLUSTER_NAME/dbsync-config.yaml" --port %%SMASH_PORT%% --admins "$SMASH_ADMINS_FILE"

cardano_node_tests/cluster_scripts/conway_fast/start-cluster

Lines changed: 24 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -2,6 +2,7 @@
22

33
# controlling environment variables:
44
# DBSYNC_REPO - will start and configure db-sync if the value is path to db-sync repository
5+
# SMASH - if set, will start and configure smash
56
# ENABLE_LEGACY - if set, local cluster will use legacy networking
67
# MIXED_P2P - if set, local cluster will use P2P for some nodes and legacy topology for others
78
# UTXO_BACKEND - 'mem' or 'disk', default is 'mem' (or legacy) if unset
@@ -206,6 +207,23 @@ startsecs=5
206207
EoF
207208
fi
208209

210+
# enable smash service
211+
if [ -n "${DBSYNC_REPO:-""}" ] && [ -n "${SMASH:-""}" ]; then
212+
[ -e "${DBSYNC_REPO}/smash-server/bin/cardano-smash-server" ] || \
213+
{ echo "The \`${DBSYNC_REPO}/smash-server/bin/cardano-smash-server\` not found, line $LINENO" >&2; exit 1; } # assert
214+
215+
cat >> "${STATE_CLUSTER}/supervisor.conf" <<EoF
216+
217+
[program:smash]
218+
command=${SCRIPT_DIR}/run-cardano-smash
219+
stderr_logfile=./${STATE_CLUSTER_NAME}/smash.stderr
220+
stdout_logfile=./${STATE_CLUSTER_NAME}/smash.stdout
221+
autostart=false
222+
autorestart=false
223+
startsecs=5
224+
EoF
225+
fi
226+
209227
# enable cardano-submit-api service
210228
if [ "$ENABLE_SUBMIT_API" -eq 1 ]; then
211229
cat >> "${STATE_CLUSTER}/supervisor.conf" <<EoF
@@ -625,6 +643,12 @@ if [ -n "${DBSYNC_REPO:-""}" ]; then
625643
supervisorctl -s "unix:///${SUPERVISORD_SOCKET_PATH}" start dbsync
626644
fi
627645

646+
# start smash
647+
if [ -n "${DBSYNC_REPO:-""}" ] && [ -n "${SMASH:-""}" ]; then
648+
echo "Starting smash"
649+
supervisorctl -s "unix:///${SUPERVISORD_SOCKET_PATH}" start smash
650+
fi
651+
628652
echo "Sleeping for initial Tx submission delay of $TX_SUBMISSION_DELAY seconds"
629653
sleep "$TX_SUBMISSION_DELAY"
630654

cardano_node_tests/tests/conftest.py

Lines changed: 7 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -119,6 +119,13 @@ def pytest_configure(config: tp.Any) -> None:
119119
config.stash[metadata_key]["db-sync ghc"] = VERSIONS.dbsync_ghc
120120
config.stash[metadata_key]["db-sync exe"] = str(configuration.DBSYNC_BIN)
121121

122+
config.stash[metadata_key]["HAS_SMASH"] = str(configuration.HAS_SMASH)
123+
if configuration.HAS_SMASH:
124+
config.stash[metadata_key]["smash"] = str(VERSIONS.smash)
125+
config.stash[metadata_key]["smash rev"] = VERSIONS.smash_git_rev
126+
config.stash[metadata_key]["smash ghc"] = VERSIONS.smash_ghc
127+
config.stash[metadata_key]["smash exe"] = str(configuration.SMASH_BIN)
128+
122129
if "nix/store" not in config.stash[metadata_key]["cardano-cli exe"]:
123130
LOGGER.warning(" WARNING: Using `cardano-cli` from custom path!")
124131
if "nix/store" not in config.stash[metadata_key]["cardano-node exe"]:

cardano_node_tests/tests/test_pools.py

Lines changed: 7 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -28,6 +28,7 @@
2828
from cardano_node_tests.utils import dbsync_utils
2929
from cardano_node_tests.utils import helpers
3030
from cardano_node_tests.utils import locking
31+
from cardano_node_tests.utils import smash_utils
3132
from cardano_node_tests.utils import temptools
3233
from cardano_node_tests.utils import tx_view
3334
from cardano_node_tests.utils import web
@@ -696,6 +697,7 @@ def _query_func():
696697
@pytest.mark.testnets
697698
@pytest.mark.smoke
698699
@pytest.mark.dbsync
700+
@pytest.mark.smash
699701
def test_stake_pool_not_avail_metadata(
700702
self,
701703
cluster_manager: cluster_management.ClusterManager,
@@ -720,14 +722,15 @@ def test_stake_pool_not_avail_metadata(
720722
out_file=f"{pool_name}_registration_metadata.json", content=pool_metadata
721723
)
722724
pool_metadata_url = "https://www.where_metadata_file_is_located.com"
725+
pool_metadata_hash = cluster.g_stake_pool.gen_pool_metadata_hash(pool_metadata_file)
723726

724727
pool_data = clusterlib.PoolData(
725728
pool_name=pool_name,
726729
pool_pledge=1_000,
727730
pool_cost=cluster.g_query.get_protocol_params().get("minPoolCost", 500),
728731
pool_margin=0.2,
729732
pool_metadata_url=pool_metadata_url,
730-
pool_metadata_hash=cluster.g_stake_pool.gen_pool_metadata_hash(pool_metadata_file),
733+
pool_metadata_hash=pool_metadata_hash,
731734
)
732735

733736
# Create pool owners
@@ -766,6 +769,7 @@ def _query_func():
766769
)
767770

768771
dbsync_utils.retry_query(query_func=_query_func, timeout=360)
772+
smash_utils.check_smash_pool_errors(pool_creation_out.stake_pool_id, pool_metadata_hash)
769773

770774
@allure.link(helpers.get_vcs_link())
771775
@common.PARAM_USE_BUILD_CMD
@@ -820,6 +824,7 @@ def test_create_stake_pool(
820824
@common.PARAM_USE_BUILD_CMD
821825
@pytest.mark.testnets
822826
@pytest.mark.dbsync
827+
@pytest.mark.smash
823828
def test_deregister_stake_pool(
824829
self,
825830
cluster_manager: cluster_management.ClusterManager,
@@ -944,6 +949,7 @@ def test_deregister_stake_pool(
944949
dbsync_utils.check_pool_deregistration(
945950
pool_id=pool_creation_out.stake_pool_id, retiring_epoch=depoch
946951
)
952+
smash_utils.check_smash_pool_retired(pool_id=pool_creation_out.stake_pool_id)
947953

948954
# Check `transaction view` command
949955
tx_view.check_tx_view(cluster_obj=cluster, tx_raw_output=tx_raw_output)
Lines changed: 197 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,197 @@
1+
"""Tests for basic SMASH operations."""
2+
3+
import logging
4+
import random
5+
import re
6+
from http import HTTPStatus
7+
8+
import pytest
9+
import requests
10+
from cardano_clusterlib import clusterlib
11+
12+
from cardano_node_tests.utils import dbsync_queries
13+
from cardano_node_tests.utils import dbsync_types
14+
from cardano_node_tests.utils import dbsync_utils
15+
from cardano_node_tests.utils import helpers
16+
from cardano_node_tests.utils import logfiles
17+
from cardano_node_tests.utils import smash_utils
18+
19+
LOGGER = logging.getLogger(__name__)
20+
21+
22+
def check_request_error(
23+
err: requests.exceptions.RequestException,
24+
expected_status: HTTPStatus,
25+
expected_code: str | None,
26+
expected_description: str,
27+
) -> None:
28+
"""Assert expected HTTP errors in requests, handling both JSON and text responses."""
29+
response = err.response
30+
assert response is not None, "No error response"
31+
assert response.status_code == expected_status
32+
33+
try:
34+
error_data = response.json()
35+
actual_code = error_data.get("code")
36+
actual_description = error_data.get("description")
37+
except ValueError:
38+
# If not JSON, treat the entire response as text
39+
actual_code = None
40+
actual_description = response.text.strip()
41+
42+
if expected_code:
43+
assert actual_code == expected_code
44+
45+
assert actual_description == expected_description
46+
47+
48+
@pytest.mark.smash
49+
class TestBasicSmash:
50+
"""Basic tests for SMASH service."""
51+
52+
@pytest.fixture()
53+
def locked_pool(
54+
self,
55+
cluster_lock_pool: tuple[clusterlib.ClusterLib, str],
56+
) -> dbsync_types.PoolDataRecord:
57+
"""Get id of locked pool from cluster_lock_pool fixture."""
58+
cluster_obj, pool_name = cluster_lock_pool
59+
pools_ids = cluster_obj.g_query.get_stake_pools()
60+
locked_pool_number = pool_name.replace("node-pool", "")
61+
pattern = re.compile(r"pool" + re.escape(locked_pool_number) + r"(\D|$)")
62+
pools = [next(dbsync_queries.query_pool_data(p)) for p in pools_ids]
63+
locked_pool = next(p for p in pools if p.metadata_url and pattern.search(p.metadata_url))
64+
locked_pool_data = dbsync_utils.get_pool_data(locked_pool.view)
65+
assert locked_pool_data is not None, "Locked pool data not found!"
66+
return locked_pool_data
67+
68+
@pytest.fixture(scope="session")
69+
def smash(
70+
self,
71+
) -> None | smash_utils.SmashClient:
72+
"""Create SMASH client."""
73+
smash = smash_utils.get_client()
74+
if smash is None:
75+
pytest.skip("SMASH client is not available. Skipping test.")
76+
return smash
77+
78+
def test_fetch_pool_metadata(
79+
self, locked_pool: dbsync_types.PoolDataRecord, smash: smash_utils.SmashClient
80+
):
81+
pool_id = locked_pool.hash
82+
83+
# Offchain metadata is inserted into database few minutes after start of a cluster
84+
def _query_func():
85+
pool_metadata = next(
86+
iter(dbsync_queries.query_off_chain_pool_data(locked_pool.view)), None
87+
)
88+
assert pool_metadata is not None, dbsync_utils.NO_RESPONSE_STR
89+
return pool_metadata
90+
91+
metadata_dbsync = dbsync_utils.retry_query(query_func=_query_func, timeout=360)
92+
93+
expected_metadata = smash_utils.PoolMetadata(
94+
name=metadata_dbsync.json["name"],
95+
description=metadata_dbsync.json["description"],
96+
ticker=metadata_dbsync.ticker_name,
97+
homepage=metadata_dbsync.json["homepage"],
98+
)
99+
actual_metadata = smash.get_pool_metadata(pool_id, metadata_dbsync.hash.hex())
100+
assert expected_metadata == actual_metadata
101+
102+
def test_delist_pool(
103+
self,
104+
locked_pool: dbsync_types.PoolDataRecord,
105+
smash: smash_utils.SmashClient,
106+
request: pytest.FixtureRequest,
107+
worker_id: str,
108+
):
109+
pool_id = locked_pool.hash
110+
111+
# Define and register function that ensures pool is re-enlisted after test completion
112+
def pool_cleanup():
113+
smash.enlist_pool(pool_id)
114+
115+
request.addfinalizer(pool_cleanup)
116+
117+
# Delist the pool
118+
expected_delisted_pool = smash_utils.PoolData(pool_id=pool_id)
119+
actual_delisted_pool = smash.delist_pool(pool_id)
120+
assert expected_delisted_pool == actual_delisted_pool
121+
122+
# Check if fetching metadata for a delisted pool returns an error
123+
try:
124+
smash.get_pool_metadata(pool_id, locked_pool.metadata_hash)
125+
except requests.exceptions.RequestException as err:
126+
check_request_error(err, HTTPStatus.FORBIDDEN, None, f"Pool {pool_id} is delisted")
127+
128+
# Ignore expected errors in logs that would fail test in teardown phase
129+
err_msg = "Delisted pool already exists!"
130+
expected_err_regexes = [err_msg]
131+
logfiles.add_ignore_rule(
132+
files_glob="smash.stdout",
133+
regex="|".join(expected_err_regexes),
134+
ignore_file_id=worker_id,
135+
)
136+
# Ensure re-delisting an already delisted pool returns an error
137+
try:
138+
smash.delist_pool(pool_id)
139+
except requests.exceptions.RequestException as err:
140+
check_request_error(err, HTTPStatus.BAD_REQUEST, "DbInsertError", err_msg)
141+
142+
def test_enlist_pool(
143+
self,
144+
locked_pool: dbsync_types.PoolDataRecord,
145+
smash: smash_utils.SmashClient,
146+
):
147+
pool_id = locked_pool.hash
148+
# Ensure enlisting an already enlisted pool returns an error
149+
try:
150+
smash.enlist_pool(pool_id)
151+
except requests.exceptions.RequestException as err:
152+
check_request_error(
153+
err,
154+
HTTPStatus.NOT_FOUND,
155+
"RecordDoesNotExist",
156+
"The requested record does not exist.",
157+
)
158+
159+
# Delist the pool
160+
smash.delist_pool(pool_id)
161+
try:
162+
smash.get_pool_metadata(pool_id, locked_pool.metadata_hash)
163+
except requests.exceptions.RequestException as err:
164+
check_request_error(err, HTTPStatus.FORBIDDEN, None, f"Pool {pool_id} is delisted")
165+
166+
# Enlist the pool
167+
actual_res_enlist = smash.enlist_pool(pool_id)
168+
expected_res_enlist = smash_utils.PoolData(pool_id=pool_id)
169+
assert expected_res_enlist == actual_res_enlist
170+
171+
def test_reserve_ticker(
172+
self,
173+
cluster: clusterlib.ClusterLib,
174+
smash: smash_utils.SmashClient,
175+
request: pytest.FixtureRequest,
176+
):
177+
pool_id = random.choice(cluster.g_query.get_stake_pools())
178+
179+
# Register cleanup function that removes ticker from database after test completion
180+
request.addfinalizer(dbsync_queries.delete_reserved_pool_tickers)
181+
182+
# Reserve ticker
183+
ticker = helpers.get_rand_str(length=3)
184+
actual_response = smash.reserve_ticker(ticker_name=ticker, pool_hash=pool_id)
185+
expected_response = smash_utils.PoolTicker(name=f"{ticker}")
186+
assert expected_response == actual_response
187+
188+
# Reserve already taken ticker
189+
try:
190+
smash.reserve_ticker(ticker_name=ticker, pool_hash=pool_id)
191+
except requests.exceptions.RequestException as err:
192+
check_request_error(
193+
err,
194+
HTTPStatus.BAD_REQUEST,
195+
"TickerAlreadyReserved",
196+
f'Ticker name "{ticker}" is already reserved',
197+
)

cardano_node_tests/utils/cluster_scripts.py

Lines changed: 7 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -51,6 +51,7 @@ class InstancePorts:
5151
webserver: int
5252
metrics_submit_api: int
5353
submit_api: int
54+
smash: int
5455
supervisor: int
5556
relay1: int
5657
ekg_relay1: int
@@ -189,6 +190,7 @@ def _get_node_ports(num: int) -> NodePorts:
189190
webserver=last_port,
190191
metrics_submit_api=last_port - 1,
191192
submit_api=last_port - 2,
193+
smash=last_port - 3,
192194
supervisor=12001 + instance_num,
193195
# Relay1
194196
relay1=0,
@@ -266,6 +268,8 @@ def _replace_instance_files(
266268
new_content = new_content.replace(
267269
"%%METRICS_SUBMIT_API_PORT%%", str(instance_ports.metrics_submit_api)
268270
)
271+
# Reconfigure smash port
272+
new_content = new_content.replace("%%SMASH_PORT%%", str(instance_ports.smash))
269273
# Reconfigure webserver port
270274
new_content = new_content.replace("%%WEBSERVER_PORT%%", str(instance_ports.webserver))
271275
return new_content
@@ -542,6 +546,7 @@ def get_instance_ports(self, instance_num: int) -> InstancePorts:
542546
webserver=last_port,
543547
metrics_submit_api=last_port - 1,
544548
submit_api=last_port - 2,
549+
smash=last_port - 3,
545550
supervisor=12001 + instance_num,
546551
relay1=relay1_ports.node,
547552
ekg_relay1=relay1_ports.ekg,
@@ -615,6 +620,8 @@ def _reconfigure_testnet(
615620
new_content = new_content.replace(
616621
"%%METRICS_SUBMIT_API_PORT%%", str(instance_ports.metrics_submit_api)
617622
)
623+
# Reconfigure smash port
624+
new_content = new_content.replace("%%SMASH_PORT%%", str(instance_ports.smash))
618625
# Reconfigure EKG metrics port
619626
new_content = new_content.replace("%%EKG_PORT_RELAY1%%", str(instance_ports.ekg_relay1))
620627
# Reconfigure prometheus metrics port

0 commit comments

Comments
 (0)