[ML] Fix the limit control on the Anomaly explorer page (#65459)

darnautov · darnautov · commit 528bf32bfd64 · 2020-05-07T11:53:26.000+02:00
* [ML] persist limit control value

* [ML] remove console statement

* [ML] fix default value
diff --git a/x-pack/plugins/ml/public/application/explorer/explorer.js b/x-pack/plugins/ml/public/application/explorer/explorer.js
@@ -14,7 +14,7 @@ import { i18n } from '@kbn/i18n';
 import { FormattedMessage } from '@kbn/i18n/react';
 import DragSelect from 'dragselect/dist/ds.min.js';
 import { Subject } from 'rxjs';
-import { map, takeUntil } from 'rxjs/operators';
+import { takeUntil } from 'rxjs/operators';
 
 import {
   EuiFlexGroup,
@@ -169,12 +169,7 @@ export class Explorer extends React.Component {
   };
 
   componentDidMount() {
-    limit$
-      .pipe(
-        takeUntil(this._unsubscribeAll),
-        map(d => d.val)
-      )
-      .subscribe(explorerService.setSwimlaneLimit);
+    limit$.pipe(takeUntil(this._unsubscribeAll)).subscribe(explorerService.setSwimlaneLimit);
 
     // Required to redraw the time series chart when the container is resized.
     this.resizeChecker = new ResizeChecker(this.resizeRef.current);
diff --git a/x-pack/plugins/ml/public/application/explorer/select_limit/select_limit.test.tsx b/x-pack/plugins/ml/public/application/explorer/select_limit/select_limit.test.tsx
@@ -9,8 +9,6 @@ import { act } from 'react-dom/test-utils';
 import { shallow } from 'enzyme';
 import { SelectLimit } from './select_limit';
 
-jest.useFakeTimers();
-
 describe('SelectLimit', () => {
   test('creates correct initial selected value', () => {
     const wrapper = shallow(<SelectLimit />);
diff --git a/x-pack/plugins/ml/public/application/explorer/select_limit/select_limit.tsx b/x-pack/plugins/ml/public/application/explorer/select_limit/select_limit.tsx
@@ -9,7 +9,7 @@
  */
 import React from 'react';
 import useObservable from 'react-use/lib/useObservable';
-import { Subject } from 'rxjs';
+import { BehaviorSubject } from 'rxjs';
 
 import { EuiSelect } from '@elastic/eui';
 
@@ -20,13 +20,13 @@ const euiOptions = limitOptions.map(limit => ({
   text: `${limit}`,
 }));
 
-export const limit$ = new Subject<number>();
 export const defaultLimit = limitOptions[1];
+export const limit$ = new BehaviorSubject<number>(defaultLimit);
 
 export const useSwimlaneLimit = (): [number, (newLimit: number) => void] => {
   const limit = useObservable(limit$, defaultLimit);
 
-  return [limit, (newLimit: number) => limit$.next(newLimit)];
+  return [limit!, (newLimit: number) => limit$.next(newLimit)];
 };
 
 export const SelectLimit = () => {
