Closed (fixed)
Project:
Drupal core
Version:
11.x-dev
Component:
documentation
Priority:
Normal
Category:
Task
Assigned:
Unassigned
Issue tags:
Reporter:
Created:
21 Jul 2024 at 23:38 UTC
Updated:
2 Dec 2024 at 10:44 UTC
Jump to comment: Most recent
Comments
Comment #2
quietone commentedComment #4
rodrigoaguileraLook at this commit
https://git.drupalcode.org/project/drupal/-/commit/c70620dc49df2ca821857...
and write similar guidance about where core tests(not in modules) should be located.
Comment #5
ultimikeIs the following correct for core lib tests?
-mike
Comment #6
alexpott@ultimike if by core lib tests you mean testing things in
core/lib/Drupal/Corethen yes. Things are a bit messy though.\Drupal\FunctionalJavascriptTests\Ajax\AjaxCallbacksTestis testing core stuff for example. And \Drupal\FunctionalJavascriptTests\Components\ComponentRenderTest is not aboutcore/lib/Drupal/ComponentThe unit test directory is probably the most sorted... where tests for
core/lib/Drupal/Corelive incore/tests/Drupal/Tests/Coreand tests forcore/lib/Drupal/Componentlive incore/tests/Drupal/Tests/ComponentComment #9
ultimikeComment #10
ultimikeTests are passing. Needs review.
-mike
Comment #11
ctrladelLooks good to me.
Comment #12
larowlanRequeued tests, added question about another possible enhancement we could add here
Comment #13
smustgrave commentedProbably does need a rebase now, 150+ commits back, not that I think the comments will fail a test.
But there is also an open thread about more improvements and can't see a reason off the top of my head not to.
Comment #14
mattlc commentedThank you @larowlan & @smustgrave for your comments.
I rebased on 11.x and added missing part for contrib modules in BuildTestBase class.
Moving issue to Needs review.
Comment #15
mattlc commentedI'm sorry, I don't know why functional tests do not pass.
As an example : Drupal\Tests\layout_builder\Functional\LayoutSection gives following error :
As it does not relates on this issue, I don't know how to get the whole test pipeline work.
Comment #16
quietone commented@mattlc, thanks for rebasing this. Some of the tests often fail randomly, and this is what you have encountered. We keep track of the random fails and in this issue, #2829040: [meta] Known intermittent, random, and environment-specific test failures. That is where anyone who wants to the challenge of fixing a randomly failing test can find one to work on.
I have restarted the failing tests and they have come back passing this time. You should also be able to restart the tests using the GitLab UI.
Comment #17
smustgrave commentedBelieve feedback has been addressed for this one.
Comment #18
dwwSpotted one problem with existing docs right next to the ones we're adding in
core/tests/Drupal/Tests/UnitTestCase.php. Should we fix those here or in a follow-up?Comment #19
dwwI should have added: the new docs look great! Big improvement. Thanks for working on this!
Comment #20
mattlc commentedFixed #14 (& #15). CI is running.
Comment #21
smustgrave commentedThanks @dww! Believe feedback has been addressed.
Comment #22
alexpottCommitted 6025a14 and pushed to 11.x. Thanks!
Committed f54e0fc and pushed to 11.1.x. Thanks!