Skip to content

Commit c65f187

Browse files
pditommasoclaude
andcommitted
Fix AssetManagerTest.testPullHashTwice using commit hash resolved as tag-relative ref [ci fast]
The previous commit hash was resolved by JGit nameRev to v1.3~21, causing a RefNotAdvertisedException on the second download. Use a commit hash that is beyond all tags to avoid this issue. Signed-off-by: Paolo Di Tommaso <paolo.ditommaso@gmail.com> Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com> Signed-off-by: Paolo Di Tommaso <paolo.ditommaso@gmail.com>
1 parent 85a42fa commit c65f187

File tree

1 file changed

+2
-2
lines changed

1 file changed

+2
-2
lines changed

modules/nextflow/src/test/groovy/nextflow/scm/AssetManagerTest.groovy

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -207,12 +207,12 @@ class AssetManagerTest extends Specification {
207207
def manager = new AssetManager().build('nextflow-io/hello', [providers: [github: [auth: token]]])
208208

209209
when:
210-
manager.download("6b9515aba6c7efc6a9b3f273ce116fc0c224bf68")
210+
manager.download("d828daeef77f391c456896a7a6384b5f76ceea41")
211211
then:
212212
folder.resolve('nextflow-io/hello/.git').isDirectory()
213213

214214
when:
215-
def result = manager.download("6b9515aba6c7efc6a9b3f273ce116fc0c224bf68")
215+
def result = manager.download("d828daeef77f391c456896a7a6384b5f76ceea41")
216216
then:
217217
noExceptionThrown()
218218
result == "Already-up-to-date"

0 commit comments

Comments
 (0)