test: cover Berlin Group v1.3 alias in resource-doc registry parity test

The alias surface is gated by berlin_group_v1_3_alias_path, which is
unset in the default test environment, so its operation-id list is
legitimately empty there -- skip the non-empty assertion for it while
still running the membership check against getAllResourceDocs.

Author: hongwei1

obp-api/src/test/scala/code/api/util/ResourceDocRegistryParityTest.scala

@@ -11,11 +11,12 @@ import org.scalatest.Tag
  *
  * These are two parallel registries (ResourceDocsAPIMethods dispatches per
  * standard/version; getAllResourceDocs aggregates them all), and they have drifted
- * twice: Berlin Group v2 was served by the dispatcher but missing from the global
- * registry (so BGv2-getAccountDetails could not be added to an API collection),
- * and the global registry was based on the v6 aggregation, excluding v7-only
- * operation ids. When you add a NEW API standard, register its docs in BOTH
- * places — and add its surface to this list.
+ * three times: Berlin Group v2 was served by the dispatcher but missing from the
+ * global registry (so BGv2-getAccountDetails could not be added to an API
+ * collection), the global registry was based on the v6 aggregation, excluding
+ * v7-only operation ids, and the Berlin Group v1.3 alias (active only when
+ * berlin_group_v1_3_alias_path is set) was missing too. When you add a NEW API
+ * standard, register its docs in BOTH places — and add its surface to this list.
  */
 class ResourceDocRegistryParityTest extends ServerSetup {
 
@@ -24,19 +25,24 @@ class ResourceDocRegistryParityTest extends ServerSetup {
   private lazy val allOperationIds: Set[String] =
     APIUtil.getAllResourceDocs.map(_.operationId).toSet
 
-  private lazy val surfaces: List[(String, Seq[String])] = List(
-    ("OBP standard (v7 aggregation)", code.api.v7_0_0.Http4s700.allResourceDocs.map(_.operationId).toSeq),
-    ("Berlin Group v1.3", code.api.berlin.group.v1_3.Http4sBGv13.resourceDocs.map(_.operationId).toSeq),
-    ("Berlin Group v2", code.api.berlin.group.v2.Http4sBGv2.resourceDocs.map(_.operationId).toSeq),
-    ("UK Open Banking 2.0.0", code.api.UKOpenBanking.v2_0_0.OBP_UKOpenBanking_200.allResourceDocs.map(_.operationId).toSeq),
-    ("UK Open Banking 3.1.0", code.api.UKOpenBanking.v3_1_0.OBP_UKOpenBanking_310.allResourceDocs.map(_.operationId).toSeq),
-    ("UK Open Banking 4.0.1", code.api.UKOpenBanking.v4_0_1.OBP_UKOpenBanking_401.allResourceDocs.map(_.operationId).toSeq)
+  // optional=true surfaces are gated by a prop that is unset by default (test props
+  // included), so their operation-id list is legitimately empty in most environments --
+  // the membership check still runs (trivially true when empty) but the non-empty
+  // assertion is skipped for them.
+  private lazy val surfaces: List[(String, Seq[String], Boolean)] = List(
+    ("OBP standard (v7 aggregation)", code.api.v7_0_0.Http4s700.allResourceDocs.map(_.operationId).toSeq, false),
+    ("Berlin Group v1.3", code.api.berlin.group.v1_3.Http4sBGv13.resourceDocs.map(_.operationId).toSeq, false),
+    ("Berlin Group v1.3 alias", code.api.berlin.group.v1_3.Http4sBGv13Alias.resourceDocs.map(_.operationId).toSeq, true),
+    ("Berlin Group v2", code.api.berlin.group.v2.Http4sBGv2.resourceDocs.map(_.operationId).toSeq, false),
+    ("UK Open Banking 2.0.0", code.api.UKOpenBanking.v2_0_0.OBP_UKOpenBanking_200.allResourceDocs.map(_.operationId).toSeq, false),
+    ("UK Open Banking 3.1.0", code.api.UKOpenBanking.v3_1_0.OBP_UKOpenBanking_310.allResourceDocs.map(_.operationId).toSeq, false),
+    ("UK Open Banking 4.0.1", code.api.UKOpenBanking.v4_0_1.OBP_UKOpenBanking_401.allResourceDocs.map(_.operationId).toSeq, false)
   )
 
   feature("getAllResourceDocs contains every per-standard resource-doc surface") {
-    surfaces.foreach { case (label, operationIds) =>
+    surfaces.foreach { case (label, operationIds, optional) =>
       scenario(s"$label operation ids are all resolvable globally", RegistryParityTag) {
-        operationIds should not be empty
+        if (!optional) operationIds should not be empty
         val missing = operationIds.filterNot(allOperationIds.contains)
         withClue(s"$label operation ids missing from getAllResourceDocs: ${missing.take(10).mkString(", ")} ") {
           missing shouldBe empty