Feature/update vertical domain

cypress/integration/1_detectors.spec.js

@@ -5,6 +5,7 @@
 
 import { OPENSEARCH_DASHBOARDS_URL } from '../support/constants';
 import sample_index_settings from '../fixtures/sample_index_settings.json';
+import { recurse } from 'cypress-recurse';
 
 const testMappings = {
   properties: {
@@ -227,10 +228,16 @@ describe('Detectors', () => {
     });
 
     // Change detector name
-    cy.get(`input[placeholder="Enter a name for the detector."]`)
-      .realClick()
-      .ospClear()
-      .realType('test detector edited');
+    const detectorNameText = 'test detector edited';
+    const detectorNameSelector = 'input[placeholder="Enter a name for the detector."]';
+    cy.get(detectorNameSelector).should('have.value', 'test detector');
+
+    recurse(
+      () => cy.get(detectorNameSelector).clear().type(detectorNameText),
+      ($input) => $input.val() === detectorNameText
+    ).then(() => {
+      cy.get(detectorNameSelector).should('have.value', 'test detector edited');
+    });
 
     // Change detector description
     cy.get(`[data-test-subj="define-detector-detector-description"]`)

package.json

@@ -55,14 +55,15 @@
     "@types/react-dom": "^16.9.8",
     "@types/react-router-dom": "^5.3.2",
     "cypress": "^6.0.0",
+    "cypress-real-events": "1.7.6",
+    "cypress-recurse": "^1.27.0",
     "eslint-plugin-no-unsanitized": "^3.0.2",
     "eslint-plugin-prefer-object-spread": "^1.2.1",
     "husky": "^3.0.0",
     "jest-cli": "^27.5.1",
     "jest-environment-jsdom": "^27.5.1",
     "lint-staged": "^9.2.0",
-    "ts-loader": "^6.2.1",
-    "cypress-real-events": "1.7.6"
+    "ts-loader": "^6.2.1"
   },
   "engines": {
     "yarn": "^1.21.1"

public/pages/Overview/utils/__snapshots__/helper.test.ts.snap

@@ -335,6 +335,12 @@ Object {
             "grid": true,
           },
           "field": "finding",
+          "scale": Object {
+            "domain": Array [
+              0,
+              0.1,
+            ],
+          },
           "title": "Count",
           "type": "quantitative",
         },

public/pages/Overview/utils/helpers.ts

@@ -10,6 +10,7 @@ import dateMath from '@elastic/datemath';
 import _ from 'lodash';
 import { DEFAULT_DATE_RANGE } from '../../../utils/constants';
 import { severityOptions } from '../../Alerts/utils/constants';
+import moment from 'moment';
 
 export interface TimeUnit {
   unit: string;
@@ -67,13 +68,14 @@ export const defaultScaleDomain = [
   parseDateString(DEFAULT_DATE_RANGE.end),
 ];
 
-export const getYAxis = (field: string, title: string, axisGrid: boolean = true) => ({
-  aggregate: 'sum',
-  field: field,
-  type: 'quantitative',
-  title: title,
-  axis: { grid: axisGrid },
-});
+export const getYAxis = (field: string, title: string, axisGrid: boolean = true, opts: any = {}) =>
+  _.defaultsDeep(opts, {
+    aggregate: 'sum',
+    field: field,
+    type: 'quantitative',
+    title: title,
+    axis: { grid: axisGrid },
+  });
 
 export const getXAxis = (dateOpts: DateOpts, opts: any = {}) =>
   _.defaultsDeep(opts, {
@@ -121,6 +123,21 @@ export function getVisualizationSpec(description: string, data: any, layers: any
   };
 }
 
+export const getYDomainRange = (data: any[], timeUnit: string): number[] => {
+  data = data.filter((item) => item.finding === 1);
+
+  let dateFormat = 'mm';
+  const timeUnitSize = timeUnit.match(/.*(seconds|minutes|hours|date|month|year)$/);
+  if (timeUnitSize) {
+    if (timeUnitSize && timeUnitSize[1]) {
+      dateFormat = `${timeUnitSize[1][0]}${timeUnitSize[1][0]}`;
+    }
+  }
+  let dataGroups = _.groupBy(data, (item) => moment(item.time).format(dateFormat));
+  const domainMax = _.maxBy(Object.values(dataGroups), (group) => group.length) || [];
+  return [0, domainMax.length + 0.1];
+};
+
 export function getOverviewVisualizationSpec(
   visualizationData: SummaryData[],
   groupBy: string,
@@ -132,7 +149,11 @@ export function getOverviewVisualizationSpec(
 ): TopLevelSpec {
   const findingsEncoding: { [x: string]: any } = {
     x: getXAxis(dateOpts),
-    y: getYAxis('finding', 'Count'),
+    y: getYAxis('finding', 'Count', true, {
+      scale: {
+        domain: getYDomainRange(visualizationData, dateOpts.timeUnit.unit),
+      },
+    }),
     tooltip: [getYAxis('finding', 'Findings'), getTimeTooltip(dateOpts)],
     color: {
       field: 'fieldType',

yarn.lock

@@ -2046,6 +2046,13 @@ [email protected]:
   resolved "https://registry.yarnpkg.com/cypress-real-events/-/cypress-real-events-1.7.6.tgz#6f17e0b2ceea1d6dc60f6737d8f84cc517bbbb4c"
   integrity sha512-yP6GnRrbm6HK5q4DH6Nnupz37nOfZu/xn1xFYqsE2o4G73giPWQOdu6375QYpwfU1cvHNCgyD2bQ2hPH9D7NMw==
 
+cypress-recurse@^1.27.0:
+  version "1.27.0"
+  resolved "https://registry.yarnpkg.com/cypress-recurse/-/cypress-recurse-1.27.0.tgz#0c61e809c5f7740a7e907714614c49c72dcb5c1f"
+  integrity sha512-BCD83UqaxlD+JiqZn1PvIhHRXasgfCt57vLC1Fcyifvxh4QklELRcYUJV3MdhKamMkmajaErLfnCNbZ8VJ5SIg==
+  dependencies:
+    humanize-duration "^3.27.3"
+
 cypress@^6.0.0:
   version "6.9.1"
   resolved "https://registry.yarnpkg.com/cypress/-/cypress-6.9.1.tgz#ce1106bfdc47f8d76381dba63f943447883f864c"
@@ -3143,6 +3150,11 @@ human-signals@^2.1.0:
   resolved "https://registry.yarnpkg.com/human-signals/-/human-signals-2.1.0.tgz#dc91fcba42e4d06e4abaed33b3e7a3c02f514ea0"
   integrity sha512-B4FFZ6q/T2jhhksgkbEW3HBvWIfDW85snkQgawt07S7J5QXTk6BkNV+0yAeZrM5QpMAdYlocGoljn0sJ/WQkFw==
 
+humanize-duration@^3.27.3:
+  version "3.28.0"
+  resolved "https://registry.yarnpkg.com/humanize-duration/-/humanize-duration-3.28.0.tgz#f79770c0bec34d3bfd4899338cc40643bc04df72"
+  integrity sha512-jMAxraOOmHuPbffLVDKkEKi/NeG8dMqP8lGRd6Tbf7JgAeG33jjgPWDbXXU7ypCI0o+oNKJFgbSB9FKVdWNI2A==
+
 husky@^3.0.0:
   version "3.1.0"
   resolved "https://registry.yarnpkg.com/husky/-/husky-3.1.0.tgz#5faad520ab860582ed94f0c1a77f0f04c90b57c0"