Fix test for D10 core

Created on 31 January 2024, 5 months ago
Updated 28 March 2024, 3 months ago
πŸ“Œ Task
Status

Fixed

Version

4.0

Component

Code

Created by

πŸ‡ΊπŸ‡¦Ukraine Bobik

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

Merge Requests

Comments & Activities

  • Issue created by @Bobik
  • πŸ‡¦πŸ‡ΊAustralia darvanen Sydney, Australia

    The current test system is passing on D10:

    https://www.drupal.org/pift-ci-job/2848692 β†’

    So what is different about running it on gitlab?

  • Merge request !23Issue #3418294: Fix test for D10 core β†’ (Merged) created by Unnamed author
  • Open in Jenkins β†’ Open on Drupal.org β†’
    Core: 10.2.1 + Environment: PHP 8.1 & MySQL 8
    last update 5 months ago
    14 pass
  • Open in Jenkins β†’ Open on Drupal.org β†’
    Core: 10.2.1 + Environment: PHP 8.1 & MySQL 8
    last update 5 months ago
    14 pass
  • Pipeline finished with Success
    5 months ago
    Total: 245s
    #89832
  • πŸ‡ΊπŸ‡¦Ukraine Bobik

    Hi @
    Comment#2darvanen

    We have the following
    - all testing of modules will migrate to Drupal GitLab CI
    - the process of testing is more flexible
    - run tests more fast (it is very useful)
    more details here - https://www.drupal.org/about/core/blog/drupal-cores-gitlab-ci-testing-is... β†’

  • Issue was unassigned.
  • Status changed to Needs review 5 months ago
  • πŸ‡ΊπŸ‡¦Ukraine Bobik

    I've fixed the test. Please check it

  • Status changed to Needs work 4 months ago
  • πŸ‡¦πŸ‡ΊAustralia darvanen Sydney, Australia

    Thanks @bobi-mel, finally got to this. Please see my comment, I may be off the mark but I'd like your input.

  • Status changed to Needs review 4 months ago
  • πŸ‡ΊπŸ‡¦Ukraine Bobik
  • Open in Jenkins β†’ Open on Drupal.org β†’
    Core: 10.2.1 + Environment: PHP 8.1 & MySQL 8
    last update 4 months ago
    14 pass
  • Status changed to RTBC 4 months ago
  • πŸ‡¦πŸ‡ΊAustralia darvanen Sydney, Australia

    I realise I have started a conversation here about the contents of the gitlab-ci.yml file which rightly belongs in πŸ“Œ Add gitlab CI Needs work . So I'm removing that file from this branch so we can commit the fixes to the tests themselves. I will move my comments there.

  • Pipeline finished with Skipped
    4 months ago
    #118620
  • Open in Jenkins β†’ Open on Drupal.org β†’
    Core: 10.2.1 + Environment: PHP 8.1 & MySQL 8
    last update 4 months ago
    14 pass
  • Status changed to Fixed 4 months ago
  • πŸ‡¦πŸ‡ΊAustralia darvanen Sydney, Australia
  • Status changed to Needs work 4 months ago
  • πŸ‡¦πŸ‡ΊAustralia darvanen Sydney, Australia

    There were two errors when I merged this into πŸ“Œ Add gitlab CI Needs work :

    1) Drupal\Tests\collapsiblock\Functional\BlockInstanceTest::testBlockInstanceConfigForm
    TypeError: Behat\Mink\Element\TraversableElement::findButton(): Argument #1 ($locator) must be of type string, Drupal\Core\StringTranslation\TranslatableMarkup given, called in /builds/project/collapsiblock/web/core/tests/Drupal/Tests/WebAssert.php on line 143
    /builds/project/collapsiblock/vendor/behat/mink/src/Element/TraversableElement.php:97
    /builds/project/collapsiblock/web/core/tests/Drupal/Tests/WebAssert.php:143
    /builds/project/collapsiblock/web/core/tests/Drupal/Tests/UiHelperTrait.php:78
    /builds/project/collapsiblock/tests/src/Functional/BlockInstanceTest.php:62
    /builds/project/collapsiblock/vendor/phpunit/phpunit/src/Framework/TestResult.php:728
    2) Drupal\Tests\collapsiblock\Functional\GlobalConfigurationTest::testGlobalConfigForm
    TypeError: Behat\Mink\Element\TraversableElement::findButton(): Argument #1 ($locator) must be of type string, Drupal\Core\StringTranslation\TranslatableMarkup given, called in /builds/project/collapsiblock/web/core/tests/Drupal/Tests/WebAssert.php on line 143
    /builds/project/collapsiblock/vendor/behat/mink/src/Element/TraversableElement.php:97
    /builds/project/collapsiblock/web/core/tests/Drupal/Tests/WebAssert.php:143
    /builds/project/collapsiblock/web/core/tests/Drupal/Tests/UiHelperTrait.php:78
    /builds/project/collapsiblock/tests/src/Functional/GlobalConfigurationTest.php:72
    /builds/project/collapsiblock/vendor/phpunit/phpunit/src/Framework/TestResult.php:728

  • πŸ‡¦πŸ‡ΊAustralia darvanen Sydney, Australia

    Actually let's address them in that issue.

  • Status changed to Fixed 4 months ago
  • πŸ‡¦πŸ‡ΊAustralia darvanen Sydney, Australia
  • Automatically closed - issue fixed for 2 weeks with no activity.

Production build 0.69.0 2024