Don't start session on filter reset
1 unresolved thread
1 unresolved thread
Closes #3471220
Merge request reports
Activity
added 711 commits
-
75d782c0...cdd9e92a - 709 commits from branch
project:11.x - 3e529093 - Add proposed code and test
- 9924168c - Use actual reset form submit with test.
-
75d782c0...cdd9e92a - 709 commits from branch
566 569 $this->assertTrue($this->assertSession()->optionExists('type', 'page')->isSelected()); 567 570 } 568 571 572 /** 573 * Asserts whether a session cookie is present on the client or not. 574 * 575 * @internal 576 */ 577 public function assertSessionCookieOnClient(bool $expected_present): void { 
These similar methods exist:
Drupal\Tests\system\Functional\Cache\SessionExistsCacheContextTest::assertSessionCookieOnClient()Drupal\Tests\system\Functional\Session\SessionTest::assertSessionCookie()Drupal\Tests\big_pipe\Functional\BigPipeTest::assertSessionCookieExists()
Those methods are all used more than once, but it seems like it might make sense to consolidate into the base class or a trait. If so, I would leave this method in place here to make it easier to find and replace later in a follow up.
Edited by godotislatechanged this line in version 3 of the diff
Please register or sign in to reply