- Issue created by @catch
- @catch opened merge request.
- π¬π§United Kingdom catch
I think this got slower because of π Not able to add to List field with 0 as the first machine name. Needs work , we can actually consolidate that test case with the main list_string one, and retain the list_float coverage for now. That way we definitely don't lose test coverage but also save some iterations.
- π¬π§United Kingdom catch
https://git.drupalcode.org/issue/drupal-3522004/-/jobs/5114056 this change is enough to have this test finish quicker than BigPipeRegressionTest, so I think it's 'good enough' until it becomes the slowest test again.
Another option here, and the first one I considered, was dropping list_float testing since we don't support adding those fields via the UI at all now.
- π¬π§United Kingdom alexpott πͺπΊπ
Fair enough. We're not losing test coverage here.
Committed and pushed 6b577ab50be to 11.x and fb605312a2e to 11.2.x. Thanks!
-
alexpott β
committed fb605312 on 11.2.x
Issue #3522004 by catch: Consolidate list_string test cases in...
-
alexpott β
committed fb605312 on 11.2.x
-
alexpott β
committed 6b577ab5 on 11.x
Issue #3522004 by catch: Consolidate list_string test cases in...
-
alexpott β
committed 6b577ab5 on 11.x