Skip to content

Add wwan test for lakeport (New) - #2709

Open
rickwu666666 wants to merge 18 commits into
mainfrom
add-wwan-test-for-lakeport
Open

Add wwan test for lakeport (New)#2709
rickwu666666 wants to merge 18 commits into
mainfrom
add-wwan-test-for-lakeport

Conversation

@rickwu666666

Copy link
Copy Markdown
Contributor

Description

Add wwan test jobs for the wwan module control by AT command. We have a project that is not able to make connection by upstream's modem-manager. Therefore, we add test job for testing wwan connection via AT commands.

Resolved issues

Documentation

Tests

https://certification.canonical.com/hardware/202604-38611/submission/500910/

Since we have WWAN SImodule M7672G-LNGV which is not able to make the connection via modem-manager, and have no plan to upstream the change. Therefore, we need to control it via AT command
Add recovery job
Copilot AI review requested due to automatic review settings July 20, 2026 08:50
@rickwu666666 rickwu666666 added the ce-oem The PR only contains to the ce-oem provider under the contrib area label Jul 20, 2026

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Pull request overview

This PR extends the contrib CE-OEM provider with an automated WWAN connectivity and recovery test flow that drives modem bring-up via AT commands through ModemManager (mmcli --command), intended for platforms where ModemManager’s higher-level connection logic is insufficient.

Changes:

  • Add a new CE-OEM WWAN automated test plan and wire it into the CE-OEM umbrella test plan.
  • Introduce templated WWAN jobs gated by new manifest entries (AT-command control + optional reset/recovery).
  • Add the AT-command runner script, its JSON-driven module step configuration, documentation, and unit tests.

Reviewed changes

Copilot reviewed 7 out of 8 changed files in this pull request and generated 1 comment.

Show a summary per file
File Description
contrib/checkbox-ce-oem/checkbox-provider-ce-oem/units/wwan/test-plan.pxu Adds a CE-OEM WWAN automated test plan including the new AT-command jobs.
contrib/checkbox-ce-oem/checkbox-provider-ce-oem/units/wwan/manifest.pxu Introduces manifest entries to gate AT-command WWAN testing and optional reset/recovery.
contrib/checkbox-ce-oem/checkbox-provider-ce-oem/units/wwan/jobs.pxu Adds templated WWAN connection + reset/recovery jobs that call the new script.
contrib/checkbox-ce-oem/checkbox-provider-ce-oem/units/test-plan-ce-oem.pxu Wires the new WWAN automated plan into the overall CE-OEM plan.
contrib/checkbox-ce-oem/checkbox-provider-ce-oem/tests/test_wwan_connection_at_command.py Adds unit test coverage for the new AT-command WWAN script helpers and flows.
contrib/checkbox-ce-oem/checkbox-provider-ce-oem/data/wwan_at_command.json Provides the initial module-specific AT-step sequence configuration (SIM7672G-LNGV).
contrib/checkbox-ce-oem/checkbox-provider-ce-oem/data/README-wwan_at_command.md Documents the JSON schema and recommended step ordering/semantics.
contrib/checkbox-ce-oem/checkbox-provider-ce-oem/bin/wwan_connection_at_command.py Implements the AT-command driven connection test and reset/recovery flow.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread contrib/checkbox-ce-oem/checkbox-provider-ce-oem/data/README-wwan_at_command.md Outdated
Comment thread contrib/checkbox-ce-oem/checkbox-provider-ce-oem/units/wwan/manifest.pxu Outdated

@baconYao baconYao left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

See my inline comments

@rickwu666666
rickwu666666 requested a review from baconYao July 22, 2026 05:01
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

ce-oem The PR only contains to the ce-oem provider under the contrib area

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants