Skip to content

Commit 18f7643

Browse files
pditommasoclaude
andcommitted
Fix SeqeraExecutorTest missing SysEnv.push in new tests
The cleanup() method calls SysEnv.pop(), so every test must push first. 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 ecf0793 commit 18f7643

1 file changed

Lines changed: 2 additions & 0 deletions

File tree

plugins/nf-seqera/src/test/io/seqera/executor/SeqeraExecutorTest.groovy

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -112,6 +112,7 @@ class SeqeraExecutorTest extends Specification {
112112

113113
def 'should set fusion default version when not configured' () {
114114
given:
115+
SysEnv.push([:])
115116
def fusionConfig = [enabled: true]
116117
def config = [fusion: fusionConfig]
117118
def session = Mock(Session) { getConfig() >> config }
@@ -126,6 +127,7 @@ class SeqeraExecutorTest extends Specification {
126127

127128
def 'should not override fusion version when containerConfigUrl is set' () {
128129
given:
130+
SysEnv.push([:])
129131
def fusionConfig = [enabled: true, containerConfigUrl: 'https://custom.url/v3.0-amd64.json']
130132
def config = [fusion: fusionConfig]
131133
def session = Mock(Session) { getConfig() >> config }

0 commit comments

Comments
 (0)