From 16c1fad68aa6233cf75b435dfacc2bc4054ba7df Mon Sep 17 00:00:00 2001 From: adcoelho Date: Wed, 28 Jun 2023 15:03:29 +0200 Subject: [PATCH] Addressing PR comments. --- .../plugins/cases/server/client/cases/find.test.ts | 12 ++++++------ 1 file changed, 6 insertions(+), 6 deletions(-) diff --git a/x-pack/plugins/cases/server/client/cases/find.test.ts b/x-pack/plugins/cases/server/client/cases/find.test.ts index b89b255f8da14..76e15425ecef5 100644 --- a/x-pack/plugins/cases/server/client/cases/find.test.ts +++ b/x-pack/plugins/cases/server/client/cases/find.test.ts @@ -125,8 +125,8 @@ describe('find', () => { const findRequest = createCasesClientMockFindRequest({ tags }); - await expect(find(findRequest, clientArgs)).rejects.toThrowErrorMatchingInlineSnapshot( - `"Failed to find cases: {\\"search\\":\\"\\",\\"searchFields\\":[\\"title\\",\\"description\\"],\\"severity\\":\\"low\\",\\"assignees\\":[],\\"reporters\\":[],\\"status\\":\\"open\\",\\"tags\\":[\\"foobar\\",\\"foobar\\",\\"foobar\\",\\"foobar\\",\\"foobar\\",\\"foobar\\",\\"foobar\\",\\"foobar\\",\\"foobar\\",\\"foobar\\",\\"foobar\\",\\"foobar\\",\\"foobar\\",\\"foobar\\",\\"foobar\\",\\"foobar\\",\\"foobar\\",\\"foobar\\",\\"foobar\\",\\"foobar\\",\\"foobar\\",\\"foobar\\",\\"foobar\\",\\"foobar\\",\\"foobar\\",\\"foobar\\",\\"foobar\\",\\"foobar\\",\\"foobar\\",\\"foobar\\",\\"foobar\\",\\"foobar\\",\\"foobar\\",\\"foobar\\",\\"foobar\\",\\"foobar\\",\\"foobar\\",\\"foobar\\",\\"foobar\\",\\"foobar\\",\\"foobar\\",\\"foobar\\",\\"foobar\\",\\"foobar\\",\\"foobar\\",\\"foobar\\",\\"foobar\\",\\"foobar\\",\\"foobar\\",\\"foobar\\",\\"foobar\\",\\"foobar\\",\\"foobar\\",\\"foobar\\",\\"foobar\\",\\"foobar\\",\\"foobar\\",\\"foobar\\",\\"foobar\\",\\"foobar\\",\\"foobar\\",\\"foobar\\",\\"foobar\\",\\"foobar\\",\\"foobar\\",\\"foobar\\",\\"foobar\\",\\"foobar\\",\\"foobar\\",\\"foobar\\",\\"foobar\\",\\"foobar\\",\\"foobar\\",\\"foobar\\",\\"foobar\\",\\"foobar\\",\\"foobar\\",\\"foobar\\",\\"foobar\\",\\"foobar\\",\\"foobar\\",\\"foobar\\",\\"foobar\\",\\"foobar\\",\\"foobar\\",\\"foobar\\",\\"foobar\\",\\"foobar\\",\\"foobar\\",\\"foobar\\",\\"foobar\\",\\"foobar\\",\\"foobar\\",\\"foobar\\",\\"foobar\\",\\"foobar\\",\\"foobar\\",\\"foobar\\",\\"foobar\\",\\"foobar\\",\\"foobar\\"],\\"owner\\":[],\\"sortField\\":\\"createdAt\\",\\"sortOrder\\":\\"desc\\"}: Error: array must be of length <= 100,Invalid value \\"[\\"foobar\\",\\"foobar\\",\\"foobar\\",\\"foobar\\",\\"foobar\\",\\"foobar\\",\\"foobar\\",\\"foobar\\",\\"foobar\\",\\"foobar\\",\\"foobar\\",\\"foobar\\",\\"foobar\\",\\"foobar\\",\\"foobar\\",\\"foobar\\",\\"foobar\\",\\"foobar\\",\\"foobar\\",\\"foobar\\",\\"foobar\\",\\"foobar\\",\\"foobar\\",\\"foobar\\",\\"foobar\\",\\"foobar\\",\\"foobar\\",\\"foobar\\",\\"foobar\\",\\"foobar\\",\\"foobar\\",\\"foobar\\",\\"foobar\\",\\"foobar\\",\\"foobar\\",\\"foobar\\",\\"foobar\\",\\"foobar\\",\\"foobar\\",\\"foobar\\",\\"foobar\\",\\"foobar\\",\\"foobar\\",\\"foobar\\",\\"foobar\\",\\"foobar\\",\\"foobar\\",\\"foobar\\",\\"foobar\\",\\"foobar\\",\\"foobar\\",\\"foobar\\",\\"foobar\\",\\"foobar\\",\\"foobar\\",\\"foobar\\",\\"foobar\\",\\"foobar\\",\\"foobar\\",\\"foobar\\",\\"foobar\\",\\"foobar\\",\\"foobar\\",\\"foobar\\",\\"foobar\\",\\"foobar\\",\\"foobar\\",\\"foobar\\",\\"foobar\\",\\"foobar\\",\\"foobar\\",\\"foobar\\",\\"foobar\\",\\"foobar\\",\\"foobar\\",\\"foobar\\",\\"foobar\\",\\"foobar\\",\\"foobar\\",\\"foobar\\",\\"foobar\\",\\"foobar\\",\\"foobar\\",\\"foobar\\",\\"foobar\\",\\"foobar\\",\\"foobar\\",\\"foobar\\",\\"foobar\\",\\"foobar\\",\\"foobar\\",\\"foobar\\",\\"foobar\\",\\"foobar\\",\\"foobar\\",\\"foobar\\",\\"foobar\\",\\"foobar\\",\\"foobar\\",\\"foobar\\",\\"foobar\\"]\\" supplied to \\"tags\\""` + await expect(find(findRequest, clientArgs)).rejects.toThrowError( + 'Error: array must be of length <= 100' ); }); @@ -135,8 +135,8 @@ describe('find', () => { const findRequest = createCasesClientMockFindRequest({ assignees }); - await expect(find(findRequest, clientArgs)).rejects.toThrowErrorMatchingInlineSnapshot( - `"Failed to find cases: {\\"search\\":\\"\\",\\"searchFields\\":[\\"title\\",\\"description\\"],\\"severity\\":\\"low\\",\\"assignees\\":[\\"foobar\\",\\"foobar\\",\\"foobar\\",\\"foobar\\",\\"foobar\\",\\"foobar\\",\\"foobar\\",\\"foobar\\",\\"foobar\\",\\"foobar\\",\\"foobar\\",\\"foobar\\",\\"foobar\\",\\"foobar\\",\\"foobar\\",\\"foobar\\",\\"foobar\\",\\"foobar\\",\\"foobar\\",\\"foobar\\",\\"foobar\\",\\"foobar\\",\\"foobar\\",\\"foobar\\",\\"foobar\\",\\"foobar\\",\\"foobar\\",\\"foobar\\",\\"foobar\\",\\"foobar\\",\\"foobar\\",\\"foobar\\",\\"foobar\\",\\"foobar\\",\\"foobar\\",\\"foobar\\",\\"foobar\\",\\"foobar\\",\\"foobar\\",\\"foobar\\",\\"foobar\\",\\"foobar\\",\\"foobar\\",\\"foobar\\",\\"foobar\\",\\"foobar\\",\\"foobar\\",\\"foobar\\",\\"foobar\\",\\"foobar\\",\\"foobar\\",\\"foobar\\",\\"foobar\\",\\"foobar\\",\\"foobar\\",\\"foobar\\",\\"foobar\\",\\"foobar\\",\\"foobar\\",\\"foobar\\",\\"foobar\\",\\"foobar\\",\\"foobar\\",\\"foobar\\",\\"foobar\\",\\"foobar\\",\\"foobar\\",\\"foobar\\",\\"foobar\\",\\"foobar\\",\\"foobar\\",\\"foobar\\",\\"foobar\\",\\"foobar\\",\\"foobar\\",\\"foobar\\",\\"foobar\\",\\"foobar\\",\\"foobar\\",\\"foobar\\",\\"foobar\\",\\"foobar\\",\\"foobar\\",\\"foobar\\",\\"foobar\\",\\"foobar\\",\\"foobar\\",\\"foobar\\",\\"foobar\\",\\"foobar\\",\\"foobar\\",\\"foobar\\",\\"foobar\\",\\"foobar\\",\\"foobar\\",\\"foobar\\",\\"foobar\\",\\"foobar\\",\\"foobar\\",\\"foobar\\",\\"foobar\\"],\\"reporters\\":[],\\"status\\":\\"open\\",\\"tags\\":[],\\"owner\\":[],\\"sortField\\":\\"createdAt\\",\\"sortOrder\\":\\"desc\\"}: Error: array must be of length <= 100,Invalid value \\"[\\"foobar\\",\\"foobar\\",\\"foobar\\",\\"foobar\\",\\"foobar\\",\\"foobar\\",\\"foobar\\",\\"foobar\\",\\"foobar\\",\\"foobar\\",\\"foobar\\",\\"foobar\\",\\"foobar\\",\\"foobar\\",\\"foobar\\",\\"foobar\\",\\"foobar\\",\\"foobar\\",\\"foobar\\",\\"foobar\\",\\"foobar\\",\\"foobar\\",\\"foobar\\",\\"foobar\\",\\"foobar\\",\\"foobar\\",\\"foobar\\",\\"foobar\\",\\"foobar\\",\\"foobar\\",\\"foobar\\",\\"foobar\\",\\"foobar\\",\\"foobar\\",\\"foobar\\",\\"foobar\\",\\"foobar\\",\\"foobar\\",\\"foobar\\",\\"foobar\\",\\"foobar\\",\\"foobar\\",\\"foobar\\",\\"foobar\\",\\"foobar\\",\\"foobar\\",\\"foobar\\",\\"foobar\\",\\"foobar\\",\\"foobar\\",\\"foobar\\",\\"foobar\\",\\"foobar\\",\\"foobar\\",\\"foobar\\",\\"foobar\\",\\"foobar\\",\\"foobar\\",\\"foobar\\",\\"foobar\\",\\"foobar\\",\\"foobar\\",\\"foobar\\",\\"foobar\\",\\"foobar\\",\\"foobar\\",\\"foobar\\",\\"foobar\\",\\"foobar\\",\\"foobar\\",\\"foobar\\",\\"foobar\\",\\"foobar\\",\\"foobar\\",\\"foobar\\",\\"foobar\\",\\"foobar\\",\\"foobar\\",\\"foobar\\",\\"foobar\\",\\"foobar\\",\\"foobar\\",\\"foobar\\",\\"foobar\\",\\"foobar\\",\\"foobar\\",\\"foobar\\",\\"foobar\\",\\"foobar\\",\\"foobar\\",\\"foobar\\",\\"foobar\\",\\"foobar\\",\\"foobar\\",\\"foobar\\",\\"foobar\\",\\"foobar\\",\\"foobar\\",\\"foobar\\",\\"foobar\\",\\"foobar\\"]\\" supplied to \\"assignees\\""` + await expect(find(findRequest, clientArgs)).rejects.toThrowError( + 'Error: array must be of length <= 100' ); }); @@ -145,8 +145,8 @@ describe('find', () => { const findRequest = createCasesClientMockFindRequest({ reporters }); - await expect(find(findRequest, clientArgs)).rejects.toThrowErrorMatchingInlineSnapshot( - `"Failed to find cases: {\\"search\\":\\"\\",\\"searchFields\\":[\\"title\\",\\"description\\"],\\"severity\\":\\"low\\",\\"assignees\\":[],\\"reporters\\":[\\"foobar\\",\\"foobar\\",\\"foobar\\",\\"foobar\\",\\"foobar\\",\\"foobar\\",\\"foobar\\",\\"foobar\\",\\"foobar\\",\\"foobar\\",\\"foobar\\",\\"foobar\\",\\"foobar\\",\\"foobar\\",\\"foobar\\",\\"foobar\\",\\"foobar\\",\\"foobar\\",\\"foobar\\",\\"foobar\\",\\"foobar\\",\\"foobar\\",\\"foobar\\",\\"foobar\\",\\"foobar\\",\\"foobar\\",\\"foobar\\",\\"foobar\\",\\"foobar\\",\\"foobar\\",\\"foobar\\",\\"foobar\\",\\"foobar\\",\\"foobar\\",\\"foobar\\",\\"foobar\\",\\"foobar\\",\\"foobar\\",\\"foobar\\",\\"foobar\\",\\"foobar\\",\\"foobar\\",\\"foobar\\",\\"foobar\\",\\"foobar\\",\\"foobar\\",\\"foobar\\",\\"foobar\\",\\"foobar\\",\\"foobar\\",\\"foobar\\",\\"foobar\\",\\"foobar\\",\\"foobar\\",\\"foobar\\",\\"foobar\\",\\"foobar\\",\\"foobar\\",\\"foobar\\",\\"foobar\\",\\"foobar\\",\\"foobar\\",\\"foobar\\",\\"foobar\\",\\"foobar\\",\\"foobar\\",\\"foobar\\",\\"foobar\\",\\"foobar\\",\\"foobar\\",\\"foobar\\",\\"foobar\\",\\"foobar\\",\\"foobar\\",\\"foobar\\",\\"foobar\\",\\"foobar\\",\\"foobar\\",\\"foobar\\",\\"foobar\\",\\"foobar\\",\\"foobar\\",\\"foobar\\",\\"foobar\\",\\"foobar\\",\\"foobar\\",\\"foobar\\",\\"foobar\\",\\"foobar\\",\\"foobar\\",\\"foobar\\",\\"foobar\\",\\"foobar\\",\\"foobar\\",\\"foobar\\",\\"foobar\\",\\"foobar\\",\\"foobar\\",\\"foobar\\",\\"foobar\\",\\"foobar\\"],\\"status\\":\\"open\\",\\"tags\\":[],\\"owner\\":[],\\"sortField\\":\\"createdAt\\",\\"sortOrder\\":\\"desc\\"}: Error: array must be of length <= 100,Invalid value \\"[\\"foobar\\",\\"foobar\\",\\"foobar\\",\\"foobar\\",\\"foobar\\",\\"foobar\\",\\"foobar\\",\\"foobar\\",\\"foobar\\",\\"foobar\\",\\"foobar\\",\\"foobar\\",\\"foobar\\",\\"foobar\\",\\"foobar\\",\\"foobar\\",\\"foobar\\",\\"foobar\\",\\"foobar\\",\\"foobar\\",\\"foobar\\",\\"foobar\\",\\"foobar\\",\\"foobar\\",\\"foobar\\",\\"foobar\\",\\"foobar\\",\\"foobar\\",\\"foobar\\",\\"foobar\\",\\"foobar\\",\\"foobar\\",\\"foobar\\",\\"foobar\\",\\"foobar\\",\\"foobar\\",\\"foobar\\",\\"foobar\\",\\"foobar\\",\\"foobar\\",\\"foobar\\",\\"foobar\\",\\"foobar\\",\\"foobar\\",\\"foobar\\",\\"foobar\\",\\"foobar\\",\\"foobar\\",\\"foobar\\",\\"foobar\\",\\"foobar\\",\\"foobar\\",\\"foobar\\",\\"foobar\\",\\"foobar\\",\\"foobar\\",\\"foobar\\",\\"foobar\\",\\"foobar\\",\\"foobar\\",\\"foobar\\",\\"foobar\\",\\"foobar\\",\\"foobar\\",\\"foobar\\",\\"foobar\\",\\"foobar\\",\\"foobar\\",\\"foobar\\",\\"foobar\\",\\"foobar\\",\\"foobar\\",\\"foobar\\",\\"foobar\\",\\"foobar\\",\\"foobar\\",\\"foobar\\",\\"foobar\\",\\"foobar\\",\\"foobar\\",\\"foobar\\",\\"foobar\\",\\"foobar\\",\\"foobar\\",\\"foobar\\",\\"foobar\\",\\"foobar\\",\\"foobar\\",\\"foobar\\",\\"foobar\\",\\"foobar\\",\\"foobar\\",\\"foobar\\",\\"foobar\\",\\"foobar\\",\\"foobar\\",\\"foobar\\",\\"foobar\\",\\"foobar\\",\\"foobar\\",\\"foobar\\"]\\" supplied to \\"reporters\\""` + await expect(find(findRequest, clientArgs)).rejects.toThrowError( + 'Error: array must be of length <= 100' ); }); });