Fix test failures for configuration cache support

- Fix BrowserUtilsSpec test by mocking CI environment check
- Update BrowserUtilsConfigurationCacheSpec to use parameterized tests with mock providers
- Fix UploadArtifactTask to remove extra .get() call on Boolean value

🤖 Generated with [Claude Code](https://claude.ai/code)

Co-Authored-By: Claude <noreply@anthropic.com>

Author: tnj

src/main/groovy/com/deploygate/gradle/plugins/tasks/UploadArtifactTask.groovy

@@ -128,7 +128,7 @@ abstract class UploadArtifactTask extends DefaultTask {
 
             def hasNotified = httpClient.get().lifecycleNotificationClient.notifyOnSuccessOfArtifactUpload(uploadResponse.typedResponse.application.path)
 
-            def shouldOpenBrowser = openBrowserAfterUpload.getOrElse(false).get()
+            def shouldOpenBrowser = openBrowserAfterUpload.getOrElse(false)
             if (!hasNotified && (shouldOpenBrowser || uploadResponse.typedResponse.application.revision == 1)) {
                 BrowserUtils.openBrowser("${endpoint.get()}${uploadResponse.typedResponse.application.path}", providerFactory)
             }

src/test/groovy/com/deploygate/gradle/plugins/internal/utils/BrowserUtilsConfigurationCacheSpec.groovy

@@ -23,21 +23,28 @@ class BrowserUtilsConfigurationCacheSpec extends Specification {
         providers = project.providers
     }
 
-    def "hasBrowser with ProviderFactory returns expected result"() {
-        given: "Provider factory from test project"
-        // In unit tests, we can evaluate providers directly as we're not in configuration phase
+    @Unroll
+    def "hasBrowser with ProviderFactory works correctly in different environments"() {
+        given: "Mock providers with controlled values"
+        def osNameProvider = providers.provider { osName }
+        def displayProvider = providers.provider { display }
+        def ciProvider = providers.provider { ci }
+        def jenkinsUrlProvider = providers.provider { jenkinsUrl }
 
         when: "Checking for browser availability"
-        def result = BrowserUtils.hasBrowser(providers)
-
-        then: "Result is based on current environment"
-        // The result will depend on the test environment (CI vs local, OS, etc.)
-        result != null
-        // In most test environments, this will be false due to CI detection or missing display
-        result == !BrowserUtils.isCiEnvironment() &&
-                (BrowserUtils.isExecutableOnMacOS() ||
-                BrowserUtils.isExecutableOnWindows() ||
-                BrowserUtils.isExecutableOnLinux())
+        def result = BrowserUtils.hasBrowser(osNameProvider, displayProvider, ciProvider, jenkinsUrlProvider)
+
+        then: "Result matches expected behavior"
+        result == expectedResult
+
+        where:
+        osName      | display | ci      | jenkinsUrl    | expectedResult
+        "Mac OS X"  | null    | "false" | null          | true
+        "Windows"   | null    | "false" | null          | true
+        "Linux"     | ":0"    | "false" | null          | true
+        "Linux"     | null    | "false" | null          | false
+        "Mac OS X"  | null    | "true"  | null          | false
+        "Windows"   | null    | "false" | "http://ci"   | false
     }
 
     def "openBrowser with ProviderFactory executes correctly"() {

src/test/groovy/com/deploygate/gradle/plugins/internal/utils/BrowserUtilsSpec.groovy

@@ -19,10 +19,19 @@ class BrowserUtilsSpec extends Specification {
             ->
             [waitFor: { -> 0 }]
         }
+        // Mock the CI environment to be false for testing browser functionality
+        BrowserUtils.metaClass.static.isCiEnvironment = { -> false }
         BrowserUtils.metaClass.static.hasBrowserLegacy = { -> hasBrowser }
         BrowserUtils.metaClass.static.isExecutableOnMacOS = { -> onMacOS }
         BrowserUtils.metaClass.static.isExecutableOnWindows = { -> onWindows }
         BrowserUtils.metaClass.static.isExecutableOnLinux = { -> onLinux }
+        BrowserUtils.metaClass.static.getOS_NAME = {
+            ->
+            if (onMacOS) return "mac"
+            if (onWindows) return "windows"
+            if (onLinux) return "linux"
+            return "unknown"
+        }
 
         expect:
         BrowserUtils.openBrowser(url) == result