[FIX] web : remove useless reload of failed _callButtonAction

Before this fix, when a _callButtonAction failed, it would reload the model.
After this fix we will not reload if a _callButtonAction fails

addons/web/static/src/js/views/basic/basic_controller.js

@@ -254,9 +254,7 @@ var BasicController = AbstractController.extend(FieldManagerMixin, {
                 resIDs: record.res_ids,
             },
             on_success: def.resolve.bind(def),
-            on_fail: function () {
+            on_fail: def.reject.bind(def),
-                reload().always(def.reject.bind(def));
-            },
             on_closed: reload,
         });
         return this.alive(def);

addons/web/static/tests/views/form_tests.js

@@ -1032,7 +1032,7 @@ QUnit.module('Views', {
         });
         form.$('.o_form_statusbar button.s').click();
 
-        assert.strictEqual(rpcCount, 2, "should have done 2 rpcs to reload");
+        assert.strictEqual(rpcCount, 1, "should have done 1 rpc, because we do not reload anymore if the server action fails");
         form.destroy();
     });