fix UnusedMethodParameter linter issue
refs DE-643 flag = none test-plan: - Build passes Change-Id: I4e00b018158c33f823bae8ebeea1f4e4c37721d3 Reviewed-on: https://gerrit.instructure.com/c/canvas-lms/+/264706 Tested-by: Service Cloud Jenkins <svc.cloudjenkins@instructure.com> Reviewed-by: James Butters <jbutters@instructure.com> Reviewed-by: Aaron Ogata <aogata@instructure.com> QA-Review: Andrea Cirulli <andrea.cirulli@instructure.com> Product-Review: Andrea Cirulli <andrea.cirulli@instructure.com>
This commit is contained in:
parent
457898f814
commit
5f76e48163
|
@ -31,7 +31,6 @@ import groovy.time.*
|
|||
def appendStagesAsBuildNodes(nodes,
|
||||
stageCount,
|
||||
stageNamePrefix,
|
||||
testLabel,
|
||||
stageBlock) {
|
||||
for (int i = 0; i < stageCount; i++) {
|
||||
// make this a local variable so when the closure resolves
|
||||
|
@ -65,7 +64,7 @@ def unstashBuildScripts() {
|
|||
def addRSpecSuites(stages) {
|
||||
def rspecNodeTotal = rspec.rspecConfig().node_total
|
||||
echo 'adding RSpec Test Sets'
|
||||
appendStagesAsBuildNodes(stages, rspecNodeTotal, 'RSpec Test Set', 'rspec') { index ->
|
||||
appendStagesAsBuildNodes(stages, rspecNodeTotal, 'RSpec Test Set') { index ->
|
||||
rspec.runRSpecSuite(rspecNodeTotal, index)
|
||||
}
|
||||
}
|
||||
|
@ -76,7 +75,7 @@ def addRSpecSuites(stages) {
|
|||
def addSeleniumSuites(stages) {
|
||||
def seleniumNodeTotal = rspec.seleniumConfig().node_total
|
||||
echo 'adding Selenium Test Sets'
|
||||
appendStagesAsBuildNodes(stages, seleniumNodeTotal, 'Selenium Test Set', 'selenium') { index ->
|
||||
appendStagesAsBuildNodes(stages, seleniumNodeTotal, 'Selenium Test Set') { index ->
|
||||
rspec.runSeleniumSuite(seleniumNodeTotal, index)
|
||||
}
|
||||
}
|
||||
|
|
|
@ -178,14 +178,14 @@ def _runRspecTestSuite(
|
|||
}
|
||||
|
||||
def uploadSeleniumCoverage() {
|
||||
_uploadCoverage('selenium', 'canvas-lms-selenium')
|
||||
_uploadCoverage('canvas-lms-selenium')
|
||||
}
|
||||
|
||||
def uploadRSpecCoverage() {
|
||||
_uploadCoverage('rspec', 'canvas-lms-rspec')
|
||||
_uploadCoverage('canvas-lms-rspec')
|
||||
}
|
||||
|
||||
def _uploadCoverage(prefix, coverageName) {
|
||||
def _uploadCoverage(coverageName) {
|
||||
reports.publishSpecCoverageToS3('tmp/spec_coverage/**/*', "$coverageName/coverage")
|
||||
}
|
||||
|
||||
|
|
Loading…
Reference in New Issue