[CI] fix null string with contains (elastic#20182)

v1v · melchiormoulin · commit aebc37240574 · 2020-10-14T15:00:52.000+02:00
diff --git a/Jenkinsfile b/Jenkinsfile
@@ -1343,7 +1343,7 @@ def loadConfigEnvVars(){
   env.BUILD_ON_MACOS = (params.macosTest                  // UI Input parameter is set to true
                         || !isPR()                        // For branches and tags
                         || matchesPrLabel(label: 'macOS') // If `macOS` GH label (Case-Sensitive)
-                        || (env.GITHUB_COMMENT?.toLowerCase().contains('/test macos'))) // If `/test macos` in the GH comment (Case-Insensitive)
+                        || (env.GITHUB_COMMENT?.toLowerCase()?.contains('/test macos'))) // If `/test macos` in the GH comment (Case-Insensitive)
 }
 
 /**

Original file line number	Diff line number	Diff line change
`@@ -1343,7 +1343,7 @@ def loadConfigEnvVars(){`
`1343`	`1343`	`env.BUILD_ON_MACOS = (params.macosTest // UI Input parameter is set to true`
`1344`	`1344`	`\|\| !isPR() // For branches and tags`
`1345`	`1345`	\|\| matchesPrLabel(label: 'macOS') // If `macOS` GH label (Case-Sensitive)
`1346`		- \|\| (env.GITHUB_COMMENT?.toLowerCase().contains('/test macos'))) // If `/test macos` in the GH comment (Case-Insensitive)
	`1346`	+ \|\| (env.GITHUB_COMMENT?.toLowerCase()?.contains('/test macos'))) // If `/test macos` in the GH comment (Case-Insensitive)
`1347`	`1347`	`}`
`1348`	`1348`
`1349`	`1349`	`/**`