- Issue created by @manuel.adan
- Status changed to Needs review
about 1 year ago 5:48pm 14 December 2023 - πͺπΈSpain manuel.adan π
First browser test covering the module's main feature.
- πͺπΈSpain aaron gil martinez Zaragoza
Aaron Gil Martinez β made their first commit to this issueβs fork.
- Merge request !1Modify test, add existing block modification. β (Closed) created by aaron gil martinez
- Merge request !8Test fail #3408499 "Provide test coverage fail" β (Open) created by aaron gil martinez
- Status changed to RTBC
12 months ago 10:59am 28 January 2024 - πͺπΈSpain aaron gil martinez Zaragoza
Manuel.Adam β Seems a good starting point, the test works and checks the fields exposed in the form by the module.
Added a MR where I played a bit with the form and fields, it checks the same that the one provided in MR6 but wanted to try with a modification of a block, something its not working, could be my changes.
I think its a good idea to merge 6 and enable CI, and continue building more tests about the features that the module provides like additional configuration or more cases.
-
manuel.adan β
committed 4d57ae60 on 1.0.x
Issue #3408499 by manuel.adan, Aaron Gil Martinez: Provide test coverage
-
manuel.adan β
committed 4d57ae60 on 1.0.x
- Status changed to Fixed
12 months ago 11:59am 28 January 2024 - πͺπΈSpain manuel.adan π
Thank you for the review! Committed as suggested, so now we have basic test coverage to be extended.
Automatically closed - issue fixed for 2 weeks with no activity.