Skip to content

Conversation

@platform-sa
Copy link
Collaborator

Update game bridge (build from ts-immutable-sdk version 2.12.6)

@platform-sa platform-sa requested a review from a team as a code owner February 4, 2026 01:45
@rodrigo-fournier-immutable rodrigo-fournier-immutable force-pushed the chore/update-game-bridge-2.12.6 branch from 92a1033 to b6ba396 Compare February 4, 2026 02:50
time.sleep(5)
print("Logged out")

self.altdriver.stop()
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Keep logout and wait for authenticated screen so that we test logout too

@rodrigo-fournier-immutable rodrigo-fournier-immutable force-pushed the chore/update-game-bridge-2.12.6 branch from b6ba396 to 7ea6a56 Compare February 4, 2026 03:40
@rodrigo-fournier-immutable rodrigo-fournier-immutable force-pushed the chore/update-game-bridge-2.12.6 branch 4 times, most recently from 08a7b92 to 7e7bc50 Compare February 4, 2026 05:57
…bility

Add controlled browser logout for macOS to handle deep-link dispatch
issues introduced by Chrome 137 code-splitting changes.

Changes:
- Add logout_with_controlled_browser() in test_mac_helpers.py
- Add get_logout_url_from_unity_logs() to monitor Unity logs
- Add get_product_name() to extract product name from ProjectSettings
- Update logout() method to use controlled browser approach
- Manually trigger immutablerunner://logout deep-link via 'open' command
- Reduce Windows login timeout from 90s to 30s

This matches the Windows implementation and resolves logout timeouts
caused by Chrome 137's increased code-splitting affecting protocol
handler reliability.
@rodrigo-fournier-immutable rodrigo-fournier-immutable force-pushed the chore/update-game-bridge-2.12.6 branch from 7e7bc50 to dc755d5 Compare February 4, 2026 06:15
Windows:
- Extract returnTo deep-link from logout URL
- Write deep-link to Windows Registry for Unity to process
- Fixes app crash by ensuring logout callback is received

macOS:
- Kill existing Brave processes before launching with debugging port
- Prevents 'cannot connect to chrome at localhost:9222' error
- Ensures controlled logout can connect to browser

Both platforms now properly dispatch immutablerunner://logout deep-link
back to Unity after browser logout completes.
@rodrigo-fournier-immutable rodrigo-fournier-immutable force-pushed the chore/update-game-bridge-2.12.6 branch from 6041098 to 1b391d9 Compare February 4, 2026 07:23
Extract tab closing logic into reusable methods:
- Windows: close_all_browser_tabs() in test_windows_helpers.py
- macOS: close_all_browser_tabs() method in MacTest class

Call in setUpClass to ensure clean browser state before test suite runs.
This prevents leftover tabs from previous runs from interfering with tests.

Also refactor stop_browser() to use the new tab closing methods.
…) and tab cleanup (macOS)

Windows:
- Add logic to click 'click here' link on logout page
- Tries multiple selectors (partial link text, link text, xpath)
- Falls back to registry deep-link dispatch if link not found

macOS:
- Call close_all_browser_tabs() in logout() after launch_browser()
- Ensures clean browser state before logout interaction
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

4 participants