diff --git a/addons/mrp/static/tests/mrp_tests.js b/addons/mrp/static/tests/mrp_tests.js index 3d94c2d1ca3..ed4dab4f91b 100644 --- a/addons/mrp/static/tests/mrp_tests.js +++ b/addons/mrp/static/tests/mrp_tests.js @@ -77,8 +77,8 @@ QUnit.module('mrp', { "there should not be a any visible 'Upload' button"); assert.notOk(form.$('.o_field_widget iframe.o_pdfview_iframe').hasClass('o_hidden'), "there should be an visible iframe"); - assert.strictEqual(form.$('.o_field_widget iframe.o_pdfview_iframe').attr('src'), - '#test:/web/static/lib/pdfjs/web/viewer.html?file=%2Fweb%2Fimage%3Fmodel%3Dpartner%26field%3Ddocument%26id%3D1', + assert.strictEqual(form.$('.o_field_widget iframe.o_pdfview_iframe').attr('data-src'), + '/web/static/lib/pdfjs/web/viewer.html?file=%2Fweb%2Fimage%3Fmodel%3Dpartner%26field%3Ddocument%26id%3D1', "the src attribute should be correctly set on the iframe"); form.destroy(); diff --git a/addons/web/static/tests/helpers/mock_server.js b/addons/web/static/tests/helpers/mock_server.js index 60024292edd..db6d7449a19 100644 --- a/addons/web/static/tests/helpers/mock_server.js +++ b/addons/web/static/tests/helpers/mock_server.js @@ -62,6 +62,9 @@ var MockServer = Class.extend({ * @returns {Object} an object with 2 keys: arch and fields */ fieldsViewGet: function (arch, model, toolbar) { + if (!(model in this.data)) { + throw new Error('Model ' + model + ' was not defined in mock server data'); + } var fields = $.extend(true, {}, this.data[model].fields); var fvg = this._fieldsViewGet(arch, model, fields); var fields_views = {}; @@ -898,7 +901,7 @@ var MockServer = Class.extend({ } var model = this.data[args.model]; if (model && typeof model[args.method] === 'function') { - return this.data[args.model][args.method](args.args, args.kwargs); + return $.when(this.data[args.model][args.method](args.args, args.kwargs)); } console.error("Unimplemented route", route, args); diff --git a/addons/web/static/tests/helpers/test_utils.js b/addons/web/static/tests/helpers/test_utils.js index 516971f96fe..e460b0555b3 100644 --- a/addons/web/static/tests/helpers/test_utils.js +++ b/addons/web/static/tests/helpers/test_utils.js @@ -143,7 +143,7 @@ function createAsyncView(params) { var view = new params.View(viewInfo, viewOptions); // make sure images do not trigger a GET on the server - $('#qunit-fixture').on('DOMNodeInserted.removeSRC', function () { + $target.on('DOMNodeInserted.removeSRC', function () { removeSrcAttribute($(this), widget); }); @@ -436,9 +436,15 @@ function triggerPositionalMouseEvent(x, y, type){ */ function removeSrcAttribute($el, widget) { $el.find('img, iframe[src]').each(function () { - var src = $(this).attr('src'); - if (src[0] !== '#') { - $(this).attr('src', '#test:' + $(this).attr('src')); + var $el = $(this); + var src = $el.attr('src'); + if (src[0] !== '#' && src !== 'about:blank') { + if ($el[0].nodeName === 'IMG') { + $el.attr('src', '#test:' + src); + } else { + $el.attr('data-src', src); + $el.attr('src', 'about:blank'); + } if (widget) { widget._rpc({route: src}); }