- Issue created by @DamienMcKenna
- πΊπΈUnited States DamienMcKenna NH, USA
Starting point, that fails locally, because some page requests give a 403 response.
- Status changed to Needs review
over 1 year ago 8:57pm 26 July 2023 - last update
over 1 year ago 3 pass, 2 fail The last submitted patch, 2: masquerade-n3377262-2.patch, failed testing. View results β
- Status changed to Needs work
over 1 year ago 9:08pm 26 July 2023 - π«π·France andypost
+++ b/tests/src/Functional/MasqueradeAccessTest.php @@ -131,7 +134,9 @@ protected function assertCanMasqueradeAs(AccountInterface $target_account) { + $this->assertSession()->statusCodeEquals(200); $this->submitForm($edit, 'Switch'); + $this->assertSession()->statusCodeEquals(200); $this->assertSession() ->responseNotContains($this->t('You are not allowed to masquerade as %name.', [
Test said it's line MasqueradeAccessTest.php:139 but not the submitForm()
- π«π·France andypost
I mean as expected result is 403 then no reason to try submit form
- Status changed to Needs review
over 1 year ago 9:13pm 26 July 2023 - last update
over 1 year ago 4 pass - Status changed to RTBC
over 1 year ago 9:41pm 26 July 2023 -
andypost β
committed a88e2e15 on 8.x-2.x authored by
DamienMcKenna β
Issue #3377262 by DamienMcKenna: Extend test coverage to note when the...
-
andypost β
committed a88e2e15 on 8.x-2.x authored by
DamienMcKenna β
- Status changed to Fixed
over 1 year ago 9:53pm 26 July 2023 - πΊπΈUnited States DamienMcKenna NH, USA
Thank you for the quick commit!
Automatically closed - issue fixed for 2 weeks with no activity.