Merged
Conversation
add regulator test
stanley31huang
requested review from
LiaoU3,
rickwu666666,
seankingyang and
weizhenwu
stanley31huang
added
the
ce-oem
stanley31huang
force-pushed
the
add_regulator_test
branch
from
a5cfc58 to
eeffce3
Compare
fixed pep8 error
stanley31huang
force-pushed
the
add_regulator_test
branch
from
eeffce3 to
693e494
Compare
revised regulator tests
stanley31huang
force-pushed
the
add_regulator_test
branch
from
5e05966 to
2e4bd77
Compare
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## main #2016 +/- ##
=======================================
Coverage 50.60% 50.60%
=======================================
Files 384 384
Lines 41180 41180
Branches 7642 7642
=======================================
Hits 20841 20841
Misses 19592 19592
Partials 747 747 ☔ View full report in Codecov by Sentry. 🚀 New features to boost your workflow:
|
seankingyang
requested changes
Jul 15, 2025
Contributor
seankingyang
left a comment
seankingyang
left a comment
There was a problem hiding this comment.
Please check the following suggestions.
contrib/checkbox-ce-oem/checkbox-provider-ce-oem/bin/regulator_test.py
Outdated
Show resolved
Hide resolved
contrib/checkbox-ce-oem/checkbox-provider-ce-oem/bin/regulator_test.py
Outdated
Show resolved
Hide resolved
contrib/checkbox-ce-oem/checkbox-provider-ce-oem/bin/regulator_test.py
Outdated
Show resolved
Hide resolved
contrib/checkbox-ce-oem/checkbox-provider-ce-oem/bin/regulator_test.py
Outdated
Show resolved
Hide resolved
revised the regulator test scripts
stanley31huang
force-pushed
the
add_regulator_test
branch
from
4027eb8 to
5dde7d2
Compare
stanley31huang
force-pushed
the
add_regulator_test
branch
from
5dde7d2 to
69049a1
Compare
revised regulator test scripts
add regulator test plan into ce-oem test plan
stanley31huang
force-pushed
the
add_regulator_test
branch
from
69049a1 to
f5a4ddc
Compare
stanley31huang
added a commit
that referenced
this pull request
Aug 14, 2025
* add regulator test add regulator test * fixed pep8 error fixed pep8 error * revised regulator tests revised regulator tests * revised the regulator test scripts revised the regulator test scripts * revised regulator test scripts revised regulator test scripts * add regulator test plan into ce-oem test plan add regulator test plan into ce-oem test plan
bladernr
pushed a commit
that referenced
this pull request
Aug 28, 2025
* add regulator test add regulator test * fixed pep8 error fixed pep8 error * revised regulator tests revised regulator tests * revised the regulator test scripts revised the regulator test scripts * revised regulator test scripts revised regulator test scripts * add regulator test plan into ce-oem test plan add regulator test plan into ce-oem test plan
stanley31huang
added a commit
that referenced
this pull request
Oct 3, 2025
* add regulator test add regulator test * fixed pep8 error fixed pep8 error * revised regulator tests revised regulator tests * revised the regulator test scripts revised the regulator test scripts * revised regulator test scripts revised regulator test scripts * add regulator test plan into ce-oem test plan add regulator test plan into ce-oem test plan
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
3 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Description
Adding a new voltage regulator tests.
Resolved issues
https://warthogs.atlassian.net/browse/BND-103
Documentation
Tests
Side-loaded test results:
https://certification.canonical.com/hardware/202507-37000/submission/439585/