R Austin Review: more logic in selector

x-pack/plugins/security_solution/public/resolver/store/data/graphing.test.ts

@@ -221,7 +221,7 @@ describe('resolver graph with too much lineage', () => {
   });
 
   describe('should select from state properly', () => {
-    it('has the correct ancestor cursor', () => {
+    it('should indicate there are too many ancestors', () => {
       const action: DataAction = {
         type: 'serverReturnedResolverData',
         payload: {
@@ -232,9 +232,9 @@ describe('resolver graph with too much lineage', () => {
       };
       store.dispatch(action);
       const { ancestors } = limitsReached(store.getState());
-      expect(ancestors).toEqual(ancestorCursor);
+      expect(ancestors).toEqual(true);
     });
-    it('has the correct children cursor', () => {
+    it('should indicate there are too many children', () => {
       const action: DataAction = {
         type: 'serverReturnedResolverData',
         payload: {
@@ -245,7 +245,7 @@ describe('resolver graph with too much lineage', () => {
       };
       store.dispatch(action);
       const { children } = limitsReached(store.getState());
-      expect(children).toEqual(childrenCursor);
+      expect(children).toEqual(true);
     });
   });
 });

x-pack/plugins/security_solution/public/resolver/store/data/selectors.ts

@@ -535,6 +535,9 @@ export const processNodePositionsAndEdgeLineSegments = createSelector(
  *
  * @param state {DataState} the DataState from the reducer
  */
-export const limitsReached = (state: DataState) => {
-  return state.lineageLimits;
+export const limitsReached = (state: DataState): { children: boolean; ancestors: boolean } => {
+  return {
+    children: state.lineageLimits.children !== null,
+    ancestors: state.lineageLimits.ancestors !== null,
+  };
 };