Skip to content

Commit ee181ce

Browse files
VijitSingh97claude
andcommitted
test(integration): wait for Tari to resync in the subnet phase (#201)
The moved-subnet phase does a full down->up, restarting Tari, then asserts 'tari synced (required)' via assert_running_state — but unlike the per-scenario deploy path (run_scenario) it never waits for Tari to close its post-restart offline gap. Tari re-peers over Tor slowly, so the assertion catches it at 'loading' and the phase flakes (seen on the v1.6.0 release e2e: 387 passed, the sole failure 'tari synced expected [done] got [loading]'; monerod synced fine). Mirror run_scenario's guard: wait_tari_synced 300 before the battery when dashboard.tari_required is true. Product behaviour is unchanged — Tari always syncs, the test just asserted too early. Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
1 parent c6e2cb0 commit ee181ce

1 file changed

Lines changed: 5 additions & 0 deletions

File tree

tests/integration/run.sh

Lines changed: 5 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -1724,6 +1724,11 @@ run_subnet_scenario() {
17241724
wait_monero_synced 120 || true
17251725
wait_miner_running 180 || true
17261726
wait_hashes_flowing 300 || true
1727+
# The down/up restarted Tari too, so — like the per-scenario deploy path — let it close its
1728+
# post-restart offline gap before assert_running_state's sync check, or we catch it mid-"loading".
1729+
if [ "$(jq_get "$config" '.dashboard.tari_required')" = "true" ]; then
1730+
wait_tari_synced 300 || true
1731+
fi
17271732

17281733
# Subnet-specific live checks + the standard running-state battery on the moved subnet.
17291734
assert_subnet_live "$config"

0 commit comments

Comments
 (0)