The 2026 Drupal Business Survey is open. Your response will help shape the future of the Drupal business ecosystem.Currently the tests do not pass on Drupal CI even though it works locally.
After a few research I think I found the problem that is the Drupal CI try to request "/subdirectory/admin/config/development" instead of "/admin/config/development".
The problem seems to be the presence of the initial "/" in our tests paths.
I'll try to patch this and remove this initial "/" so see if it's working better.
Comments
Comment #2
nixou commentedComment #3
nixou commentedOk so the problem is that the behavior of the module is not correct when Drupal is installed in a subdirectory.
The tests are launched from a subdirectory precisely to detect this kind of problem.
The problem is that the redirect destination service is used to determine the current url.
If Drupal is installed in a subdirectory, this service will return the full path including the subdirectory and that's where it goes wrong.
So instead of using this service, we can, more logically, use the current path service to retrieve the current url.
Comment #4
nixou commentedComment #5
nixou commentedWe just need to ensure to keep query string when necessary as it was the case with the redirect destination service.
This was not covered by the existing tests so I created a new one.
Comment #6
nixou commentedComment #7
nixou commentedComment #8
nixou commentedOk all good with the new tests.
Comment #10
nixou commented