- Issue created by @quietone
- 🇮🇳India virag.jain
I'd like to contribute by adding explicit test coverage for UiHelperTrait’s handling of $base_url. Before proceeding, I’d appreciate some clarification to ensure alignment with expectations.
Should the tests be placed in a new class (e.g.,
UiHelperTraitBaseUrlTest
extendingBrowserTestBase
) or added toUiHelperTrait
existing one? Also, should the focus be on all relevant methods (buildUrl(), getAbsoluteUrl(), drupalGet()
, and base path stripping logic), or just a subset?Lastly, since this issue was originally raised in #2986962 around drupalGet() not handling base URLs correctly, should tests specifically verify that scenario, or more broadly validate the trait's overall handling?
Happy to begin implementation with your guidance. Thanks!