- πΊπΈUnited States smustgrave
Came up as the daily BSI target.
May still be relevant but agree probably should have simple test coverage to make sure a trait works.
- First commit to issue fork.
- π¦πΊAustralia acbramley
Why not do something like this... move the breadcrumb assertions to the tests that are actually testing the field UI pages. They don't really belong in those trait functions IMO. The other assertions in there are making sure the form is progressing correctly.
As for CommentNonNodeTest, it does more assertions on other breadcrumbs so we can't remove the breadcrumb block from there anyway.
This new MR decouples breadcrumbs from fieldUIAddNewField and fieldUIDeleteField though. No documentation changes, CR, or additional tests required.
- Merge request !11377Issue #2417517: Remove breadcrumb dependency from FieldUiTestTrait β (Closed) created by acbramley