bbsim related fixes
jjb/pipeline/voltha/master/bbsim-tests.groovy
o Remove self from installKind() call.
o Jenkins script object pointer needed for passing but syntax {self,this,etc} not correct.
vars/iam.groovy
vars/installKind.groovy
-----------------------
o Why do languages still support this snowflake condition ?!?
o Remove comma separator from the final argument specified in a function declaration.
o Simplified editing when removal of trailing delimiter is not a requirement.
o Always code lines with delimiter results in no potential for a syntax error with edits.
Change-Id: I8989c1cf2e783ffc49a99f470a3484c3205fb254
diff --git a/vars/iam.groovy b/vars/iam.groovy
index 845bdd0..e50fc8d 100644
--- a/vars/iam.groovy
+++ b/vars/iam.groovy
@@ -32,7 +32,7 @@
String iam = [src, func].join('::')
if (argv.containsKey('version'))
{
- iam += sprintf("[%s]", argv.version)
+ iam += sprintf('[%s]', argv.version)
}
return(iam)
}
@@ -112,7 +112,7 @@
// -----------------------------------------------------------------------
Boolean call\
(
- Closure body, // jenkins closure attached to the call iam() {closure}
+ Closure body // jenkins closure attached to the call iam() {closure}
// def self, // jenkins env object for access to primitives like echo()
)
{
@@ -140,9 +140,7 @@
ranToCompletion = true
}
}
- /* groovylint-disable*/ /* yuck! */
catch (Exception err)
- /* groovylint-enable */
{
println("** ${iam}: EXCEPTION ${err}")
throw err