Created on 6 May 2024, 6 months ago
Updated 1 July 2024, 5 months ago

Problem/Motivation

Placeholder for d11 tests.

๐Ÿ“Œ Task
Status

Fixed

Version

4.0

Component

Code

Created by

๐Ÿ‡บ๐Ÿ‡ธUnited States japerry KVUO

Live updates comments and jobs are added and updated live.
Sign in to follow issues

Merge Requests

Comments & Activities

  • Issue created by @japerry
  • Merge request !61Drupal 11 Compatibility. โ†’ (Open) created by japerry
  • ๐Ÿ‡ญ๐Ÿ‡บHungary Balu Ertl Budapest ๐Ÿ‡ช๐Ÿ‡บ

    Perhaps would be nice to clean up the situation around ๐Ÿ“Œ Drupal 10 compatibility issues Needs review and close it first, then continue with focusing on compatibility with Drupal 11.

  • ๐Ÿ‡ฎ๐Ÿ‡ณIndia vishalkhode

    vishalkhode โ†’ made their first commit to this issueโ€™s fork.

  • Pipeline finished with Failed
    6 months ago
    Total: 358s
    #172789
  • Pipeline finished with Failed
    6 months ago
    Total: 176402s
    #177145
  • Pipeline finished with Failed
    6 months ago
    Total: 571s
    #179590
  • Pipeline finished with Failed
    6 months ago
    Total: 449s
    #179606
  • Pipeline finished with Failed
    6 months ago
    Total: 531s
    #182560
  • Pipeline finished with Success
    6 months ago
    Total: 489s
    #182588
  • Pipeline finished with Success
    6 months ago
    Total: 397s
    #182708
  • Status changed to Needs review 6 months ago
  • ๐Ÿ‡ฎ๐Ÿ‡ณIndia vishalkhode

    Hi @japerry
    The CI is green now. All PHPUnit tests are passing. FYI: I've reverted the two commits: 12cf6068 & 94cdba03 as I don't think it's good idea to disable ConfigSchema checking. To fix this, I've added an event_subscriber to update config value at runtime.

    For Drupal 11, following two PHPUnit tests were failing:

    1) Drupal\Tests\ctools_views\Functional\CToolsViewsBasicViewBlockTest::testHideFields
    Failed asserting that 0 matches expected 5.
    /builds/issue/ctools-3445547/modules/ctools_views/tests/src/Functional/CToolsViewsBasicViewBlockTest.php:250
    2) Drupal\Tests\ctools_views\Functional\CToolsViewsBasicViewBlockTest::testOrderFields
    Failed asserting that 0 matches expected 4.
    /builds/issue/ctools-3445547/modules/ctools_views/tests/src/Functional/CToolsViewsBasicViewBlockTest.php:314
    ERRORS!
    

    When I debugged further, I found that the test was failing due to following schema validation check:

    Drupal\Core\Config\Schema\SchemaIncompleteException: Schema errors for block.block.views_block__ctools_views_test_view_block_fields with the following errors: block.block.views_block__ctools_views_test_view_block_fields:settings.fields.id missing schema, block.block.views_block__ctools_views_test_view_block_fields:settings.fields.name missing schema, block.block.views_block__ctools_views_test_view_block_fields:settings.fields.age missing schema, block.block.views_block__ctools_views_test_view_block_fields:settings.fields.job missing schema, block.block.views_block__ctools_views_test_view_block_fields:settings.fields.created missing schema, block.block.views_block__ctools_views_test_view_block_fields:settings.fields.name_1 missing schema in Drupal\Core\Config\Development\ConfigSchemaChecker->onConfigSave() (line 101 of core/lib/Drupal/Core/Config/Development/ConfigSchemaChecker.php).
    

    I wasn't sure on how to fix it, so I temporary disable the schema validation check (same you did earlier). If you know, how to fix it properly, that would be great.
    Thanks.

  • ๐Ÿ‡บ๐Ÿ‡ธUnited States japerry KVUO

    Moving to 4.x as thats where we full support Drupal 11.

    +1 to the event subscriber! Regarding the config, yah thats what mglaman suggested to do as well, that should be fine for testing.

  • ๐Ÿ‡บ๐Ÿ‡ธUnited States japerry KVUO

    reverted my removal of Drupal 9. 4.1.0 can be D9 compatible, and we can drop 9.5 later if the code requires it. Tests will only work in Drupal 10.2+, which is fine.

  • Status changed to Fixed 5 months ago
  • ๐Ÿ‡บ๐Ÿ‡ธUnited States japerry KVUO

    Committed to 4.x!

  • ๐Ÿ‡ฆ๐Ÿ‡บAustralia dpi Perth, Australia

    This issue caused a break in ๐Ÿ› PageBlockDisplayVariant::__sleep() is no longer compatible with parent signature from ctools leading to PHP fatals RTBC . Mind reviewing that @japerry, since you're also a maintainer there.

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

Production build 0.71.5 2024