- Issue created by @admirlju
- last update
over 1 year ago 2 fail - Status changed to Needs work
over 1 year ago 8:15am 21 August 2023 No idea why it doesn't pass here. Locally the test passes.
I went through generated HTML files, it starts diverting When TestNode7 and TestNode8 are getting created, the call happens on this test, but is not put in the list, this happens correctly locally. For this reason, the test here fails at line 134
$session->linkExists('TestNode8');
.- Issue was unassigned.
- @admirlju opened merge request.
Pushed the fix to issue fork. Adding a patch file to run tests here.
- last update
over 1 year ago PHPLint Failed - last update
over 1 year ago 2 fail Okay so it looks like the BrowserTest is ignoring ordering from the modules view config, that gets installed. For now, changed it so it displays 6 items.
- last update
over 1 year ago 1 pass - last update
over 1 year ago 1 pass - Status changed to Needs review
over 1 year ago 1:19pm 6 September 2023 While I'm not yet happy with the test. Since the test is missing a lot of cases. The MR does solve the original problem and every other test case/bug should be handled by its own issue, for that reason I'm changing the status to Needs Review.
So the biggest problem is that tests create data really fast so nodes can potentially have the same timestamp and the ordering was incorrect. Added a small delay of one second between each creation, and fixed the final ordering problem.
- last update
over 1 year ago 1 pass - last update
over 1 year ago 1 pass