-
Notifications
You must be signed in to change notification settings - Fork 3k
WiFi: Re-order Greentea tests with minor fixes #11841
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
Conversation
@AriParkkila, thank you for your changes. |
7c59405
to
bd4adc0
Compare
@AriParkkila Can you amend your commit msg to add there these:
|
Run WIFI-GET-RSSI and WIFI-CONNECT-DISCONNECT-REPEAT tests in secure mode Remove WIFI-CONNECT-PARAMS-VALID-UNSECURE, it's duplicate to WIFI_CONNECT with remains moved to WIFI-CONNECT-SECURE
bd4adc0
to
ad1107e
Compare
@0xc0170 fixed commit msg. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I tested this PR overnight (9 times) with one of our Wifi boards and saw four times less test failures on average.
CI started |
Test run: FAILEDSummary: 1 of 5 test jobs failed Failed test jobs:
|
Test failed because of #11862, we will investigate |
CI restarted |
Test run: SUCCESSSummary: 5 of 5 test jobs passed |
This is sitting on top of changes for #11409 which is currently targeting 6.0 |
Description (required)
WiFi test case changes:
Summary of change (What the change is for and why)
Prefer testing in secure mode as it's much more common.
Documentation (Details of any document updates required)
Pull request type (required)
Test results (required)
Reviewers (optional)
@kivaisan
Release Notes (required for feature/major PRs)
Summary of changes
Impact of changes
Migration actions required