From f284cd9eaf49600b2eefe2d85d752f8bfc73307b Mon Sep 17 00:00:00 2001 From: "Eloy Lafuente (stronk7)" Date: Sun, 1 Oct 2023 23:23:30 +0200 Subject: [PATCH] MDL-79247 forum: Workaround same-time discussion modified dates bug Within the forum_get_discussion_neighbours() function of forum when the discussions timemodified (last post) is the same, there is a bug that does strange things when calculating the prev and next discussions. Note that, in real life, this is really hard to achieve, but in tests, when multiple discussions and post can be created by generators in the same second (specially when the test machine is quick), chances of facing that problem are higher. By adding 1 second wait, we ensure that the discussions won't have the same timemodified (last post) and workaround the problem. No mater of that, the problem deserves an issue to be created so we guarantee from code that it also works ok when the same timemodified (last post) situation happens. --- mod/forum/tests/behat/discussion_navigation.feature | 4 ++++ 1 file changed, 4 insertions(+) diff --git a/mod/forum/tests/behat/discussion_navigation.feature b/mod/forum/tests/behat/discussion_navigation.feature index 4caa84a7081..50eca8ebe38 100644 --- a/mod/forum/tests/behat/discussion_navigation.feature +++ b/mod/forum/tests/behat/discussion_navigation.feature @@ -51,6 +51,10 @@ Feature: A user can navigate to previous and next discussions And I follow "Reply" And I set the following fields to these values: | Message | Answer to discussion | + # We need to wait a bit to guarantee that the post is created after the previous ones. + # because there is a bug in the forum_get_discussion_neighbours() when all the discussion + # last modified times are the same. See MDL-79247 for more details. + And I wait "1" seconds And I press "Post to forum" And I should not see "Discussion 2" And I should see "Discussion 3"