- πΊπΈUnited States tr Cascadia
Updated patch for latest changes to D10.
The test failure seen in #14 has not been fixed - the update test checks to see if then non-updated database contains the hostname column. It should not contain that column before the update, but it does, causing the test to fail. I do not know why this happens, and I have found no similar test in core to use as an example for why it fails. Fixing this test is why this issue is still open. The last submitted patch, 17: 3121331-17-hostname-column-missing.patch, failed testing. View results β
- codesniffer_fixes.patch Interdiff of automated coding standards fixes only.The last submitted patch, 19: 3121331-19-hostname-column-missing.patch, failed testing. View results β
- codesniffer_fixes.patch Interdiff of automated coding standards fixes only.- Status changed to Fixed
almost 2 years ago 7:58am 11 March 2023 - πΊπΈUnited States tr Cascadia
Stupid Drupal. Wasted more than 15 months because of a stupid undocumented problem in update tests, where IF YOU'RE on DRUPALCI you can't access the database the same as you would in any other Functional test. It works on my local just fine of course. And there were no examples in core or contrib where people tested the addition of a database column - probably because no-one could get it to work. And I got no help in the various channels where I asked for help with this.
Committed. Good riddance.
Automatically closed - issue fixed for 2 weeks with no activity.