Skip creation of Search Criteria object for PCS-STAGING (#1990)

Author: toby-plunkett

src/main/java/uk/gov/hmcts/reform/pcs/ccd/CaseType.java

@@ -51,6 +51,18 @@ private static String withSuffix(String base, String separator) {
             .orElse(base);
     }
 
+    /**
+     * Whether this deployment runs a suffixed case type, e.g. PCS-STAGING or a
+     * per-PR preview type, driven by the CASE_TYPE_SUFFIX env var.
+     */
+    public static boolean isSuffixedCaseType() {
+        return isSuffixed(getenv().get("CASE_TYPE_SUFFIX"));
+    }
+
+    static boolean isSuffixed(String suffix) {
+        return suffix != null && !suffix.isBlank();
+    }
+
     @Override
     public void configure(final ConfigBuilder<PCSCase, State, AccessProfile> builder) {
         builder.setCallbackHost(caseApiUrl);

src/main/java/uk/gov/hmcts/reform/pcs/ccd/PCSCaseView.java

@@ -110,7 +110,10 @@ public PCSCase getCase(CaseViewRequest<State> request) {
 
         caseFieldsView.setCaseFields(pcsCase);
 
-        pcsCase.setSearchCriteria(searchCriteriaIndexer.buildSearchCriteria(pcsCase));
+        // Only the canonical PCS case type is indexed into the shared global_search index.
+        if (!CaseType.isSuffixedCaseType()) {
+            pcsCase.setSearchCriteria(searchCriteriaIndexer.buildSearchCriteria(pcsCase));
+        }
 
         return pcsCase;
     }

src/test/java/uk/gov/hmcts/reform/pcs/ccd/CaseTypeTest.java

@@ -41,6 +41,22 @@ void shouldGetCaseType() {
         assertThat(caseType).contains("PCS");
     }
 
+    @Test
+    void shouldNotBeSuffixedCaseTypeWhenSuffixAbsentOrBlank() {
+        // Canonical PCS (indexed into global search): unset or blank CASE_TYPE_SUFFIX.
+        // The pipeline encodes "canonical" as either unset or an empty string (see Jenkinsfile_CNP).
+        assertThat(CaseType.isSuffixed(null)).isFalse();
+        assertThat(CaseType.isSuffixed("")).isFalse();
+        assertThat(CaseType.isSuffixed("   ")).isFalse();
+    }
+
+    @Test
+    void shouldBeSuffixedCaseTypeWhenSuffixSet() {
+        // Suffixed (e.g. PCS-STAGING, PR previews) -> NOT indexed into global search.
+        assertThat(CaseType.isSuffixed("staging")).isTrue();
+        assertThat(CaseType.isSuffixed("1234")).isTrue();
+    }
+
     @Test
     void shouldGetJurisdictionId() {
         // When

src/test/java/uk/gov/hmcts/reform/pcs/ccd/PCSCaseViewTest.java

@@ -5,6 +5,7 @@
 import org.junit.jupiter.api.extension.ExtendWith;
 import org.mockito.ArgumentCaptor;
 import org.mockito.Mock;
+import org.mockito.MockedStatic;
 import org.mockito.junit.jupiter.MockitoExtension;
 import org.modelmapper.ModelMapper;
 import uk.gov.hmcts.ccd.sdk.CaseViewRequest;
@@ -55,6 +56,7 @@
 import static org.mockito.Mock.Strictness.LENIENT;
 import static org.mockito.Mockito.doNothing;
 import static org.mockito.Mockito.mock;
+import static org.mockito.Mockito.mockStatic;
 import static org.mockito.Mockito.never;
 import static org.mockito.Mockito.verify;
 import static org.mockito.Mockito.when;
@@ -209,6 +211,22 @@ void shouldSetSearchCriteriaFromIndexer() {
         assertThat(pcsCase.getSearchCriteria()).isSameAs(searchCriteria);
     }
 
+    @Test
+    void shouldNotSetSearchCriteriaForSuffixedCaseType() {
+        // Given a suffixed (non-canonical) case type, e.g. PCS-STAGING or a PR preview type
+        try (MockedStatic<CaseType> caseTypeMock = mockStatic(CaseType.class)) {
+            caseTypeMock.when(CaseType::isSuffixedCaseType).thenReturn(true);
+
+            // When
+            PCSCase pcsCase = underTest.getCase(request(CASE_REFERENCE, DEFAULT_STATE));
+
+            // Then - the indexer is never invoked and SearchCriteria is left null, so the
+            // decentralised indexer won't write a global_search document for this case type
+            verify(searchCriteriaIndexer, never()).buildSearchCriteria(any(PCSCase.class));
+            assertThat(pcsCase.getSearchCriteria()).isNull();
+        }
+    }
+
     @Test
     void shouldMapPartyEntity() {
         // Given