[FIX] web: reload list view after button action

Before this commit, the code that handled buttons in list view was kind
of simple, and did not reload the view after.

But clicking on action button is not only a problem for the list view,
the form view already had something in the controller doing exactly
that.  With this commit, we move that code to the basic controller, and
use the method in the list view.

1 files changed, 8 insertions, 1 deletions

diff --git a/addons/web/static/tests/views/list_tests.js b/addons/web/static/tests/views/list_tests.js
index 59a7aa774b8..a3abe59892b 100644
--- a/addons/web/static/tests/views/list_tests.js
+++ b/addons/web/static/tests/views/list_tests.js
@@ -969,7 +969,7 @@ QUnit.module('Views', {
     });
 
     QUnit.test('click on a button in a list view', function (assert) {
-        assert.expect(6);
+        assert.expect(9);
 
         var list = createView({
             View: ListView,
@@ -979,6 +979,10 @@ QUnit.module('Views', {
                     '<field name="foo"/>' +
                     '<button string="a button" name="button_action" icon="fa-car" type="object"/>' +
                 '</tree>',
+            mockRPC: function (route) {
+                assert.step(route);
+                return this._super.apply(this, arguments);
+            },
             intercepts: {
                 execute_action: function (event) {
                     assert.strictEqual(event.data.record_id, 1,
@@ -989,6 +993,7 @@ QUnit.module('Views', {
                         "should call correct method");
                     assert.strictEqual(event.data.action_data.type, 'object',
                         'should have correct type');
+                    event.data.on_closed();
                 },
             },
         });
@@ -999,6 +1004,8 @@ QUnit.module('Views', {
             'buttons should have correct icon');
 
         list.$('.o_list_button:first > button').click(); // click on the button
+        assert.verifySteps(['/web/dataset/search_read', '/web/dataset/search_read'],
+            "should have reloaded the view (after the action is complete)");
         list.destroy();
     });