-
Notifications
You must be signed in to change notification settings - Fork 25
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
update jenkins tests with new execute-test command #471
update jenkins tests with new execute-test command #471
Conversation
Signed-off-by: Michael Oviedo <[email protected]>
Signed-off-by: Michael Oviedo <[email protected]>
c341640
to
b697c13
Compare
Signed-off-by: Michael Oviedo <[email protected]>
c351860
to
f71bd48
Compare
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.
LGTM
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## main #471 +/- ##
=========================================
Coverage 84.26% 84.26%
Complexity 80 80
=========================================
Files 106 106
Lines 521 521
Branches 61 61
=========================================
Hits 439 439
Misses 26 26
Partials 56 56 ☔ View full report in Codecov by Sentry. |
@@ -74,6 +74,7 @@ void call(Map args = [:]) { | |||
def command = [ | |||
'./test.sh', | |||
'benchmark-test', | |||
args.command, |
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.
Please add the param description like others at the top.
Signed-off-by: Michael Oviedo <[email protected]> (cherry picked from commit 1042431) Signed-off-by: github-actions[bot] <github-actions[bot]@users.noreply.github.com>
Description
update jenkins tests to call benchmark with new 'execute-test' command
Issues Resolved
#4837
By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.
For more information on following Developer Certificate of Origin and signing off your commits, please check here.