- Issue created by @VladimirAus
- Merge request !7Issue #3444027 by VladimirAus: Add CI to the module → (Merged) created by VladimirAus
- Status changed to Needs review
8 months ago 8:33am 4 May 2024 - 🇦🇺Australia VladimirAus Brisbane, Australia
PHPCs was fixed.
Created separate PHP Stan and PHP Unit fixes: ✨ Gitlab CI: PHP Stan and PHP Unit fixes Active - Status changed to RTBC
8 months ago 12:09am 6 May 2024 -
VladimirAus →
committed a086a295 on 8.x-1.x
Issue #3444027 by VladimirAus, jannakha: Add CI to the module
-
VladimirAus →
committed a086a295 on 8.x-1.x
- Status changed to Fixed
8 months ago 12:51pm 6 May 2024 - 🇮🇳India manish-31
This MR introduced an error when running the Test ParagraphListingTest.
Not re-opening the issue as this has been fixed with the MR changes done in https://www.drupal.org/project/paragraphs_admin/issues/3443579 ✨ Paragraph delete should have confirm form Needs work (under review).
Test output -
www-data@c06cb4a3c870:/app/web/core$ ../../vendor/bin/phpunit ../modules/contrib/paragraphs_admin/tests/src/Functional/ParagraphListingTest.php --debug PHPUnit 9.6.19 by Sebastian Bergmann and contributors. Testing Drupal\Tests\paragraphs_admin\Functional\ParagraphListingTest Test 'Drupal\Tests\paragraphs_admin\Functional\ParagraphListingTest::testParagraphListingPages' started Test 'Drupal\Tests\paragraphs_admin\Functional\ParagraphListingTest::testParagraphListingPages' ended Time: 00:10.387, Memory: 10.00 MB There was 1 error: 1) Drupal\Tests\paragraphs_admin\Functional\ParagraphListingTest::testParagraphListingPages TypeError: array_merge(): Argument #2 must be of type array, Drupal\user\Entity\User given /app/web/modules/contrib/paragraphs/tests/src/Functional/WidgetLegacy/ParagraphsTestBase.php:89 /app/web/modules/contrib/paragraphs_admin/tests/src/Functional/ParagraphListingTest.php:48 /app/vendor/phpunit/phpunit/src/Framework/TestResult.php:729 ERRORS! Tests: 1, Assertions: 9, Errors: 1.
Automatically closed - issue fixed for 2 weeks with no activity.