Add missing tests for a non-empty Authenticated annotation and

custom access annotation on the same web action.

GitOrigin-RevId: 5b27040f40b8a805c50058677a28b32baa688c1c

Author: adrw

misk/src/test/kotlin/misk/web/actions/AuthenticationTest.kt

@@ -22,6 +22,7 @@ import org.junit.jupiter.params.ParameterizedTest
 import org.junit.jupiter.params.provider.ValueSource
 import jakarta.inject.Inject
 import misk.security.authz.AccessInterceptor
+import misk.web.toResponseBody
 import wisp.logging.LogCollector
 import kotlin.test.assertFailsWith
 
@@ -273,6 +274,34 @@ class AuthenticationTest {
     )
   }
 
+  @Test
+  fun `stacking @Authenticated with other access annotations is an error`() {
+    val unauthService = MiskCaller(service = "test")
+    assertThat(
+      executeRequest(
+        path = "/auth-and-custom-capability",
+        service = unauthService.service
+      )
+    ).isEqualTo("unauthorized")
+
+    val authService = MiskCaller(service = "dingo")
+    assertThat(
+      executeRequest(
+        path = "/auth-and-custom-capability",
+        service = authService.service
+      )
+    ).isEqualTo("$authService authorized with custom capability")
+
+    val caller = MiskCaller(user = "bob", capabilities = setOf("admin"))
+    assertThat(
+      executeRequest(
+        path = "/auth-and-custom-capability",
+        user = caller.user,
+        capabilities = caller.capabilities.first()
+      )
+    ).isEqualTo("$caller authorized with custom capability")
+  }
+
   /** Executes a request and returns the response body as a string. */
   private fun executeRequest(
     path: String = "/",

misk/src/test/kotlin/misk/web/actions/TestWebActionModule.kt

@@ -45,10 +45,10 @@ class TestWebActionModule : KAbstractModule() {
     install(WebActionModule.create<GreetServiceWebAction>())
     install(WebActionModule.create<EmptyAuthenticatedAccessAction>())
     install(WebActionModule.create<EmptyAuthenticatedWithCustomAnnototationAccessAction>())
-    install(WebActionModule.create<EmptyAuthenticatedAccessAction>())
     install(WebActionModule.create<AllowAnyServiceAccessAction>())
     install(WebActionModule.create<AllowAnyServicePlusAuthenticatedAccessAction>())
     install(WebActionModule.create<AllowAnyUserAccessAction>())
+    install(WebActionModule.create<AuthenticatedServiceWithCustomAnnotations>())
 
     multibind<AccessAnnotationEntry>().toInstance(
       AccessAnnotationEntry<CustomServiceAccess>(services = listOf("payments"))
@@ -167,6 +167,16 @@ class EmptyAuthenticatedWithCustomAnnototationAccessAction @Inject constructor()
   fun get() = "${scopedCaller.get()} authorized with CustomCapabilityAccess".toResponseBody()
 }
 
+class AuthenticatedServiceWithCustomAnnotations @Inject constructor() : WebAction {
+  @Inject
+  lateinit var scopedCaller: ActionScoped<MiskCaller?>
+
+  @Get("/auth-and-custom-capability")
+  @Authenticated(services = ["dingo"])
+  @CustomCapabilityAccess
+  fun get() = "${scopedCaller.get()} authorized with custom capability".toResponseBody()
+}
+
 class AllowAnyServiceAccessAction @Inject constructor() : WebAction {
   @Inject
   lateinit var scopedCaller: ActionScoped<MiskCaller?>