Update tests

getredash · Nov 21, 2019 · 70b9dc8 · 70b9dc8
1 parent 2f9637d
commit 70b9dc8
Show file tree

Hide file tree

Showing 20 changed files with 47 additions and 47 deletions.
diff --git a/client/app/visualizations/chart/Editor/index.test.js b/client/app/visualizations/chart/Editor/index.test.js
@@ -28,12 +28,12 @@ describe('Visualizations -> Chart -> Editor (wrapper)', () => {
   test('Renders generic wrapper', () => {
     const el = mount({ globalSeriesType: 'column' }, { columns: [], rows: [] });
 
-    expect(elementExists(el, 'Chart.EditorTabs.General')).toBeTruthy();
-    expect(elementExists(el, 'Chart.EditorTabs.XAxis')).toBeTruthy();
-    expect(elementExists(el, 'Chart.EditorTabs.YAxis')).toBeTruthy();
-    expect(elementExists(el, 'Chart.EditorTabs.Series')).toBeTruthy();
-    expect(elementExists(el, 'Chart.EditorTabs.Colors')).toBeTruthy();
-    expect(elementExists(el, 'Chart.EditorTabs.DataLabels')).toBeTruthy();
+    expect(elementExists(el, 'VisualizationEditor.Tabs.General')).toBeTruthy();
+    expect(elementExists(el, 'VisualizationEditor.Tabs.XAxis')).toBeTruthy();
+    expect(elementExists(el, 'VisualizationEditor.Tabs.YAxis')).toBeTruthy();
+    expect(elementExists(el, 'VisualizationEditor.Tabs.Series')).toBeTruthy();
+    expect(elementExists(el, 'VisualizationEditor.Tabs.Colors')).toBeTruthy();
+    expect(elementExists(el, 'VisualizationEditor.Tabs.DataLabels')).toBeTruthy();
 
     expect(elementExists(el, 'Chart.GlobalSeriesType')).toBeTruthy(); // general settings block exists
     expect(elementExists(el, 'Chart.Custom.Code')).toBeFalsy(); // custom settings block does not exist
@@ -42,12 +42,12 @@ describe('Visualizations -> Chart -> Editor (wrapper)', () => {
   test('Renders wrapper for custom charts', () => {
     const el = mount({ globalSeriesType: 'custom' }, { columns: [], rows: [] });
 
-    expect(elementExists(el, 'Chart.EditorTabs.General')).toBeTruthy();
-    expect(elementExists(el, 'Chart.EditorTabs.XAxis')).toBeFalsy();
-    expect(elementExists(el, 'Chart.EditorTabs.YAxis')).toBeFalsy();
-    expect(elementExists(el, 'Chart.EditorTabs.Series')).toBeFalsy();
-    expect(elementExists(el, 'Chart.EditorTabs.Colors')).toBeFalsy();
-    expect(elementExists(el, 'Chart.EditorTabs.DataLabels')).toBeFalsy();
+    expect(elementExists(el, 'VisualizationEditor.Tabs.General')).toBeTruthy();
+    expect(elementExists(el, 'VisualizationEditor.Tabs.XAxis')).toBeFalsy();
+    expect(elementExists(el, 'VisualizationEditor.Tabs.YAxis')).toBeFalsy();
+    expect(elementExists(el, 'VisualizationEditor.Tabs.Series')).toBeFalsy();
+    expect(elementExists(el, 'VisualizationEditor.Tabs.Colors')).toBeFalsy();
+    expect(elementExists(el, 'VisualizationEditor.Tabs.DataLabels')).toBeFalsy();
 
     expect(elementExists(el, 'Chart.GlobalSeriesType')).toBeTruthy(); // general settings block exists
     expect(elementExists(el, 'Chart.Custom.Code')).toBeTruthy(); // custom settings block exists

diff --git a/client/app/visualizations/chart/plotly/fixtures/prepareData/bar/default.json b/client/app/visualizations/chart/plotly/fixtures/prepareData/bar/default.json
@@ -48,7 +48,7 @@
         "text": ["10 ± 0", "20 ± 0", "30 ± 0", "40 ± 0"],
         "textposition": "inside",
         "marker": { "color": "red" },
-        "insidetextfont": { "color": "#333333" },
+        "insidetextfont": { "color": "#ffffff" },
         "yaxis": "y"
       }
     ]

diff --git a/client/app/visualizations/chart/plotly/fixtures/prepareData/bar/normalized.json b/client/app/visualizations/chart/plotly/fixtures/prepareData/bar/normalized.json
@@ -58,7 +58,7 @@
         "text": ["20% (10 ± 0)", "40% (20 ± 0)", "60% (30 ± 0)", "80% (40 ± 0)"],
         "textposition": "inside",
         "marker": { "color": "red" },
-        "insidetextfont": { "color": "#333333" },
+        "insidetextfont": { "color": "#ffffff" },
         "yaxis": "y"
       },
       {
@@ -73,7 +73,7 @@
         "text": ["80% (40 ± 0)", "60% (30 ± 0)", "40% (20 ± 0)", "20% (10 ± 0)"],
         "textposition": "inside",
         "marker": { "color": "blue" },
-        "insidetextfont": { "color": "#333333" },
+        "insidetextfont": { "color": "#ffffff" },
         "yaxis": "y"
       }
     ]

diff --git a/client/app/visualizations/chart/plotly/fixtures/prepareData/bar/stacked.json b/client/app/visualizations/chart/plotly/fixtures/prepareData/bar/stacked.json
@@ -58,7 +58,7 @@
         "text": ["10 ± 0", "20 ± 0", "30 ± 0", "40 ± 0"],
         "textposition": "inside",
         "marker": { "color": "red" },
-        "insidetextfont": { "color": "#333333" },
+        "insidetextfont": { "color": "#ffffff" },
         "yaxis": "y"
       },
       {
@@ -73,7 +73,7 @@
         "text": ["1 ± 0", "2 ± 0", "3 ± 0", "4 ± 0"],
         "textposition": "inside",
         "marker": { "color": "blue" },
-        "insidetextfont": { "color": "#333333" },
+        "insidetextfont": { "color": "#ffffff" },
         "yaxis": "y"
       }
     ]

diff --git a/client/app/visualizations/chart/plotly/fixtures/prepareData/box/default.json b/client/app/visualizations/chart/plotly/fixtures/prepareData/box/default.json
@@ -49,7 +49,7 @@
         "error_y": { "array": [0, 0, 0, 0], "color": "red" },
         "hover": [],
         "text": ["10 ± 0", "20 ± 0", "30 ± 0", "40 ± 0"],
-        "insidetextfont": { "color": "#333333" },
+        "insidetextfont": { "color": "#ffffff" },
         "yaxis": "y"
       }
     ]

diff --git a/client/app/visualizations/chart/plotly/fixtures/prepareData/box/with-points.json b/client/app/visualizations/chart/plotly/fixtures/prepareData/box/with-points.json
@@ -52,7 +52,7 @@
         "error_y": { "array": [0, 0, 0, 0], "color": "red" },
         "hover": [],
         "text": ["10 ± 0", "20 ± 0", "30 ± 0", "40 ± 0"],
-        "insidetextfont": { "color": "#333333" },
+        "insidetextfont": { "color": "#ffffff" },
         "yaxis": "y"
       }
     ]

diff --git a/client/app/visualizations/chart/plotly/fixtures/prepareData/bubble/default.json b/client/app/visualizations/chart/plotly/fixtures/prepareData/bubble/default.json
@@ -47,7 +47,7 @@
         "hoverinfo": "text+x+name",
         "hover": [],
         "text": ["10 ± 0: 51", "20 ± 0: 52", "30 ± 0: 53", "40 ± 0: 54"],
-        "insidetextfont": { "color": "#333333" },
+        "insidetextfont": { "color": "#ffffff" },
         "yaxis": "y"
       }
     ]

diff --git a/client/app/visualizations/chart/plotly/fixtures/prepareData/line-area/default.json b/client/app/visualizations/chart/plotly/fixtures/prepareData/line-area/default.json
@@ -47,7 +47,7 @@
         "hover": [],
         "text": ["10 ± 0", "20 ± 0", "30 ± 0", "40 ± 0"],
         "marker": { "color": "red" },
-        "insidetextfont": { "color": "#333333" },
+        "insidetextfont": { "color": "#ffffff" },
         "yaxis": "y"
       }
     ]

diff --git a/...t/app/visualizations/chart/plotly/fixtures/prepareData/line-area/keep-missing-values.json b/...t/app/visualizations/chart/plotly/fixtures/prepareData/line-area/keep-missing-values.json
@@ -55,7 +55,7 @@
         "hover": [],
         "text": ["10 ± 0", "20 ± 0", "30 ± 0", "40 ± 0"],
         "marker": { "color": "red" },
-        "insidetextfont": { "color": "#333333" },
+        "insidetextfont": { "color": "#ffffff" },
         "yaxis": "y"
       },
       {
@@ -69,7 +69,7 @@
         "hover": [],
         "text": ["", "2 ± 0", "", "4 ± 0"],
         "marker": { "color": "blue" },
-        "insidetextfont": { "color": "#333333" },
+        "insidetextfont": { "color": "#ffffff" },
         "yaxis": "y"
       }
     ]

diff --git a/client/app/visualizations/chart/plotly/fixtures/prepareData/line-area/missing-values-0.json b/client/app/visualizations/chart/plotly/fixtures/prepareData/line-area/missing-values-0.json
@@ -55,7 +55,7 @@
         "hover": [],
         "text": ["10 ± 0", "20 ± 0", "30 ± 0", "40 ± 0"],
         "marker": { "color": "red" },
-        "insidetextfont": { "color": "#333333" },
+        "insidetextfont": { "color": "#ffffff" },
         "yaxis": "y"
       },
       {
@@ -69,7 +69,7 @@
         "hover": [],
         "text": ["0", "2 ± 0", "0", "4 ± 0"],
         "marker": { "color": "blue" },
-        "insidetextfont": { "color": "#333333" },
+        "insidetextfont": { "color": "#ffffff" },
         "yaxis": "y"
       }
     ]

diff --git a/...nt/app/visualizations/chart/plotly/fixtures/prepareData/line-area/normalized-stacked.json b/...nt/app/visualizations/chart/plotly/fixtures/prepareData/line-area/normalized-stacked.json
@@ -57,7 +57,7 @@
         "hover": [],
         "text": ["20% (10 ± 0)", "40% (20 ± 0)", "60% (30 ± 0)", "80% (40 ± 0)"],
         "marker": { "color": "red" },
-        "insidetextfont": { "color": "#333333" },
+        "insidetextfont": { "color": "#ffffff" },
         "yaxis": "y"
       },
       {
@@ -71,7 +71,7 @@
         "hover": [],
         "text": ["80% (40 ± 0)", "60% (30 ± 0)", "40% (20 ± 0)", "20% (10 ± 0)"],
         "marker": { "color": "blue" },
-        "insidetextfont": { "color": "#333333" },
+        "insidetextfont": { "color": "#ffffff" },
         "yaxis": "y"
       }
     ]

diff --git a/client/app/visualizations/chart/plotly/fixtures/prepareData/line-area/normalized.json b/client/app/visualizations/chart/plotly/fixtures/prepareData/line-area/normalized.json
@@ -57,7 +57,7 @@
         "hover": [],
         "text": ["20% (10 ± 0)", "40% (20 ± 0)", "60% (30 ± 0)", "80% (40 ± 0)"],
         "marker": { "color": "red" },
-        "insidetextfont": { "color": "#333333" },
+        "insidetextfont": { "color": "#ffffff" },
         "yaxis": "y"
       },
       {
@@ -71,7 +71,7 @@
         "hover": [],
         "text": ["80% (40 ± 0)", "60% (30 ± 0)", "40% (20 ± 0)", "20% (10 ± 0)"],
         "marker": { "color": "blue" },
-        "insidetextfont": { "color": "#333333" },
+        "insidetextfont": { "color": "#ffffff" },
         "yaxis": "y"
       }
     ]

diff --git a/client/app/visualizations/chart/plotly/fixtures/prepareData/line-area/stacked.json b/client/app/visualizations/chart/plotly/fixtures/prepareData/line-area/stacked.json
@@ -57,7 +57,7 @@
         "hover": [],
         "text": ["10 ± 0", "20 ± 0", "30 ± 0", "40 ± 0"],
         "marker": { "color": "red" },
-        "insidetextfont": { "color": "#333333" },
+        "insidetextfont": { "color": "#ffffff" },
         "yaxis": "y"
       },
       {
@@ -71,7 +71,7 @@
         "hover": [],
         "text": ["1 ± 0", "2 ± 0", "3 ± 0", "4 ± 0"],
         "marker": { "color": "blue" },
-        "insidetextfont": { "color": "#333333" },
+        "insidetextfont": { "color": "#ffffff" },
         "yaxis": "y"
       }
     ]

diff --git a/client/app/visualizations/chart/plotly/fixtures/prepareData/scatter/default.json b/client/app/visualizations/chart/plotly/fixtures/prepareData/scatter/default.json
@@ -48,7 +48,7 @@
         "hover": [],
         "text": ["10 ± 0", "20 ± 0", "30 ± 0", "40 ± 0"],
         "marker": { "color": "red" },
-        "insidetextfont": { "color": "#333333" },
+        "insidetextfont": { "color": "#ffffff" },
         "yaxis": "y"
       }
     ]

diff --git a/client/app/visualizations/chart/plotly/fixtures/prepareData/scatter/without-labels.json b/client/app/visualizations/chart/plotly/fixtures/prepareData/scatter/without-labels.json
@@ -48,7 +48,7 @@
         "hover": [],
         "text": ["10 ± 0", "20 ± 0", "30 ± 0", "40 ± 0"],
         "marker": { "color": "red" },
-        "insidetextfont": { "color": "#333333" },
+        "insidetextfont": { "color": "#ffffff" },
         "yaxis": "y"
       }
     ]

diff --git a/client/cypress/integration/visualizations/choropleth_spec.js b/client/cypress/integration/visualizations/choropleth_spec.js
@@ -48,7 +48,7 @@ describe('Choropleth', () => {
     `);
 
     cy.clickThrough(`
-      Choropleth.EditorTabs.General
+      VisualizationEditor.Tabs.General
       Choropleth.Editor.MapType
       Choropleth.Editor.MapType.Countries
       Choropleth.Editor.KeyColumn
@@ -59,7 +59,7 @@ describe('Choropleth', () => {
       Choropleth.Editor.ValueColumn.value
     `);
 
-    cy.clickThrough('Choropleth.EditorTabs.Colors');
+    cy.clickThrough('VisualizationEditor.Tabs.Colors');
     cy.clickThrough('Choropleth.Editor.Colors.Min');
     cy.fillInputs({ 'ColorPicker.CustomColor': 'yellow{enter}' });
     cy.wait(100); // eslint-disable-line cypress/no-unnecessary-waiting
@@ -71,7 +71,7 @@ describe('Choropleth', () => {
     cy.wait(100); // eslint-disable-line cypress/no-unnecessary-waiting
 
     cy.clickThrough(`
-      Choropleth.EditorTabs.Format
+      VisualizationEditor.Tabs.Format
       Choropleth.Editor.LegendPosition
       Choropleth.Editor.LegendPosition.TopRight
     `);

diff --git a/client/cypress/integration/visualizations/cohort_spec.js b/client/cypress/integration/visualizations/cohort_spec.js
@@ -38,13 +38,13 @@ describe('Cohort', () => {
     `);
 
     cy.clickThrough(`
-      Cohort.EditorTabs.Options
+      VisualizationEditor.Tabs.Options
       Cohort.TimeInterval
       Cohort.TimeInterval.monthly
       Cohort.Mode
       Cohort.Mode.simple
 
-      Cohort.EditorTabs.Columns
+      VisualizationEditor.Tabs.Columns
       Cohort.DateColumn
       Cohort.DateColumn.date
       Cohort.StageColumn
@@ -61,7 +61,7 @@ describe('Cohort', () => {
     cy.percySnapshot('Visualizations - Cohort (simple)', { widths: [viewportWidth] });
 
     cy.clickThrough(`
-      Cohort.EditorTabs.Options
+      VisualizationEditor.Tabs.Options
       Cohort.Mode
       Cohort.Mode.diagonal
     `);

diff --git a/client/cypress/integration/visualizations/counter_spec.js b/client/cypress/integration/visualizations/counter_spec.js
@@ -156,7 +156,7 @@ describe('Counter', () => {
       Counter.General.TargetValueColumn
       Counter.General.TargetValueColumn.b
 
-      Counter.EditorTabs.Formatting
+      VisualizationEditor.Tabs.Format
     `);
 
     cy.fillInputs({
@@ -186,7 +186,7 @@ describe('Counter', () => {
       Counter.General.TargetValueColumn
       Counter.General.TargetValueColumn.b
 
-      Counter.EditorTabs.Formatting
+      VisualizationEditor.Tabs.Format
       Counter.Formatting.FormatTargetValue
     `);
 

diff --git a/client/cypress/integration/visualizations/funnel_spec.js b/client/cypress/integration/visualizations/funnel_spec.js
@@ -39,7 +39,7 @@ describe('Funnel', () => {
     `);
 
     cy.clickThrough(`
-      Funnel.EditorTabs.General
+      VisualizationEditor.Tabs.General
 
       Funnel.StepColumn
       Funnel.StepColumn.a
@@ -64,7 +64,7 @@ describe('Funnel', () => {
     cy.percySnapshot('Visualizations - Funnel (basic)', { widths: [viewportWidth] });
 
     cy.clickThrough(`
-      Funnel.EditorTabs.Appearance
+      VisualizationEditor.Tabs.Appearance
     `);
 
     cy.fillInputs({

diff --git a/client/cypress/integration/visualizations/map_spec.js b/client/cypress/integration/visualizations/map_spec.js
@@ -33,7 +33,7 @@ describe('Map (Markers)', () => {
     `);
 
     cy.clickThrough(`
-      Map.EditorTabs.General
+      VisualizationEditor.Tabs.General
       Map.Editor.LatitudeColumnName
       Map.Editor.LatitudeColumnName.lat
       Map.Editor.LongitudeColumnName
@@ -42,7 +42,7 @@ describe('Map (Markers)', () => {
       Map.Editor.GroupBy.country
     `);
 
-    cy.clickThrough('Map.EditorTabs.Groups');
+    cy.clickThrough('VisualizationEditor.Tabs.Groups');
     cy.clickThrough('Map.Editor.Groups.Israel.Color');
     cy.fillInputs({ 'ColorPicker.CustomColor': 'red{enter}' });
     cy.wait(100); // eslint-disable-line cypress/no-unnecessary-waiting
@@ -67,15 +67,15 @@ describe('Map (Markers)', () => {
     `);
 
     cy.clickThrough(`
-      Map.EditorTabs.General
+      VisualizationEditor.Tabs.General
       Map.Editor.LatitudeColumnName
       Map.Editor.LatitudeColumnName.lat
       Map.Editor.LongitudeColumnName
       Map.Editor.LongitudeColumnName.lng
     `);
 
     cy.clickThrough(`
-      Map.EditorTabs.Style
+      VisualizationEditor.Tabs.Style
       Map.Editor.ClusterMarkers
       Map.Editor.CustomizeMarkers
     `);