review comments

x-pack/plugins/security_solution/public/resolver/data_access_layer/mocks/one_ancestor_two_children.ts

@@ -44,9 +44,7 @@ interface Metadata {
  * A simple mock dataAccessLayer possible that returns a tree with 0 ancestors and 2 direct children. 1 related event is returned. The parameter to `entities` is ignored.
  */
 export function oneAncestorTwoChildren(
- {
- withRelatedEvents,
- }: { withRelatedEvents: Parameters<typeof withRelatedEventsOnOrigin>[1] | null } = {
+ { withRelatedEvents }: { withRelatedEvents: Iterable<[string, string]> | null } = {
  withRelatedEvents: null,
  }
 ): { dataAccessLayer: DataAccessLayer; metadata: Metadata } {

x-pack/plugins/security_solution/public/resolver/store/mocks/related_event.ts

@@ -3,8 +3,6 @@
  * or more contributor license agreements. Licensed under the Elastic License;
  * you may not use this file except in compliance with the Elastic License.
  */
-
-import uuid from 'uuid';
 import { EndpointEvent } from '../../../../common/endpoint/types';
 
 /**
@@ -15,19 +13,21 @@ export function mockRelatedEvent({
  timestamp,
  category,
  type,
+ id,
 }: {
  entityID: string;
  timestamp: number;
  category: string;
  type: string;
+ id?: string;
 }): EndpointEvent {
  return {
  '@timestamp': timestamp,
  event: {
  kind: 'event',
  type,
  category,
- id: uuid.v4(),
+ id: id ?? 'xyz',
  },
  process: {
  entity_id: entityID,

x-pack/plugins/security_solution/public/resolver/test_utilities/simulator/index.tsx

@@ -220,15 +220,9 @@ export class Simulator {
  );
  }
 
- /**
- * For a render of the Enzyme wrapper (may help update if other concerns have effected the render). May be useful for debugging.
- */
- public updateWrapper() {
- return this.wrapper.update();
- }
-
  /**
  * Dump all contents of the outer ReactWrapper (to be `console.log`ged as appropriate)
+ * This will include both DOM (div, span, etc.) and React/JSX (MyComponent, MyGrid, etc.)
  */
  public debugWrapper() {
  return this.wrapper.debug();
@@ -242,6 +236,7 @@ export class Simulator {
  public processNodeRelatedEventButton(entityID: string): ReactWrapper {
  return this.processNodeElements({ entityID }).findWhere(
  (wrapper) =>
+ // Filter out React components
  typeof wrapper.type() === 'string' &&
  wrapper.prop('data-test-subj') === 'resolver:submenu:button'
  );