Resolve "Monitoring graphs - Popover value improvements"

4 files changed, 19 insertions, 19 deletions

diff --git a/spec/javascripts/monitoring/graph/flag_spec.js b/spec/javascripts/monitoring/graph/flag_spec.js
index 2d474e9092f..19278312b6d 100644
--- a/spec/javascripts/monitoring/graph/flag_spec.js
+++ b/spec/javascripts/monitoring/graph/flag_spec.js
@@ -22,15 +22,20 @@ const defaultValuesComponent = {
   graphHeightOffset: 120,
   showFlagContent: true,
   realPixelRatio: 1,
-  timeSeries: [{
-    values: [{
-      time: new Date('2017-06-04T18:17:33.501Z'),
-      value: '1.49609375',
-    }],
-  }],
+  timeSeries: [
+    {
+      values: [
+        {
+          time: new Date('2017-06-04T18:17:33.501Z'),
+          value: '1.49609375',
+        },
+      ],
+    },
+  ],
   unitOfDisplay: 'ms',
   currentDataIndex: 0,
   legendTitle: 'Average',
+  currentCoordinates: [],
 };
 
 const deploymentFlagData = {
@@ -113,7 +118,7 @@ describe('GraphFlag', () => {
     });
 
     it('formatDate', () => {
-      expect(component.formatDate).toEqual('Sun, Jun 4');
+      expect(component.formatDate).toEqual('04 Jun 2017, ');
     });
 
     it('cursorStyle', () => {
diff --git a/spec/javascripts/monitoring/graph/track_line_spec.js b/spec/javascripts/monitoring/graph/track_line_spec.js
index 45106830a67..27602a861eb 100644
--- a/spec/javascripts/monitoring/graph/track_line_spec.js
+++ b/spec/javascripts/monitoring/graph/track_line_spec.js
@@ -39,14 +39,14 @@ describe('TrackLine component', () => {
       const svgEl = vm.$el.querySelector('svg');
       const lineEl = vm.$el.querySelector('svg line');
 
-      expect(svgEl.getAttribute('width')).toEqual('15');
-      expect(svgEl.getAttribute('height')).toEqual('6');
+      expect(svgEl.getAttribute('width')).toEqual('16');
+      expect(svgEl.getAttribute('height')).toEqual('8');
 
       expect(lineEl.getAttribute('stroke-width')).toEqual('4');
       expect(lineEl.getAttribute('x1')).toEqual('0');
-      expect(lineEl.getAttribute('x2')).toEqual('15');
-      expect(lineEl.getAttribute('y1')).toEqual('2');
-      expect(lineEl.getAttribute('y2')).toEqual('2');
+      expect(lineEl.getAttribute('x2')).toEqual('16');
+      expect(lineEl.getAttribute('y1')).toEqual('4');
+      expect(lineEl.getAttribute('y2')).toEqual('4');
     });
   });
 });
diff --git a/spec/javascripts/monitoring/graph_path_spec.js b/spec/javascripts/monitoring/graph_path_spec.js
index c83bd19345f..2515e2ad897 100644
--- a/spec/javascripts/monitoring/graph_path_spec.js
+++ b/spec/javascripts/monitoring/graph_path_spec.js
@@ -23,6 +23,7 @@ describe('Monitoring Paths', () => {
       generatedAreaPath: firstTimeSeries.areaPath,
       lineColor: firstTimeSeries.lineColor,
       areaColor: firstTimeSeries.areaColor,
+      showDot: false,
     });
     const metricArea = component.$el.querySelector('.metric-area');
     const metricLine = component.$el.querySelector('.metric-line');
@@ -40,6 +41,7 @@ describe('Monitoring Paths', () => {
         generatedAreaPath: firstTimeSeries.areaPath,
         lineColor: firstTimeSeries.lineColor,
         areaColor: firstTimeSeries.areaColor,
+        showDot: false,
       });
 
       component.lineStyle = 'dashed';
diff --git a/spec/javascripts/monitoring/graph_spec.js b/spec/javascripts/monitoring/graph_spec.js
index 1213c80ba3a..220228e5c08 100644
--- a/spec/javascripts/monitoring/graph_spec.js
+++ b/spec/javascripts/monitoring/graph_spec.js
@@ -30,7 +30,6 @@ describe('Graph', () => {
   it('has a title', () => {
     const component = createComponent({
       graphData: convertedMetrics[1],
-      classType: 'col-md-6',
       updateAspectRatio: false,
       deploymentData,
       tagsPath,
@@ -46,7 +45,6 @@ describe('Graph', () => {
     it('axisTransform translates an element Y position depending of its height', () => {
       const component = createComponent({
         graphData: convertedMetrics[1],
-        classType: 'col-md-6',
         updateAspectRatio: false,
         deploymentData,
         tagsPath,
@@ -62,7 +60,6 @@ describe('Graph', () => {
     it('outerViewBox gets a width and height property based on the DOM size of the element', () => {
       const component = createComponent({
         graphData: convertedMetrics[1],
-        classType: 'col-md-6',
         updateAspectRatio: false,
         deploymentData,
         tagsPath,
@@ -79,7 +76,6 @@ describe('Graph', () => {
   it('sends an event to the eventhub when it has finished resizing', done => {
     const component = createComponent({
       graphData: convertedMetrics[1],
-      classType: 'col-md-6',
       updateAspectRatio: false,
       deploymentData,
       tagsPath,
@@ -97,7 +93,6 @@ describe('Graph', () => {
   it('has a title for the y-axis and the chart legend that comes from the backend', () => {
     const component = createComponent({
       graphData: convertedMetrics[1],
-      classType: 'col-md-6',
       updateAspectRatio: false,
       deploymentData,
       tagsPath,
@@ -111,7 +106,6 @@ describe('Graph', () => {
   it('sets the currentData object based on the hovered data index', () => {
     const component = createComponent({
       graphData: convertedMetrics[1],
-      classType: 'col-md-6',
       updateAspectRatio: false,
       deploymentData,
       graphIdentifier: 0,
@@ -125,6 +119,5 @@ describe('Graph', () => {
 
     component.positionFlag();
     expect(component.currentData).toBe(component.timeSeries[0].values[10]);
-    expect(component.currentDataIndex).toEqual(10);
   });
 });