Skip to content

Commit ecf0793

Browse files
pditommasoclaude
andcommitted
Fix Spock data table variable names in FusionConfigTest
Rename URL/VERSION to FUSION_URL/VER/TARGET_VER to avoid clashing with class names in Spock data table headers. Signed-off-by: Paolo Di Tommaso <paolo.ditommaso@gmail.com> Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com> Signed-off-by: Paolo Di Tommaso <paolo.ditommaso@gmail.com>
1 parent 2c8103f commit ecf0793

1 file changed

Lines changed: 12 additions & 12 deletions

File tree

modules/nextflow/src/test/groovy/nextflow/fusion/FusionConfigTest.groovy

Lines changed: 12 additions & 12 deletions
Original file line numberDiff line numberDiff line change
@@ -155,9 +155,9 @@ class FusionConfigTest extends Specification {
155155
@Unroll
156156
def 'should replace version in url' () {
157157
expect:
158-
FusionConfig.replaceVersion(URL, VERSION) == EXPECTED
158+
FusionConfig.replaceVersion(FUSION_URL, VER) == EXPECTED
159159
where:
160-
URL | VERSION | EXPECTED
160+
FUSION_URL | VER | EXPECTED
161161
'https://fusionfs.seqera.io/releases/v2.5-amd64.json' | '2.6' | 'https://fusionfs.seqera.io/releases/v2.6-amd64.json'
162162
'https://fusionfs.seqera.io/releases/v2.5-arm64.json' | '2.6' | 'https://fusionfs.seqera.io/releases/v2.6-arm64.json'
163163
'https://fusionfs.seqera.io/releases/v2.5-snap_amd64.json' | '2.6' | 'https://fusionfs.seqera.io/releases/v2.6-snap_amd64.json'
@@ -168,23 +168,23 @@ class FusionConfigTest extends Specification {
168168
@Unroll
169169
def 'should resolve default fusion url with version override' () {
170170
given:
171-
def config = new FusionConfig([targetVersion: DEFAULT_VERSION])
171+
def config = new FusionConfig([targetVersion: TARGET_VER])
172172
expect:
173-
config.targetFusionUrl(URL) == EXPECTED
173+
config.targetFusionUrl(FUSION_URL) == EXPECTED
174174
where:
175-
URL | DEFAULT_VERSION | EXPECTED
176-
FusionConfig.DEFAULT_FUSION_AMD64_URL | null | FusionConfig.DEFAULT_FUSION_AMD64_URL
177-
FusionConfig.DEFAULT_FUSION_AMD64_URL | '2.6' | 'https://fusionfs.seqera.io/releases/v2.6-amd64.json'
178-
FusionConfig.DEFAULT_FUSION_ARM64_URL | '2.6' | 'https://fusionfs.seqera.io/releases/v2.6-arm64.json'
179-
FusionConfig.DEFAULT_SNAPSHOT_AMD64_URL | '2.6' | 'https://fusionfs.seqera.io/releases/v2.6-snap_amd64.json'
180-
FusionConfig.DEFAULT_SNAPSHOT_ARM64_URL | '2.6' | 'https://fusionfs.seqera.io/releases/v2.6-snap_arm64.json'
175+
FUSION_URL | TARGET_VER | EXPECTED
176+
FusionConfig.DEFAULT_FUSION_AMD64_URL | null | FusionConfig.DEFAULT_FUSION_AMD64_URL
177+
FusionConfig.DEFAULT_FUSION_AMD64_URL | '2.6' | 'https://fusionfs.seqera.io/releases/v2.6-amd64.json'
178+
FusionConfig.DEFAULT_FUSION_ARM64_URL | '2.6' | 'https://fusionfs.seqera.io/releases/v2.6-arm64.json'
179+
FusionConfig.DEFAULT_SNAPSHOT_AMD64_URL | '2.6' | 'https://fusionfs.seqera.io/releases/v2.6-snap_amd64.json'
180+
FusionConfig.DEFAULT_SNAPSHOT_ARM64_URL | '2.6' | 'https://fusionfs.seqera.io/releases/v2.6-snap_arm64.json'
181181
}
182182

183183
def 'should get version with targetVersion override' () {
184184
expect:
185-
new FusionConfig([enabled:ENABLED, targetVersion:DEFAULT_VERSION]).version() == EXPECTED
185+
new FusionConfig([enabled:ENABLED, targetVersion:TARGET_VER]).version() == EXPECTED
186186
where:
187-
ENABLED | DEFAULT_VERSION | EXPECTED
187+
ENABLED | TARGET_VER | EXPECTED
188188
false | '2.6' | null
189189
true | null | '2.5'
190190
true | '2.6' | '2.6'

0 commit comments

Comments
 (0)