[xwiki/xwiki-platform] 17d9ff: [Misc] Wait for the HTML5 uploader before setting ...
Branch: refs/heads/stable-18.8.x Home: https://github.com/xwiki/xwiki-platform Commit: 17d9ffe33b8ad177eb907b4fa0fc3bad16d88146 https://github.com/xwiki/xwiki-platform/commit/17d9ffe33b8ad177eb907b4fa0fc3... Author: Michael Hamann <[email protected]> Date: 2026-09-22 (Tue, 22 Sep 2026) Changed paths: M xwiki-platform-core/xwiki-platform-flamingo/xwiki-platform-flamingo-skin/xwiki-platform-flamingo-skin-test/xwiki-platform-flamingo-skin-test-pageobjects/src/main/java/org/xwiki/flamingo/skin/test/po/AttachmentsPane.java Log Message: ----------- [Misc] Wait for the HTML5 uploader before setting the file to upload AttachmentsPane#setFileToUpload only sent the file path to the file input. The upload itself is started by the change listener that the HTML5 uploader installs on that input, and the uploader is initialized asynchronously, once attachments.js has resolved the 'xwiki-upload' module through RequireJS. Opening the attachments doc extra pane only waits for the pane to be displayed, so the change event could be fired before the listener existed, in which case it was lost and no upload happened at all. This made AttachmentValidationIT#validateDisguisedAttachment fail on CI with a NoSuchElementException on .xnotification-error: the archived video shows the file sitting in the input with the submit buttons still visible, i.e. hideFormButtons() had never run. Wait for the html5upload-initialized marker that the uploader sets on the form, which is added right after the change listener is attached. Co-Authored-By: Claude Opus 5 (1M context) <[email protected]> (cherry picked from commit 521028d8ff774945a1ceec75401fba6952a863aa) Commit: 3bd5b58d7da365899ec499e4f756bd229f80fb65 https://github.com/xwiki/xwiki-platform/commit/3bd5b58d7da365899ec499e4f756b... Author: Michael Hamann <[email protected]> Date: 2026-09-22 (Tue, 22 Sep 2026) Changed paths: M xwiki-platform-core/xwiki-platform-flamingo/xwiki-platform-flamingo-skin/xwiki-platform-flamingo-skin-test/xwiki-platform-flamingo-skin-test-docker/src/test/it/org/xwiki/flamingo/test/docker/SecurityCacheStressIT.java Log Message: ----------- [Misc] Don't run the security cache stress test through the browser The test raised the page load timeout to 10 minutes, but that is not the timeout it was hitting: the Selenium HTTP client gives up on the "get" command after its own read timeout of 3 minutes, and that timeout is fixed when the driver is created, so a test cannot change it. Request the rendered page over HTTP straight from the test JVM instead, where no such timeout applies. Co-Authored-By: Claude Opus 5 (1M context) <[email protected]> (cherry picked from commit cbd3bb9d446fa4fde1be25b2418d929000ced3be) Compare: https://github.com/xwiki/xwiki-platform/compare/bb4ef50985b0...3bd5b58d7da3 To unsubscribe from these emails, change your notification settings at https://github.com/xwiki/xwiki-platform/settings/notifications
participants (1)
-
XWiki Notifications