- Issue created by @adamps
- First commit to issue fork.
- Status changed to Needs work
4 months ago 9:34am 26 July 2024 - First commit to issue fork.
- Status changed to Needs review
4 months ago 10:35am 26 July 2024 Observed test failures, found there are some assert() test case failures , addressed those errors, pipeline passed.
MR is mergeable now, PLease review , moving to NR.
- Status changed to RTBC
4 months ago 10:42am 28 July 2024 - 🇬🇧United Kingdom jonathanshaw Stroud, UK
This makes sense to me. There's no obvious dependency on rdf in the test, and if there's a hidden dependency someone needs to explain it.
What I wonder about is whether the explicit mark we're asserting is somehow influenced by rdf, but I don't see how it can even be installed in the test: NodeTestBase does't install it and none of the core themes tested use it.
- 🇺🇸United States smustgrave
Checking the ticket were the test was moved to RDF appears moving back seems fine.
-
longwave →
committed a4f5580c on 10.4.x
Issue #3342700 by quietone, pooja_sharma, AdamPS, jonathanshaw:...
-
longwave →
committed a4f5580c on 10.4.x
-
longwave →
committed 41122f85 on 11.x
Issue #3342700 by quietone, pooja_sharma, AdamPS, jonathanshaw:...
-
longwave →
committed 41122f85 on 11.x
- Status changed to Fixed
3 months ago 5:17pm 22 August 2024 - 🇬🇧United Kingdom longwave UK
Committed and pushed 41122f85dd to 11.x and a4f5580c86 to 10.4.x. Thanks!
Automatically closed - issue fixed for 2 weeks with no activity.