Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Extend the wwan connection tests to support multiple cycle connection tests (new) #1534

Draft
wants to merge 2 commits into
base: main
Choose a base branch
from

Conversation

stanley31huang
Copy link
Collaborator

Description

Extend the wwan connection tests to support multiple cycle connection tests, we found an issue that the LTE connection could not be established at second round.

As discussed internally with the QA team, we have all agreed that the default number of test runs for WWAN connections tests is 2.

Resolved issues

Here is the support ticket: 00379830.

Our SWE has identified this issue is related to the modem firmware and modem-manager snap, and they are trying to add a patch to modem-manager snap as a workaround.

Documentation

N/A

Tests

list bootstrapped results (with hacking resource):

(venv) stanley@stanley-ThinkPad-T490:~/Desktop/Git_Repos/checkbox-github/providers/base$ checkbox.checkbox-cli list-bootstrapped com.canonical.certification::wwan-automated
$PROVIDERPATH is defined, so following provider sources are ignored ['/home/stanley/.local/share/plainbox-providers-1', '/var/tmp/checkbox-providers-develop'] 
Using sideloaded provider: checkbox-provider-base, version 4.2.0.dev99 from /var/tmp/checkbox-providers/base
Skipped file: /var/tmp/checkbox-providers/base/bin/__pycache__/wwan_tests.cpython-38.pyc
Skipped file: /var/tmp/checkbox-providers/base/units/stress/suspend_cycles_reboot.md
com.canonical.certification::package
com.canonical.plainbox::manifest
com.canonical.certification::snap
com.canonical.certification::wwan/detect
com.canonical.certification::wwan/check-sim-present-honghai-modelc-testid-auto
com.canonical.certification::wwan/3gpp-scan-honghai-modelc-testid-auto
com.canonical.certification::wwan/gsm-connection-honghai-modelc-testid-auto
com.canonical.certification::wwan/gsm-reconnection-honghai-modelc-testid-auto
com.canonical.certification::sleep
com.canonical.certification::rtc
com.canonical.certification::suspend/suspend_advanced_auto
com.canonical.certification::after-suspend-wwan/3gpp-scan-honghai-modelc-testid-auto

add wwan reconnection test
Copy link

codecov bot commented Oct 8, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 47.76%. Comparing base (cd772da) to head (ab12ee7).
Report is 3 commits behind head on main.

Additional details and impacted files
@@            Coverage Diff             @@
##             main    #1534      +/-   ##
==========================================
+ Coverage   47.72%   47.76%   +0.04%     
==========================================
  Files         370      370              
  Lines       39739    39738       -1     
  Branches     6719     6720       +1     
==========================================
+ Hits        18965    18982      +17     
+ Misses      20059    20041      -18     
  Partials      715      715              
Flag Coverage Δ
provider-base 24.77% <100.00%> (+0.10%) ⬆️

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

fix pep8 error
@stanley31huang stanley31huang changed the title add wwan reconnection test Extend the wwan connection tests to support multiple cycle connection tests (New) Oct 8, 2024
@stanley31huang stanley31huang changed the title Extend the wwan connection tests to support multiple cycle connection tests (New) Extend the wwan connection tests to support multiple cycle connection tests (new) Oct 8, 2024
@stanley31huang stanley31huang changed the title Extend the wwan connection tests to support multiple cycle connection tests (new) Extend the wwan connection tests to support multiple cycle connection tests (new) Oct 8, 2024
@stanley31huang stanley31huang requested a review from pieqq October 8, 2024 08:34
@stanley31huang
Copy link
Collaborator Author

The PR is ready, but I did not test it on real machine.

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.

1 participant