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

[FIX] website: wait for steps to be effective in social media test

Browse Source 
Could not actually reproduce problem locally.

Looking at the logs, the randomly failing tests seem to be related to
the effects of some previous steps not having been applied even though
the test did proceed further.

This commit introduces intermediary checks to make sure that the effect
of each step is fully applied before proceeding to the next one.

runbot-3171
runbot-3172
runbot-3629
runbot-3630
runbot-3778
runbot-3779
runbot-3859
runbot-3860

closes odoo/odoo#92816

X-original-commit: 2d47b78e677f5046214cb2c8613eeb510753076d
Signed-off-by: Romain Derie (rde) <rde@odoo.com>
This commit is contained in:
Benoit Socias
2022-06-02 18:58:54 +02:00
parent ad4e528dee
commit cb6bb3f8f2
1 changed files with 48 additions and 27 deletions
Download Patch File Download Diff File Expand all files Collapse all files
addons/website/static/tests/tours/snippet_social_media.js
+48 -27
View File
@@ -3,52 +3,65 @@
import tour from 'web_tour.tour';
import wTourUtils from 'website.tour_utils';
const addNewSocialNetwork = function (optionIndex, linkIndex, url) {
return [{
content: "Click on Add New Social Network",
trigger: 'we-list we-button.o_we_list_add_optional',
},
{
content: "Ensure new option is found",
trigger: `we-list table input:eq(${optionIndex})[data-list-position=${optionIndex}][data-dom-position=${linkIndex}][data-undeletable=false]`,
run: () => {}, // This is a check.
},
{
content: "Ensure new link is found",
trigger: `.s_social_media:has(a:eq(${linkIndex})[href='https://www.example.com'])`,
run: () => {}, // This is a check.
},
{
content: "Change added Option label",
trigger: `we-list table input:eq(${optionIndex})`,
run: `text ${url}`,
},
{
content: "Ensure new link is changed",
trigger: `.s_social_media:has(a:eq(${linkIndex})[href='${url}'])`,
run: () => {}, // This is a check.
}];
};
tour.register('snippet_social_media', {
test: true,
url: '/?enable_editor=1',
}, [
wTourUtils.dragNDrop({id: 's_social_media', name: 'Social Media'}),
wTourUtils.clickOnSnippet({id: 's_social_media', name: 'Social Media'}),
{
content: 'Click on Add New Social Network',
trigger: 'we-list we-button.o_we_list_add_optional',
},
{
content: 'Change added Option label',
trigger: 'we-list table input:eq(6)',
run: 'text https://www.youtu.be/y7TlnAv6cto',
},
...addNewSocialNetwork(6, 6, 'https://www.youtu.be/y7TlnAv6cto'),
{
content: 'Click on the toggle to hide Facebook',
trigger: 'we-list table we-button.o_we_user_value_widget',
run: 'click',
},
{
content: "Ensure twitter became first",
trigger: '.s_social_media:has(a:eq(0)[href="/website/social/twitter"])',
run: () => {}, // This is a check.
},
{
content: 'Drag the facebook link at the end of the list',
trigger: 'we-list table we-button.o_we_drag_handle',
position: 'bottom',
run: "drag_and_drop we-list table tr:last-child",
},
{
content: 'Check drop completed',
trigger: 'we-list table input:eq(6)[data-media="facebook"]',
run: () => {}, // This is a check.
},
// Create a Link for which we don't have an icon to propose.
{
content: 'Click on Add New Social Network',
trigger: 'we-list we-button.o_we_list_add_optional',
},
{
content: 'Change added Option label (2)',
trigger: 'we-list table input:eq(7)',
run: 'text https://whatever.it/1EdSw9X',
},
...addNewSocialNetwork(7, 6, 'https://whatever.it/1EdSw9X'),
// Create a custom instagram link.
{
content: 'Click on Add New Social Network',
trigger: 'we-list we-button.o_we_list_add_optional',
},
{
content: 'Change added Option label (3)',
trigger: 'we-list table input:eq(8)',
run: 'text https://instagr.am/odoo.official/',
},
...addNewSocialNetwork(8, 7, 'https://instagr.am/odoo.official/'),
{
content: "Check if the result is correct before removing",
trigger: ".s_social_media" +
@@ -60,6 +73,7 @@ tour.register('snippet_social_media', {
":has(a:eq(5)[href='https://www.youtu.be/y7TlnAv6cto']:has(i.fa-youtube))" +
":has(a:eq(6)[href='https://whatever.it/1EdSw9X']:has(i.fa-pencil))" +
":has(a:eq(7)[href='https://instagr.am/odoo.official/']:has(i.fa-instagram))",
run: () => {}, // This is a check.
},
// Create a custom link, not officially supported, ensure icon is found.
{
@@ -71,12 +85,18 @@ tour.register('snippet_social_media', {
content: "Ensure paypal icon is found",
trigger: ".s_social_media" +
":has(a:eq(5)[href='https://www.paypal.com/abc']:has(i.fa-paypal))",
run: () => {}, // This is a check.
},
{
content: 'Delete the custom link',
trigger: 'we-list we-button.o_we_select_remove_option',
run: 'click',
},
{
content: "Ensure custom link was removed",
trigger: '.s_social_media:has(a:eq(5)[href="https://whatever.it/1EdSw9X"]:has(i.fa-pencil))',
run: () => {}, // This is a check.
},
{
content: 'Click on the toggle to show Facebook',
trigger: 'we-list table we-button.o_we_user_value_widget:not(.active)',
@@ -93,6 +113,7 @@ tour.register('snippet_social_media', {
":has(a:eq(5)[href='/website/social/facebook'])" +
":has(a:eq(6)[href='https://whatever.it/1EdSw9X']:has(i.fa-pencil))" +
":has(a:eq(7)[href='https://instagr.am/odoo.official/']:has(i.fa-instagram))",
run: () => {}, // This is a check.
},
{
content: 'Change url of the DB instagram link',