Add certification test plans for Ubuntu 26 (New)#2304
Add certification test plans for Ubuntu 26 (New)#2304stanley31huang wants to merge 5 commits intomainfrom
Conversation
- adding ARM vfp tests into generic cpu-cert-automated test plan - create a new media card test plan as some interfaces are no longer avaialable in recent IoT platform
adding IoT cert test plan for server/desktop image
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## main #2304 +/- ##
==========================================
+ Coverage 54.88% 55.01% +0.13%
==========================================
Files 412 413 +1
Lines 44246 44393 +147
Branches 8154 8183 +29
==========================================
+ Hits 24284 24423 +139
+ Misses 19132 19131 -1
- Partials 830 839 +9
Flags with carried forward coverage won't be shown. Click here to find out more. ☔ View full report in Codecov by Sentry. 🚀 New features to boost your workflow:
|
fix test plan check issue
Hook25
left a comment
There was a problem hiding this comment.
Pure grammar/consistency pass for now
- I think you forgot to commit these:
unable to find nested part: com.canonical.certification::after-suspend-camera-automated
unable to find nested part: com.canonical.certification::after-suspend-fingerprint-cert-Automated
- Why are we still using regex inclusions? do we have any "blocker" to using only template ids?
- Why are there quite a few "empty" testplans around?
- Why is the list of exclusions so long on many even though the testplans are new?
- I have prepared a diff from the previous coverage. Please give it a look and see if all changes make sense, some are stautus changes (non-blocker -> blocker), some seem like addition or removals. Do comment on them.
Caveat, these dont include regex inclusions, as they arent supported by expand. If you want to play around with them use this
function diff_tp(){
tp1=$1
tp2=$2
expand_tp1=$(checkbox-cli expand --format=json $tp1 | jq -r '.[] | (.["template-id"] // .id) + " " + .["certification-status"]?' | sort)
expand_tp2=$(checkbox-cli expand --format=json $tp2 | jq -r '.[] | (.["template-id"] // .id) + " " + .["certification-status"]?' | sort)
diff <(echo "$expand_tp1") <(echo "$expand_tp2")
}Diffs: (< is 26.04, > is 24.04)
| mediacard/storage-preinserted-.* | ||
| bootstrap_include: | ||
| removable_partition |
There was a problem hiding this comment.
This was empty before, do we intend changing it? fine by me, weird it was empty before
There was a problem hiding this comment.
This is a missing job under the automated part, we should add it in this test plan.
| include: | ||
|
|
There was a problem hiding this comment.
is this supposed to be empty? whats the purpose of an empty testplan?
There was a problem hiding this comment.
Yes, they are empty on purpose!
It’s a placeholder strategy for the 26.04 cycle.
If we link these Component Test Plans now (even while empty), we never have to touch the 'Big Test Plan' like IOT/Server/Desktop etc. again just to add a job. We can just update the Component Plan later, and it automatically reflects in the main plan. It saves us from having to review or might missing to add the Big Plan every single time we add a test case.
| include: | ||
|
|
There was a problem hiding this comment.
We are making this empty and shifting all under a new testplan?
There was a problem hiding this comment.
The both of jobs (suspend-on-battery-power and suspend-on-AC-power) are related the suspend function test, it's more clearly to add both in the after suspend test plan.
| include: | ||
|
|
There was a problem hiding this comment.
Yes, they are empty on purpose!
It’s a placeholder strategy for the 26.04 cycle.
If we link these Component Test Plans now (even while empty), we never have to touch the 'Big Test Plan' like IOT/Server/Desktop etc. again just to add a job. We can just update the Component Plan later, and it automatically reflects in the main plan. It saves us from having to review or might missing to add the Big Plan every single time we add a test case.
| _name: Touchpad tests after suspend (Manual) | ||
| _description: | ||
| Touchpad tests after suspend (Manual) | ||
| include: |
There was a problem hiding this comment.
Minor, whats going on with tabs here?
There was a problem hiding this comment.
Actually i has modified a lot for this, this one is lost, I will modify it
| unit: test plan | ||
| _name: USB Type-C tests (Automated) | ||
| _description: USB Type-C tests (Automated) | ||
| include: |
There was a problem hiding this comment.
Yes, they are empty on purpose!
It’s a placeholder strategy for the 26.04 cycle.
If we link these Component Test Plans now (even while empty), we never have to touch the 'Big Test Plan' like IOT/Server/Desktop etc. again just to add a job. We can just update the Component Plan later, and it automatically reflects in the main plan. It saves us from having to review or might missing to add the Big Plan every single time we add a test case.
Ans: I've updated, thanks for pointing out.
I just copied and pasted the original job definition. There is no blocker, but I suggest we do that cleanup in a follow-up PR.
Ans: If we link these Component Test Plans now (even while empty), we never have to touch the 'Big Test Plan' like IOT/Server/Desktop etc. again just to add a job. We can just update the Component Plan later, and it automatically reflects in the main plan. It saves us from having to review or might missing to add the Big Plan every single time we add a test case.
Ans: Need @stanley31huang or @rickwu666666 to answer the question.
Ans: I've modified and updated in the google spreadsheets for reference.
I also renamed desktop to base per @pieqq's suggestion. |
I think these jobs don't suit the IoT project. ideally, we plan to manage these jobs via manifest. |
WARNING: This modifies com.canonical.certification::sru-server
Description
This goal for this PR is create certification test plans for PC platform, IoT platform running server image and desktop image.
there are some key points for those changes in this PR:
Resolved issues
Documentation
Tests