You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
original is not a great name, and confounding it with DummyJenkinsfile and Jenkinsfile is even worse. Do something about this.
The text was updated successfully, but these errors were encountered:
kmanning
changed the title
Testing: Rename DummyJenkinsfile to something more sensible
Testing: Improve confusing Jenkinsfile naming references in testing
Feb 9, 2021
Jenkinsfile
class, that is meant to be a reference to global setting (https://github.com/manheim/terraform-pipeline/blob/master/src/Jenkinsfile.groovy)Jenkinsfile.init
method accepts a reference tothis
, which is the actual underlying Jenkinsfile library.this
is stored in a variable calledoriginal
- not the best name. https://github.com/manheim/terraform-pipeline/blob/master/src/Jenkinsfile.groovy#L2original
needs to be mocked/stubbed/spied on during a lot of testing. To make this easier, aDummyJenkinsfile
class is available to tests. https://github.com/manheim/terraform-pipeline/blob/master/test/DummyJenkinsfile.groovyoriginal
is not a great name, and confounding it withDummyJenkinsfile
andJenkinsfile
is even worse. Do something about this.The text was updated successfully, but these errors were encountered: