[Security Solution][Exceptions]- Increase exceptions test coverage

.buildkite/pipelines/pull_request/security_solution.yml

@@ -5,7 +5,7 @@ steps:
       queue: n2-4-spot
     depends_on: build
     timeout_in_minutes: 120
-    parallelism: 15
+    parallelism: 10
     retry:
       automatic:
         - exit_status: '-1'

packages/kbn-securitysolution-exception-list-components/src/list_header/menu_items/index.tsx

@@ -77,7 +77,7 @@ const MenuItemsComponent: FC<MenuItemsProps> = ({
       {canUserEditList && (
         <EuiFlexItem>
           <EuiButton
-            data-test-subj={`${dataTestSubj || ''}ManageRulesButton`}
+            data-test-subj={`${dataTestSubj || ''}LinkRulesButton`}
             fill
             onClick={() => {
               if (typeof onManageRules === 'function') onManageRules();

packages/kbn-securitysolution-exception-list-components/src/list_header/menu_items/menu_items.test.tsx

@@ -31,7 +31,7 @@ describe('MenuItems', () => {
     );
     expect(wrapper).toMatchSnapshot();
     expect(wrapper.getByTestId('LinkedRulesMenuItems')).toHaveTextContent('Linked to 1 rules');
-    expect(wrapper.getByTestId('ManageRulesButton')).toBeInTheDocument();
+    expect(wrapper.getByTestId('LinkRulesButton')).toBeInTheDocument();
     expect(wrapper.getByTestId('MenuActionsButtonIcon')).toBeInTheDocument();
   });
   it('should not render linkedRules HeaderMenu component, instead should render a text', () => {
@@ -101,7 +101,7 @@ describe('MenuItems', () => {
       />
     );
     expect(wrapper).toMatchSnapshot();
-    expect(wrapper.queryByTestId('ManageRulesButton')).not.toBeInTheDocument();
+    expect(wrapper.queryByTestId('LinkRulesButton')).not.toBeInTheDocument();
   });
   it('should call onManageRules', () => {
     const wrapper = render(
@@ -115,7 +115,7 @@ describe('MenuItems', () => {
         onManageRules={onManageRules}
       />
     );
-    fireEvent.click(wrapper.getByTestId('ManageRulesButton'));
+    fireEvent.click(wrapper.getByTestId('LinkRulesButton'));
     expect(onManageRules).toHaveBeenCalled();
   });
   it('should call onExportModalOpen', () => {

x-pack/plugins/lists/common/constants.mock.ts

@@ -117,3 +117,6 @@ export const _VERSION = 'WzI5NywxXQ==';
 export const VERSION = 1;
 export const IMMUTABLE = false;
 export const IMPORT_TIMEOUT = moment.duration(5, 'minutes');
+
+/** Added in 8.7 */
+export const EXPIRE_TIME = '2023-04-24T19:00:00.000Z';

x-pack/plugins/lists/common/schemas/request/create_exception_list_item_schema.mock.ts

@@ -11,6 +11,7 @@ import {
   COMMENTS,
   DESCRIPTION,
   ENTRIES,
+  EXPIRE_TIME,
   ITEM_ID,
   ITEM_TYPE,
   LIST_ID,
@@ -60,3 +61,18 @@ export const getCreateExceptionListItemMinimalSchemaMockWithoutId =
     os_types: OS_TYPES,
     type: ITEM_TYPE,
   });
+
+/**
+ * Useful for testing newer exception list item versions, as the previous
+ * versions can be used to test migration cases
+ */
+export const getCreateExceptionListItemNewerVersionSchemaMock =
+  (): CreateExceptionListItemSchema => ({
+    description: DESCRIPTION,
+    entries: ENTRIES,
+    expire_time: EXPIRE_TIME,
+    list_id: LIST_ID,
+    name: NAME,
+    os_types: OS_TYPES,
+    type: ITEM_TYPE,
+  });

x-pack/plugins/lists/common/schemas/request/import_exceptions_schema.mock.ts

@@ -12,7 +12,7 @@ import {
   ImportExceptionsListSchema,
 } from '@kbn/securitysolution-io-ts-list-types';
 
-import { ENTRIES } from '../../constants.mock';
+import { ENTRIES, EXPIRE_TIME } from '../../constants.mock';
 
 export const getImportExceptionsListSchemaMock = (
   listId = 'detection_list_id'
@@ -23,6 +23,10 @@ export const getImportExceptionsListSchemaMock = (
   type: 'detection',
 });
 
+/** 
+  This mock holds the old properties of the Exception List item
+  so that we can test the migration test cases
+*/
 export const getImportExceptionsListItemSchemaMock = (
   itemId = 'item_id_1',
   listId = 'detection_list_id'
@@ -35,6 +39,23 @@ export const getImportExceptionsListItemSchemaMock = (
   type: 'simple',
 });
 
+/**  
+  This mock will hold the new properties of the Exception List item
+  so please keep it updated with the new ones and use it to test the 
+  new scenarios 
+*/
+export const getImportExceptionsListItemNewerVersionSchemaMock = (
+  itemId = 'item_id_1',
+  listId = 'detection_list_id'
+): ImportExceptionListItemSchema => ({
+  description: 'some description',
+  entries: ENTRIES,
+  expire_time: EXPIRE_TIME,
+  item_id: itemId,
+  list_id: listId,
+  name: 'Query with a rule id',
+  type: 'simple',
+});
 export const getImportExceptionsListSchemaDecodedMock = (
   listId = 'detection_list_id'
 ): ImportExceptionListSchemaDecoded => ({