Question

似乎在下面的代码中：

def formattedPaths = affectedFiles.collect {
    "${it.editType.name} ${it.path}"
}

至少有时formattedPaths评估为GString而不是List。这段代码是更大的Jenkins Workflow脚本的片段，类似于：

node {
     currentBuild.rawBuild.changeSets[0].collect { 
         """<b>${it.user}</b> @ rev. ${it.revision}: ${it.msg}
         ${affectedFilesLog(it.affectedFiles)}"""
     }
}

def affectedFilesLog(affectedFiles) {
    println "Affected files [${affectedFiles.class}]: $affectedFiles"
    def formattedPaths = affectedFiles.collect {
        "${it.editType.name} ${it.path}"
    }

    println "formattedPaths [${formattedPaths.class}]: $formattedPaths"
    formatItemList(formattedPaths)
}

def formatItemList(list) {
    if (list) {
        return list.join('\n')
    }

    return  '(none)'
}

在Jenkins中运行此脚本会生成输出：

Running: Print Message
Affected files [class java.util.ArrayList]: [hudson.scm.SubversionChangeLogSet$Path@5030a7d8]
Running: Print Message
formattedPaths [class org.codehaus.groovy.runtime.GStringImpl]: edit my/path/flow.groovy
(...)
groovy.lang.MissingMethodException: No signature of method: java.lang.String.join() is applicable for argument types: (java.lang.String) values: [
]

这让我相信代码：

println "Affected files [${affectedFiles.class}]: $affectedFiles"
def formattedPaths = affectedFiles.collect {
    "${it.editType.name} ${it.path}"
}

println "formattedPaths [${formattedPaths.class}]: $formattedPaths"

affectedFiles为ArrayList（输出中的脚本输出Affected files [class java.util.ArrayList]: [hudson.scm.SubversionChangeLogSet$Path@5030a7d8]）

但是对它运行collect方法的结果 - 分配给formattedPaths - 是一个GString（输出：formattedPaths [class org.codehaus.groovy.runtime.GStringImpl]: edit my/path/flow.groovy）

collect方法是否总是返回List？

Answer 1

在评论中讨论指出Jenkins Workflow插件可能会产生一些副作用之后，我决定使用一个简单的for-each循环：

def affectedFilesLog(affectedFiles) {
    println "Affected files [${affectedFiles.class}]: $affectedFiles"

    def ret = ""
    for (Object affectedFile : affectedFiles) {
        ret += affectedFile.path + '\n'
    }

    println("affectedFilesLog ret [${ret.class}]: $ret")
    if (!ret) {
        return '(brak)'
    }

    return ret;
}

编辑19/11/2015： Jenkins工作流插件错误处理了闭包的功能，请参阅https://issues.jenkins-ci.org/browse/JENKINS-26481及其重复项。因此，将代码重写为普通的Java for-each循环是最好的解决方案。

Answer 2

您目前无法使用for方法。 JENKINS-26481

Answer 3

我猜你的代码不是线程安全的。如果将某些对象作为参数传递给其他函数，请不要更改它。始终创建并返回新更改的对象。不要操纵原始数据。你应该检查你的对象住在哪里。它只是在函数内还是在全局范围内？

Groovy在Jenkins工作流脚本中收集返回的GString

3 个答案: