- Issue created by @bobi-mel
- π¦πΊ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?
- last update
about 1 year ago 14 pass - last update
about 1 year ago 14 pass - πΊπ¦Ukraine bobi-mel
Hi @
Comment#2darvanenWe 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
about 1 year ago 4:48pm 7 February 2024 - Status changed to Needs work
about 1 year ago 5:53am 15 February 2024 - π¦πΊ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
about 1 year ago 12:32pm 1 March 2024 - last update
about 1 year ago 14 pass - Status changed to RTBC
about 1 year ago 9:37pm 13 March 2024 - π¦πΊ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.
- last update
about 1 year ago 14 pass -
darvanen β
committed ad6ada9f on 4.x authored by
bobi-mel β
Issue #3418294 by bobi-mel, darvanen: Fix test for D10 core
-
darvanen β
committed ad6ada9f on 4.x authored by
bobi-mel β
- Status changed to Fixed
about 1 year ago 9:39pm 13 March 2024 - Status changed to Needs work
about 1 year ago 12:30am 14 March 2024 - π¦πΊ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
about 1 year ago 1:46am 14 March 2024 Automatically closed - issue fixed for 2 weeks with no activity.