Skip to content

Ookla Speedtest Android automation to work with lf_interop_speedtest#266

Open
AbhishekCandela wants to merge 2 commits into
greearb:masterfrom
goyalsaurabh06:speedtest_core
Open

Ookla Speedtest Android automation to work with lf_interop_speedtest#266
AbhishekCandela wants to merge 2 commits into
greearb:masterfrom
goyalsaurabh06:speedtest_core

Conversation

@AbhishekCandela
Copy link
Copy Markdown
Collaborator

Details

  • Launches the Ookla Speedtest Android app using ADB
  • Taps the Go button based on device screen size
  • Dumps and parses Android UI XML to collect speed test results
  • Extracts download speed, upload speed, idle latency, download latency, and upload latency
  • Supports multiple connected Android devices using concurrent execution
  • Adds optional POST reporting for collected speed test metrics
  • Handles common Ookla error popups and retries result parsing when data is not ready

Testing

  • Supports execution with:
    • --type adb
    • --type ookla
    • --type cli
  • ADB mode can target all connected devices or a comma-separated list through --adb_devices
  • python3 ookla.py --type adb --adb_devices <DEVICE_SERIAL>

…speedtest

Signed-off-by: AbhishekCandela <rachakonda.abhishek@candelatech.com>
Copy link
Copy Markdown
Collaborator

@smileyrekiere smileyrekiere left a comment

Choose a reason for hiding this comment

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

curious the Ookla Speedtest script uses prints instead of the logger functionality, also the help is a bit light. Here are the guide lines for the header information
https://candelatech.atlassian.net/wiki/spaces/LANFORGE/history/938934277/Script+Expectations+Code+Style+Docs
Look for Template Script Header, also please include a --help_summary

If this script is an exception of some sort , please advice
44c11c3#top
11:38 AM

There is an example in the PR yet not in the script.

Copy link
Copy Markdown
Collaborator

@smileyrekiere smileyrekiere left a comment

Choose a reason for hiding this comment

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

This is a core module

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants