Support for Drupal 7 is ending on 5 January 2025—it’s time to migrate to Drupal 10! Learn about the many benefits of Drupal 10 and find migration tools in our resource center.This issue is part of #2157455: [Meta] Make Drupal 8 work with PostgreSQL or remove support from core before release.
Problem/Motivation
Some tests in BlockContentIntegrationTest fail for PostgreSQL, because the view (views.view.test_block_content_view.yml) creates a path to /test-block_content_view to request test values. But it does not ensure to return the results in the expected order.
Proposed resolution
Add order clause to view to ensure return ordering by ASC and to make PostgreSQL pass.
User interface changes
None
API changes
None
Currently there is one or more fail or exception in the block_content test group identified by the new docker based testbot that need to be fixed to make PostgreSQL finally passing all tests. Command in use:
sudo DCI_DBTYPE='pgsql' \
DCI_DBVER='9.1' \
DCI_PHPVERSION='5.5' \
DCI_CONCURRENCY='4' \
DCI_TESTGROUPS='block_content' \
DCI_DRUPALBRANCH='8.0.x' \
DCI_VERBOSE='true' \
DCI_UPDATEREPO='true' \
./containers/web/run.shBeta phase evaluation
| Issue category | Bug because of broken tests |
|---|---|
| Issue priority | Major because of broken test environment |
| Disruption | None disruptive for core/contributed and custom modules/themes because it is a bugfix only |
| Comment | File | Size | Author |
|---|---|---|---|
| block_content_integration.patch | 1.09 KB | bzrudi71 | |
Comments
Comment #1
bzrudi71 CreditAttribution: bzrudi71 commentedLet's see if this also passes MySQL...
Comment #2
dawehner+1 for adding sorts where there has been no sorts ... its removes potential random failures in the future!
Comment #3
alexpott@dawehner so should views add a sort by default to make views portable and have a warning about views with no sorts?
Committed 7ef5165 and pushed to 8.0.x. Thanks!