[xwiki/xwiki-platform] 59d566: --wip-- [skip ci]
Branch: refs/heads/notifications-it-rest-fixture Home: https://github.com/xwiki/xwiki-platform Commit: 59d566dc643b1fbe3af75b1b87ab3887060801ce https://github.com/xwiki/xwiki-platform/commit/59d566dc643b1fbe3af75b1b87ab3... Author: Manuel Leduc <[email protected]> Date: 2026-09-18 (Fri, 18 Sep 2026) Changed paths: M xwiki-platform-core/xwiki-platform-notifications/xwiki-platform-notifications-test/xwiki-platform-notifications-test-docker/src/test/it/org/xwiki/platform/notifications/test/ui/NotificationsIT.java Log Message: ----------- --wip-- [skip ci] Commit: 070f2b21b87f3db4d9409caba056def171e71d53 https://github.com/xwiki/xwiki-platform/commit/070f2b21b87f3db4d9409caba056d... Author: Manuel Leduc <[email protected]> Date: 2026-09-18 (Fri, 18 Sep 2026) Changed paths: M xwiki-platform-core/xwiki-platform-notifications/xwiki-platform-notifications-test/xwiki-platform-notifications-test-docker/src/test/it/org/xwiki/platform/notifications/test/ui/NotificationsIT.java M xwiki-platform-core/xwiki-platform-notifications/xwiki-platform-notifications-test/xwiki-platform-notifications-test-pageobjects/src/main/java/org/xwiki/platform/notifications/test/po/NotificationsContainerElement.java M xwiki-platform-core/xwiki-platform-notifications/xwiki-platform-notifications-test/xwiki-platform-notifications-test-pageobjects/src/main/java/org/xwiki/platform/notifications/test/po/NotificationsTrayPage.java Log Message: ----------- [Misc] Speedup NotificationsIT by REST calls * Create the page of the first user over REST again in simpleNotifications: it produces a single event, before the second user watches the wiki, so it is not concerned by the ordering issue that led to restoring the browser-driven version of that test. * Give the real reason why the pages of simpleNotifications are created through the browser: the date of a page event is the date of the page, which has no milliseconds, so pages created over REST within the same second get notifications whose order is arbitrary. Marking one as read re-indexes its event, which then moves behind the others of that second, out of the ten notifications the tray displays. * Say why the deletion of simpleNotifications goes through the browser too: the event date has no milliseconds while the date from which the preference applies has some, so a deletion within that same second produces an event dated before the preference, which filters it out. * Move the lookup of a notification by its type to NotificationsTrayPage and NotificationsContainerElement, rather than reaching for the notification types from the test. Compare: https://github.com/xwiki/xwiki-platform/compare/93e733bb4f20...070f2b21b87f To unsubscribe from these emails, change your notification settings at https://github.com/xwiki/xwiki-platform/settings/notifications
participants (1)
-
XWiki Notifications