[IMP] web: graph view improvements

This commit extends the support of comparison mode for the graph view
and refine the tooltips and rendering in line mode.
The graph view code has also been slightly refactored.

Task ID: 1946138

1 files changed, 8 insertions, 12 deletions

diff --git a/addons/web/static/tests/views/graph_tests.js b/addons/web/static/tests/views/graph_tests.js
index cae561e4889..7a0ff7d5846 100644
--- a/addons/web/static/tests/views/graph_tests.js
+++ b/addons/web/static/tests/views/graph_tests.js
@@ -83,10 +83,10 @@ QUnit.module('Views', {
             // not so easy, because there is an animation which means that we
             // don't really have a nice way to find the proper rect elements.
             // So, instead we will do some white box testing.
-            assert.strictEqual(graph.model.chart.data[0].value, 3,
+            assert.strictEqual(graph.model.chart.dataPoints[0].value, 3,
                 "should have first datapoint with value 3");
-            assert.strictEqual(graph.model.chart.data[1].value, 5,
-                "should have first datapoint with value 5");
+            assert.strictEqual(graph.model.chart.dataPoints[1].value, 5,
+                "should have second datapoint with value 5");
             graph.destroy();
             done();
         });
@@ -391,7 +391,6 @@ QUnit.module('Views', {
                     graph_mode: 'bar',
                     graph_measure: '__count__',
                     graph_groupbys: ['product_id'],
-                    graph_intervalMapping: {},
                 }
             }, "context should be correct");
 
@@ -405,7 +404,6 @@ QUnit.module('Views', {
                     graph_mode: 'bar',
                     graph_measure: 'foo',
                     graph_groupbys: ['product_id'],
-                    graph_intervalMapping: {},
                 },
             }, "context should be correct");
 
@@ -418,7 +416,6 @@ QUnit.module('Views', {
                     graph_mode: 'line',
                     graph_measure: 'foo',
                     graph_groupbys: ['product_id'],
-                    graph_intervalMapping: {},
                 },
             }, "context should be correct");
 
@@ -431,7 +428,6 @@ QUnit.module('Views', {
                     graph_mode: 'line',
                     graph_measure: 'foo',
                     graph_groupbys: ['product_id', 'color_id'],
-                    graph_intervalMapping: {},
                 },
             }, "context should be correct");
 
@@ -642,7 +638,7 @@ QUnit.module('Views', {
 
             assert.strictEqual(graph.renderer.state.mode, "bar",
                 "should be in bar chart mode by default");
-            assert.strictEqual(graph.model.chart.data[0].value, 2,
+            assert.strictEqual(graph.model.chart.dataPoints[0].value, 2,
                 "should have a datapoint with value 2");
             graph.destroy();
             done();
@@ -668,9 +664,9 @@ QUnit.module('Views', {
 
             assert.strictEqual(graph.renderer.state.mode, "bar",
                 "should be in bar chart mode by default");
-            assert.strictEqual(graph.model.chart.data[0].value, 1,
+            assert.strictEqual(graph.model.chart.dataPoints[0].value, 1,
                 "should have first datapoint with value 2");
-            assert.strictEqual(graph.model.chart.data[1].value, 2,
+            assert.strictEqual(graph.model.chart.dataPoints[1].value, 2,
                 "should have second datapoint with value 2");
             graph.destroy();
             done();
@@ -697,9 +693,9 @@ QUnit.module('Views', {
         await testUtils.dom.click(graph.$buttons.find('.dropdown-toggle:contains(Measures)'));
         await testUtils.dom.click(graph.$buttons.find('.dropdown-item[data-field="product_id"]'));
 
-        assert.strictEqual(graph.model.chart.data[0].value, 1,
+        assert.strictEqual(graph.model.chart.dataPoints[0].value, 1,
             "should have first datapoint with value 1");
-        assert.strictEqual(graph.model.chart.data[1].value, 1,
+        assert.strictEqual(graph.model.chart.dataPoints[1].value, 1,
             "should have second datapoint with value 1");
 
         graph.destroy();