- Issue created by @xurizaemon
- Status changed to Needs review
9 months ago 7:01am 28 September 2023 - last update
9 months ago 46 pass, 2 fail - π³πΏNew Zealand xurizaemon Εtepoti, Aotearoa π
I expect this patch to fail on 9.5 (the default), but pass on 10.1.
Do not merge this patch please :)
- last update
9 months ago 48 pass The last submitted patch, 2: pathauto-3390330-no-op-1.patch, failed testing. View results β
- last update
9 months ago 46 pass, 2 fail - last update
9 months ago 46 pass, 2 fail - Status changed to Needs work
9 months ago 11:12am 28 September 2023 - π³πΏNew Zealand xurizaemon Εtepoti, Aotearoa π
I tried to reproduce this failure against 9.5 locally, and was unable to.
From the test results in #2 above we can see that it appears to fail on Drupal 9.5, and pass on Drupal 10.1.
From https://www.drupal.org/pift-ci-job/2705047 β this test passed for Drupal 9.5 in June 2023.
Locally I see that the failing test passes in Drupal 9.5.
I'm not sure how to identify why this fails against 9.5 in DrupalCI currently. What am I missing?
- π¨πSwitzerland Berdir Switzerland
I wouldn't bother with fixing that, if you can't reproduce then that's likely a DrupalCI thing. If you'd like to contribute, that time is likely better spent on switching to GitlabCI per https://www.drupal.org/docs/develop/git/using-gitlab-to-contribute-to-dr... β .
I'm fine with switching to D10.1 as the default branch, D9 EOL is 2 month or so.
- π³πΏNew Zealand xurizaemon Εtepoti, Aotearoa π
Delighted to oblige! π Switch to Gitlab CI Needs review
- Status changed to Fixed
9 months ago 7:09pm 28 September 2023 - π¨πSwitzerland Berdir Switzerland
I've changed the default to 10.1, 9.5 will still run as a weekly test.
- π¨πSwitzerland Berdir Switzerland
For the record, according to the weekly tests with postgresql ( https://www.drupal.org/pift-ci-job/2773503 β ), this started failing between the 14th and 21th September, either due to a Drupal core or Gitlab CI change.
- last update
9 months ago 43 pass, 1 fail - π³πΏNew Zealand xurizaemon Εtepoti, Aotearoa π
FWIW I see also a couple fails at
Drupal\Tests\pathauto\FunctionalJavascript\PathautoUiTest:102back in May 2023 on the Postgres results β .(I'm going to stop thinking about this now.)
- last update
9 months ago 43 pass, 1 fail - last update
9 months ago Composer require failure - π³πΏNew Zealand jweowu
I've hijacked the no-op patch in this issue for the purposes of running tests to confirm that a failure I was seeing in π Duplicate alias entities created with 'Create a new alias. Leave the existing alias functioning' setting RTBC when testing with MySQL 8 was not on account of the patch in that issue.
I've added these tests to the patch in #2 here, which I expected to all fail:
* PHP 8.2 & MySQL 8, D10.1
* PHP 8.1 & MySQL 8, D10.1
* PHP 7.4 & MySQL 8, D10.1(I'm surprised I was permitted to use that last combo, so that will probably fail more comprehensively than the other two.)
Automatically closed - issue fixed for 2 weeks with no activity.