Add QEP test cases and plans from checkbox-iiotg-providers and iiotg provider (New)#901
Conversation
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## main #901 +/- ##
==========================================
+ Coverage 36.58% 36.78% +0.20%
==========================================
Files 310 310
Lines 34639 34637 -2
Branches 5965 5965
==========================================
+ Hits 12671 12740 +69
+ Misses 21399 21328 -71
Partials 569 569
Flags with carried forward coverage won't be shown. Click here to find out more. ☔ View full report in Codecov by Sentry. |
pieqq
left a comment
There was a problem hiding this comment.
This PR comes with new jobs, but no associated test plans. I had to create one manually to test them in Checkbox. Did you forget to add the test plan file, or is it expected?
Other than that, it looks OK to me, I just made a few suggestions below (the description text does not need \). You can accept the suggestions and create an associated commit directly inside GitHub in one click :)
Co-authored-by: Pierre Equoy <pierre.equoy@canonical.com>
* Add QEP test cases and plans from checkbox-iiotg-providers and iiotg provider * Apply suggestions from code review Co-authored-by: Pierre Equoy <pierre.equoy@canonical.com> * Add qep testplan --------- Co-authored-by: Pierre Equoy <pierre.equoy@canonical.com>
* Add QEP test cases and plans from checkbox-iiotg-providers and iiotg provider * Apply suggestions from code review Co-authored-by: Pierre Equoy <pierre.equoy@canonical.com> * Add qep testplan --------- Co-authored-by: Pierre Equoy <pierre.equoy@canonical.com>
Description
I move the MEI related test cases from checkbox-iiotg-provider to the iiotg provider in checkbox monorepo.
Resolved issues
N/A
Documentation
N/A
Tests
I run testplan
qep-fullyou can sayQuadrature Encoder Peripheral (QEP) testson EHL (202306-31656) which support QEP.submission: https://certification.canonical.com/hardware/202306-31656/submission/349195/