-
Notifications
You must be signed in to change notification settings - Fork 23k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
[FW][FIX] bus, im_livechat: fix non deterministic discuss sidebar test #165305
[FW][FIX] bus, im_livechat: fix non deterministic discuss sidebar test #165305
Conversation
@tsm-odoo @alexkuhn cherrypicking of pull request #165204 failed. stdout:
stderr:
Either perform the forward-port manually (and push to this branch, proceeding as usual) or close this PR (maybe?). In the former case, you may want to edit this PR message as well. More info at https://github.com/odoo/odoo/wiki/Mergebot#forward-port |
Before this PR, the "unknown livechat can be displayed and interacted with" test was sometimes failing. This was due to a race condition between the sending of a message and the unpinning of the live chat channel. Specifically, the test removed the live chat from the discuss sidebar, but the channel remained pinned when the message was sent. If the live chat was unpinned before the reception of the new message notification, it remained in the sidebar, causing the test to fail. fixes runbot-61160 X-original-commit: e055c08
2b3fc72
to
4d94727
Compare
@robodoo r+ |
Before this PR, the "unknown livechat can be displayed and interacted with" test was sometimes failing. This was due to a race condition between the sending of a message and the unpinning of the live chat channel. Specifically, the test removed the live chat from the discuss sidebar, but the channel remained pinned when the message was sent. If the live chat was unpinned before the reception of the new message notification, it remained in the sidebar, causing the test to fail. fixes runbot-61160 closes #165305 X-original-commit: e055c08 Signed-off-by: Alexandre Kühn (aku) <aku@odoo.com> Signed-off-by: Matthieu Stockbauer (tsm) <tsm@odoo.com>
Before this PR, the "unknown livechat can be displayed and interacted with" test was sometimes failing. This was due to a race condition between the sending of a message and the unpinning of the live chat channel. Specifically, the test removed the live chat from the discuss sidebar, but the channel remained pinned when the message was sent. If the live chat was unpinned before the reception of the new message notification, it remained in the sidebar, causing the test to fail. fixes runbot-61160 closes odoo#165305 X-original-commit: e055c08 Signed-off-by: Alexandre Kühn (aku) <aku@odoo.com> Signed-off-by: Matthieu Stockbauer (tsm) <tsm@odoo.com>
Before this PR, the "unknown livechat can be displayed and interacted with" test was sometimes failing. This was due to a race condition between the sending of a message and the unpinning of the live chat channel. Specifically, the test removed the live chat from the discuss sidebar, but the channel remained pinned when the message was sent. If the live chat was unpinned before the reception of the new message notification, it remained in the sidebar, causing the test to fail. fixes runbot-61160 closes odoo#165305 X-original-commit: e055c08 Signed-off-by: Alexandre Kühn (aku) <aku@odoo.com> Signed-off-by: Matthieu Stockbauer (tsm) <tsm@odoo.com>
Before this PR, the "unknown livechat can be displayed and interacted with" test was sometimes failing.
This was due to a race condition between the sending of a message and the unpinning of the live chat channel.
Specifically, the test removed the live chat from the discuss sidebar, but the channel remained pinned when the message was sent. If the live chat was unpinned before the reception of the new message notification, it remained in the sidebar, causing the test to fail.
fixes runbot-61160
Forward-Port-Of: #165204