test: make the suite work against a stack on another machine - #207
Open
mdozhdev wants to merge 3 commits into
Open
test: make the suite work against a stack on another machine#207mdozhdev wants to merge 3 commits into
mdozhdev wants to merge 3 commits into
Conversation
processArguments in the session capabilities apply only to the first launch. reinstallApp and the other relaunch helpers call driver.activateApp, which starts the app with no environment, so E2E_LOCAL_HOST is lost and Env.swift falls back to the Info.plist value fixed at build time. Against a stack on another machine that means the app looks for Electrum on the simulator itself and never produces a balance, so completeOnboarding times out waiting for TotalBalance-primary. Every spec reinstalls in a before hook, so it affects all of them. Relaunch through `mobile: launchApp`, which does take an environment. Guarded on iOS and on the variable being set, so nothing changes for Android or for runs against a local stack. Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>
Session creation intermittently times out waiting for WebDriverAgent, and logLevel warn hides whether it is building, launching or failing to connect. Route the appium server log to artifacts and let the level and the WDA timeouts be raised per run. Defaults are unchanged. Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>
LND issues its own cert on first start with SANs for 127.0.0.1, ::1 and its container address. Both gRPC and REST verify the hostname, so a suite running on another machine is rejected: ERR_TLS_CERT_ALTNAME_INVALID: IP 100.116.153.66 is not in the cert list: 127.0.0.1, ::1, 172.18.0.4 tlsextraip adds the address LND is actually reached on, reusing the variable externalip already takes. Defaults to 127.0.0.1, which is already covered, so a local stack is unaffected. Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>
mdozhdev
marked this pull request as ready for review
August 26, 2026 14:41
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
#206 made the backend host configurable. Running the suite that way surfaced three more places assuming localhost. All keep today's defaults, so existing CI is unaffected.
Relaunch loses the backend host.
processArgumentsapplies only to a session's first launch, soreinstallApp()droppedE2E_LOCAL_HOSTand the app fell back to theInfo.plistvalue fixed at build time — onboarding then timed out onTotalBalance-primary. Relaunch viamobile: launchApp, which takes an environment. Guarded on iOS and on the variable being set; nine call sites.LND's cert misses its reachable address. It self-issues with SANs for
127.0.0.1,::1and its container IP, and both gRPC and REST verify the hostname.--tlsextraipadds the real address, reusing the variable--externalipalready takes.WDA timeouts and logs. WDA compiles on cold hosted runners and intermittently blew the 5-minute launch timeout, while
logLevel: warnhid the Appium log. Both now overridable; log written toartifacts/.Verified: full
@lightningspec green on hosted runners with the stack on a second runner, 39 min end to end.tsc/eslintfindings unchanged.Draft until the companion
bitkit-iosworkflow is ready — it needs these onmainfirst. Not specific to #205; #204 would need them too.Related: synonymdev/pubky-stack#275, #206, #205