Simplify hyperlink tests

- Remove trivial ProgressRecord test for setting hash/workDir fields
- Simplify hyperlink rendering test to focus on actual hyperlink cases
- Fix fragile assertion that checked for BEL character

https://claude.ai/code/session_014AStcyMzd39iegXhfQL1yK
Signed-off-by: Claude <noreply@anthropic.com>

Author: claude

modules/nextflow/src/test/groovy/nextflow/trace/AnsiLogObserverTest.groovy

@@ -121,64 +121,51 @@ class AnsiLogObserverTest extends Specification {
         AnsiLogObserver.hyperlink('text', '') == 'text'
     }
 
-    @Unroll
-    def 'should render a process line with workDir hyperlink' () {
-
+    def 'should render hash as hyperlink when workDir is set' () {
         given:
         def session = Mock(Session) { getConfig() >> [cleanup: false] }
         def observer = new AnsiLogObserver()
         observer.@session = session
+        observer.@labelWidth = 3
+        observer.@cols = 190
+        and:
         def stats = new ProgressRecord(1, 'foo')
-        stats.submitted = SUBMIT
-        stats.succeeded = SUCCEEDED
-        stats.hash = HASH
+        stats.submitted = 1
+        stats.hash = '4e/486876'
         stats.workDir = WORKDIR
 
         when:
-        observer.@labelWidth = stats.name.size()
-        observer.@cols = 190
+        def result = observer.line(stats).toString()
 
         then:
-        // Strip ANSI color codes but preserve OSC 8 hyperlink sequences
-        def result = observer.line(stats).toString()
-        def strippedColors = result.replaceAll('\u001B\\[[\\d;]*[^\\d;]','')
-        strippedColors.contains(HASH_DISPLAY)
-        // Check hyperlink is present when workDir is set
-        if( WORKDIR ) {
-            assert result.contains('\033]8;;' + EXPECTED_HREF + '\007')
-            assert result.contains('\033]8;;\007')
-        }
+        result.contains('\033]8;;' + EXPECTED_HREF + '\007')
+        result.contains('\033]8;;\007')
 
         where:
-        HASH        | WORKDIR                          | SUBMIT  | SUCCEEDED | HASH_DISPLAY | EXPECTED_HREF
-        '4e/486876' | '/work/4e/486876abc'             | 1       | 0         | '4e/486876'  | 'file:///work/4e/486876abc'
-        '4e/486876' | 's3://bucket/work/4e/486876abc'  | 0       | 1         | '4e/486876'  | 's3://bucket/work/4e/486876abc'
-        '4e/486876' | null                             | 1       | 0         | '4e/486876'  | null
-        null        | null                             | 0       | 0         | '-'          | null
+        WORKDIR                          | EXPECTED_HREF
+        '/work/4e/486876abc'             | 'file:///work/4e/486876abc'
+        's3://bucket/work/4e/486876abc'  | 's3://bucket/work/4e/486876abc'
     }
 
     def 'should not render hyperlink when cleanup is enabled' () {
-
         given:
         def session = Mock(Session) { getConfig() >> [cleanup: true] }
         def observer = new AnsiLogObserver()
         observer.@session = session
+        observer.@labelWidth = 3
+        observer.@cols = 190
+        and:
         def stats = new ProgressRecord(1, 'foo')
         stats.submitted = 1
         stats.hash = '4e/486876'
         stats.workDir = '/work/4e/486876abc'
 
         when:
-        observer.@labelWidth = stats.name.size()
-        observer.@cols = 190
         def result = observer.line(stats).toString()
 
         then:
-        // Should NOT contain hyperlink escape sequences
+        // Should NOT contain hyperlink start sequence
         !result.contains('\033]8;;')
-        !result.contains('\007')
-        // But should still contain the hash
-        result.replaceAll('\u001B\\[[\\d;]*[^\\d;]','').contains('4e/486876')
     }
 
 }

modules/nextflow/src/test/groovy/nextflow/trace/ProgressRecordTest.groovy

@@ -53,19 +53,6 @@ class ProgressRecordTest extends Specification {
         }
     }
 
-    def 'should set hash and workDir' () {
-        given:
-        def rec = new ProgressRecord(10, 'foo')
-
-        when:
-        rec.hash = '4e/486876'
-        rec.workDir = 'file:///work/4e/486876abc'
-
-        then:
-        rec.hash == '4e/486876'
-        rec.workDir == 'file:///work/4e/486876abc'
-    }
-
     def 'should get counts' () {
         given:
         def PENDING = 1