Fixes #30117: hide soft-deleted owners on the Test Suite detail page

openmetadata-ui/src/main/resources/ui/src/rest/testAPI.test.ts

@@ -883,7 +883,7 @@ describe('testAPI tests', () => {
     });
 
     describe('getTestSuiteByName', () => {
-      it('should fetch test suite by name with params', async () => {
+      it('should default owners/experts to non-deleted so soft-deleted owners stay hidden (issue #30117)', async () => {
         const mockGet = jest.fn().mockResolvedValue({ data: mockTestSuite });
         jest.mock('./index', () => ({
           __esModule: true,
@@ -894,17 +894,42 @@ describe('testAPI tests', () => {
 
         const { getTestSuiteByName } = require('./testAPI');
 
-        const params = { fields: ['owners', 'tests'] };
-
-        const result = await getTestSuiteByName('test.suite.name', params);
+        const result = await getTestSuiteByName('test.suite.name', {
+          fields: ['owners', 'tests'],
+        });
 
         expect(mockGet).toHaveBeenCalledWith(
           expect.stringContaining('/dataQuality/testSuites/name/'),
-          { params }
+          {
+            params: {
+              fields: ['owners', 'tests'],
+              includeRelations: 'owners:non-deleted,experts:non-deleted',
+            },
+          }
         );
         expect(result).toEqual(mockTestSuite);
       });
 
+      it('should respect a caller-provided includeRelations', async () => {
+        const mockGet = jest.fn().mockResolvedValue({ data: mockTestSuite });
+        jest.mock('./index', () => ({
+          __esModule: true,
+          default: {
+            get: mockGet,
+          },
+        }));
+
+        const { getTestSuiteByName } = require('./testAPI');
+
+        await getTestSuiteByName('test.suite.name', {
+          includeRelations: 'owners:all',
+        });
+
+        expect(mockGet).toHaveBeenCalledWith(expect.any(String), {
+          params: { includeRelations: 'owners:all' },
+        });
+      });
+
       it('should encode FQN with special characters', async () => {
         const mockGet = jest.fn().mockResolvedValue({ data: mockTestSuite });
         jest.mock('./index', () => ({

openmetadata-ui/src/main/resources/ui/src/rest/testAPI.ts

@@ -413,7 +413,17 @@ export const getTestSuiteByName = async (
 ) => {
   const response = await APIClient.get<TestSuite>(
     `${testSuiteUrl}/name/${getEncodedFqn(name)}`,
-    { params }
+    {
+      // Resolve owners/experts as non-deleted (consistent with data-asset detail pages), so a
+      // soft-deleted owner is not surfaced and the owner-edit diff stays aligned with the server's
+      // NON_DELETED PATCH base - otherwise a positional patch throws "array item index out of
+      // range". See issue #30117.
+      params: {
+        ...params,
+        includeRelations:
+          params?.includeRelations ?? 'owners:non-deleted,experts:non-deleted',
+      },
+    }
   );
 
   return response.data;