Skip to content

Commit 2b96e3d

Browse files
committed
update SSH testing: improve CI environment compatibility (#18)
* update SSH testing: improve CI environment compatibility with mock fallback, enhance macOS SSH configuration, and add robust connection retries * update tests and workflows: make SSH username dynamic in tests and add concurrency control to GitHub Actions * update workflows: simplify device serial description in bench workflow inputs * update README: revise supported platforms table with expanded host-device compatibility details * update README: revise supported platforms table with expanded host-device compatibility details
1 parent 71c2c91 commit 2b96e3d

3 files changed

Lines changed: 15 additions & 7 deletions

File tree

β€Ž.github/workflows/bench.ymlβ€Ž

Lines changed: 6 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -8,9 +8,14 @@ on:
88
workflow_dispatch:
99
inputs:
1010
device_serial:
11-
description: 'Mobile device serial'
11+
description: 'Device serial'
1212
required: false
1313

14+
# Cancel in-progress runs when a new commit is pushed to the same PR or branch
15+
concurrency:
16+
group: ${{ github.workflow }}-${{ github.event.pull_request.number || github.ref }}
17+
cancel-in-progress: true
18+
1419
jobs:
1520
ci-matrix:
1621
strategy:

β€ŽREADME.mdβ€Ž

Lines changed: 8 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -49,11 +49,14 @@ cat experiments/results/*.csv
4949

5050
## πŸ”§ Supported Platforms
5151

52-
| Platform | Architecture | Transport | Status |
53-
|----------|-------------|-----------|--------|
54-
| Android | ARM64 (arm64-v8a) | ADB (adbutils) | βœ… Stable |
55-
| Linux | ARM64/ARM32 | SSH (paramiko) | βœ… Stable |
56-
| iOS | ARM64 | USB | 🚧 Planned |
52+
| Host OS | Host Arch | Device OS | Device Arch | Transport | Library | Status |
53+
|---------|--------------|-----------|-------------|-----------|-----------|------------|
54+
| Linux | x86_64 | Android | ARM64 | ADB | adbutils | βœ… Stable |
55+
| macOS | x86_64/ARM64 | Android | ARM64 | ADB | adbutils | βœ… Stable |
56+
| Windows | x86_64 | Android | ARM64 | ADB | adbutils | βœ… Stable |
57+
| Linux | x86_64 | Linux | ARM64/ARM32 | SSH | paramiko | βœ… Stable |
58+
| macOS | x86_64/ARM64 | Linux | ARM64/ARM32 | SSH | paramiko | βœ… Stable |
59+
| Any | Any | iOS | ARM64 | USB | - | 🚧 Planned |
5760

5861
## πŸ“‹ Requirements
5962

β€Žtests/test_generate_ssh_config.pyβ€Ž

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -82,7 +82,7 @@ def test_generate_ssh_test_script(self):
8282
content = output_file.read_text()
8383
assert "#!/usr/bin/env python3" in content
8484
assert "LinuxSSHDevice" in content
85-
assert 'username="testuser"' in content
85+
assert 'username = os.environ.get("USER"' in content # Dynamic username
8686
assert "test_ssh_device()" in content
8787

8888
def test_generate_ssh_setup_script(self):

0 commit comments

Comments
Β (0)