pengyuthon/odoo_source
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

[FIX] web_editor: change oe-id when switching record

Browse Source 
Add a test to ensure that the `MediaDialog` is opened with the correct record
`resId` (to properly link files to that record) inside an html field.

task-3502797

closes odoo/odoo#138525

X-original-commit: 5184bed2efd9923c0371d5e117a7915415ceee5a
Signed-off-by: David Monjoie (dmo) <dmo@odoo.com>
Signed-off-by: Thibault Delavallee (tde) <tde@openerp.com>
This commit is contained in:
abd-msyukyu-odoo
2023-10-12 17:20:46 +00:00
parent 0552322774
commit 5a2ee50798
1 changed files with 59 additions and 1 deletions
Download Patch File Download Diff File Expand all files Collapse all files
addons/web_editor/static/tests/html_field_tests.js
+59 -1
View File
@@ -4,7 +4,8 @@ import { click, editInput, getFixture, makeDeferred, nextTick, patchWithCleanup
import { makeView, setupViewRegistries } from "@web/../tests/views/helpers";
import { FormController } from '@web/views/form/form_controller';
import { HtmlField } from "@web_editor/js/backend/html_field";
import { parseHTML } from "@web_editor/js/editor/odoo-editor/src/utils/utils";
import { MediaDialog } from "@web_editor/components/media_dialog/media_dialog";
import { parseHTML, setSelection } from "@web_editor/js/editor/odoo-editor/src/utils/utils";
import { onRendered } from "@odoo/owl";
import { wysiwygData } from "@web_editor/../tests/test_utils";
import { OdooEditor } from '@web_editor/js/editor/odoo-editor/src/OdooEditor';
@@ -44,6 +45,63 @@ QUnit.module("WebEditor.HtmlField", ({ beforeEach }) => {
setupViewRegistries();
});
QUnit.module("Form view interactions with the HtmlField");
QUnit.test("A new MediaDialog after switching record in a Form view should have the correct resId", async (assert) => {
serverData.models.partner.records = [
{id: 1, txt: "<p>first</p>"},
{id: 2, txt: "<p>second</p>"},
];
let wysiwyg, mediaDialog;
const wysiwygPromise = makeDeferred();
const mediaDialogPromise = makeDeferred();
patchWithCleanup(HtmlField.prototype, {
async startWysiwyg() {
await super.startWysiwyg(...arguments);
wysiwyg = this.wysiwyg;
wysiwygPromise.resolve();
}
});
patchWithCleanup(MediaDialog.prototype, {
setup() {
mediaDialog = this;
mediaDialogPromise.resolve();
this.size = 'xl';
this.contentClass = 'o_select_media_dialog';
this.title = "TEST";
this.tabs = [];
this.state = {};
// no call to super to avoid services dependencies
// this test only cares about the props given to the dialog
}
});
await makeView({
type: "form",
resId: 1,
resIds: [1, 2],
resModel: "partner",
serverData,
arch: `
<form>
<field name="txt" widget="html"/>
</form>`,
});
await wysiwygPromise;
assert.containsOnce(target, ".odoo-editor-editable p:contains(first)");
// click on the pager to switch to the next record
await click(target.querySelector(".o_pager_next"));
assert.containsOnce(target, ".odoo-editor-editable p:contains(second)");
const paragraph = target.querySelector(".odoo-editor-editable p");
setSelection(paragraph, 0, paragraph, 0);
wysiwyg.openMediaDialog();
await mediaDialogPromise;
assert.equal(mediaDialog.props.resId, 2);
});
QUnit.module('Sandboxed Preview');