- Issue created by @catch
- Status changed to Needs review
about 1 year ago 11:50am 22 January 2024 - 🇬🇧United Kingdom catch
This was a bit of a pain, these test classes are massive (hence why they're so slow...), but it's one of the very last tests that wraps around all the other tests in a job.
- Status changed to Needs work
about 1 year ago 3:04pm 22 January 2024 - 🇺🇸United States smustgrave
Large MR, but moving the methods around didn't seem to break anything. Could we get a change record for the move though? Can imagine it's something contrib modules could benefit from
- Status changed to Needs review
about 1 year ago 8:15pm 25 January 2024 - 🇺🇸United States smustgrave
looks like it could use a manual rebase.
- Status changed to RTBC
about 1 year ago 9:14pm 28 January 2024 - Status changed to Needs review
about 1 year ago 9:28am 2 February 2024 - Status changed to RTBC
about 1 year ago 3:39pm 2 February 2024 - Status changed to Needs work
about 1 year ago 3:49pm 2 February 2024 - 🇺🇸United States smustgrave
Ugh rebase seemed to break a few tests.
- First commit to issue fork.
- Merge request !10637Resolve #3416181 "Allow resourcetestbase methods rebased" → (Open) created by bbrala
- 🇳🇱Netherlands bbrala Netherlands
This issue was mentioned in slack in another context, thought i'd revive this. The rebase was a little hard with the changes done by now. At least phpstan is happy by now (and baseline did shrink a bit ;)). Lets see how it goed.
- 🇳🇱Netherlands bbrala Netherlands
Think i missed a few modules to enable while mergin. Cant look right now but pretty sure thats why we have a failure left.
- 🇳🇱Netherlands bbrala Netherlands
Added missing modules. Fingers crossed.