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

[IMP] mail, *: tests: use contains in click and insertText

Browse Source 
* = account, calendar, im_livechat, mrp, project_todo, sms, snailmail,
    test_mail, web, website_livechat

`contains` is more efficient than `afterNextRender` as it does not wait
for several extra animation frames, and it is functionally more
meaningful.

closes odoo/odoo#130451

Related: odoo/enterprise#44999
Signed-off-by: Alexandre Kühn (aku) <aku@odoo.com>
This commit is contained in:
Sébastien Theys
2023-08-21 21:00:03 +02:00
parent 9a1efcbde7
commit d08a18b791
97 changed files with 4661 additions and 5123 deletions
Download Patch File Download Diff File Expand all files Collapse all files
addons/sms/static/tests/thread/message_patch_test.js
+15 -14
View File
@@ -1,6 +1,7 @@
/** @odoo-module **/
/* @odoo-module */
import { click, contains, startServer, start } from "@mail/../tests/helpers/test_utils";
import { startServer, start, click } from "@mail/../tests/helpers/test_utils";
import { makeDeferred, patchWithCleanup } from "@web/../tests/helpers/utils";
QUnit.module("message (patch)");
@@ -21,17 +22,17 @@ QUnit.test("Notification Sent", async (assert) => {
res_partner_id: partnerId,
});
const { openFormView } = await start();
await openFormView("res.partner", partnerId);
assert.containsOnce($, ".o-mail-Message");
assert.containsOnce($, ".o-mail-Message-notification");
assert.containsOnce($, ".o-mail-Message-notification i");
openFormView("res.partner", partnerId);
await contains(".o-mail-Message");
await contains(".o-mail-Message-notification");
await contains(".o-mail-Message-notification i");
assert.hasClass($(".o-mail-Message-notification i"), "fa-mobile");
await click(".o-mail-Message-notification");
assert.containsOnce($, ".o-mail-MessageNotificationPopover");
assert.containsOnce($, ".o-mail-MessageNotificationPopover i");
await contains(".o-mail-MessageNotificationPopover");
await contains(".o-mail-MessageNotificationPopover i");
assert.hasClass($(".o-mail-MessageNotificationPopover i"), "fa-check");
assert.containsOnce($, ".o-mail-MessageNotificationPopover:contains(Someone)");
await contains(".o-mail-MessageNotificationPopover:contains(Someone)");
});
QUnit.test("Notification Error", async (assert) => {
@@ -51,7 +52,7 @@ QUnit.test("Notification Error", async (assert) => {
res_partner_id: partnerId,
});
const { env, openFormView } = await start();
await openFormView("res.partner", partnerId);
openFormView("res.partner", partnerId);
patchWithCleanup(env.services.action, {
doAction(action, options) {
assert.step("do_action");
@@ -61,11 +62,11 @@ QUnit.test("Notification Error", async (assert) => {
},
});
assert.containsOnce($, ".o-mail-Message");
assert.containsOnce($, ".o-mail-Message-notification");
assert.containsOnce($, ".o-mail-Message-notification i");
await contains(".o-mail-Message");
await contains(".o-mail-Message-notification");
await contains(".o-mail-Message-notification i");
assert.hasClass($(".o-mail-Message-notification i"), "fa-mobile");
click(".o-mail-Message-notification").catch(() => {});
await click(".o-mail-Message-notification");
await openResendActionDef;
assert.verifySteps(["do_action"]);
});