Problem/Motivation

This introduced PostgreSQL failure:
- https://www.drupal.org/pift-ci-job/1147872
- https://www.drupal.org/pift-ci-job/1147909

Drupal\Tests\search\Kernel\Migrate\d7\MigrateSearchPageTest::testSearchPage
Failed asserting that Array &0 (
    'rankings' => Array &1 (
        'promote' => 0
        'relevance' => 2
        'sticky' => 0
        'views' => 0
        'comments' => 4
    )
) is identical to Array &0 (
    'rankings' => Array &1 (
        'comments' => 4
        'promote' => 0
        'relevance' => 2
        'sticky' => 0
        'views' => 0
    )
).

/var/www/html/core/modules/search/tests/src/Kernel/Migrate/d7/MigrateSearchPageTest.php:43
/var/www/html/core/modules/search/tests/src/Kernel/Migrate/d7/MigrateSearchPageTest.php:79

Proposed resolution

Remaining tasks

User interface changes

API changes

Data model changes

Release notes snippet

CommentFileSizeAuthor
#5 3021838-5.patch457 bytesalexpott
#2 3021838-2.patch1.07 KBquietone
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

heddn created an issue. See original summary.

quietone’s picture

quietone CreditAttribution: quietone at Acro Commerce commented
Status: Active » Needs review
FileSize
3021838-2.patch1.07 KB

Sort the arrays before testing as is done in /d6/MigrateTaxonomyTermTranslationTest.php

quietone’s picture

quietone CreditAttribution: quietone as a volunteer commented
Title: [Postgress] Variable to config: search_active_modules » [PostgresSQL] Variable to config: search_active_modules

Typo in title

heddn’s picture

heddn
Primary language English
Location Nicaragua
CreditAttribution: heddn at MTech, LLC, Acro Commerce commented
Status: Needs review » Reviewed & tested by the community

Fixes the issue. The order of results isn't particularly that interesting, so doing this in a ksort in the test is just fine.

alexpott’s picture

alexpott
he/they
Primary language English
Location 🇪🇺🌍
CreditAttribution: alexpott at Acro Commerce, Thunder commented
Status: Reviewed & tested by the community » Needs review
FileSize
3021838-5.patch457 bytes

We can use config key ordering to make all the things consistent.

alexpott’s picture

alexpott
he/they
Primary language English
Location 🇪🇺🌍
CreditAttribution: alexpott at Acro Commerce, Thunder commented

Note that the order in the configuration is not significant. It is never "important" - we always get the rankings from the hook system first and then use this configuration - see \Drupal\node\Plugin\Search\NodeSearch::addNodeRankings().

alexpott’s picture

alexpott
he/they
Primary language English
Location 🇪🇺🌍
CreditAttribution: alexpott at Acro Commerce, Thunder commented

#6 is why I didn't add a post update hook to potentially save all the search page config entities.

larowlan’s picture

larowlan
Location 🇦🇺🏝.au GMT+10
CreditAttribution: larowlan at PreviousNext commented
Status: Needs review » Reviewed & tested by the community

Looks good

alexpott’s picture

alexpott
he/they
Primary language English
Location 🇪🇺🌍
CreditAttribution: alexpott at Acro Commerce, Thunder commented
Status: Reviewed & tested by the community » Fixed

Committed b942654 and pushed to 8.7.x. Thanks!

Committed this because this is resulting in test fails in HEAD.

  • alexpott committed b942654 on 8.7.x 
    Issue #3021838 by alexpott, quietone: [PostgresSQL] Variable to config:...

Status: Fixed » Closed (fixed)

Automatically closed - issue fixed for 2 weeks with no activity.