Early Bird Registration for DrupalCon Portland 2024 is open! Register by 23:59 PST on 31 March 2024, to get $100 off your ticket.
Problem/Motivation
As @larowlan comment on issue #3162603: EntityStorageBase::loadByProperties() is broken on PostgreSQL when using two or more case insensitive properties (comment #31), NodeLoadMultipleTest should be a kernel test.
Proposed resolution
Split NodeLoadMultipleTest into two things, the loading functionality should be a kernel test and the frontpage view test can be an existing test.
Comment | File | Size | Author |
---|---|---|---|
#4 | 3170246-4.patch | 2.17 KB | paulocs |
#2 | 3170246-2.patch | 3.96 KB | paulocs |
Comments
Comment #2
paulocsA patch for it.
Comment #3
longwave\Drupal\Tests\node\Functional\Views\FrontPageTest already exists and is more comprehensive.
\Drupal\Tests\system\Functional\System\FrontPageTest also has some overlapping functionality.
Can we just drop NodePromotedTest?
Comment #4
paulocsFor me makes sense @longwave. I think there is no need to have FrontPageTest
I attached a new patch for it.
Comment #5
longwaveThanks, looks good to me.
Comment #8
alexpottCommitted and pushed 4566403054 to 9.1.x and b58a924af7 to 9.0.x. Thanks!
Backported to 9.0.x as this is a test change.
Comment #9
alexpott