Support for Drupal 7 is ending on 5 January 2025—it’s time to migrate to Drupal 10! Learn about the many benefits of Drupal 10 and find migration tools in our resource center.
Problem/Motivation
Speed up testbot and simplify tests by using the testing profile instead of the standard profile.
Steps to reproduce
Proposed resolution
Convert Drupal\node\Tests\NodeAccessBaseTableTest to use the testing profile, not the standard profile.
Remaining tasks
Update patch
Review
Commit
User interface changes
N/A
API changes
N/A
Data model changes
N/A
Release notes snippet
N/A
Comment | File | Size | Author |
---|---|---|---|
#25 | interdiff-23-25.txt | 968 bytes | jungle |
#25 | 2254201-25.patch | 1.25 KB | jungle |
| |||
#23 | 2254201-23.patch | 936 bytes | smustgrave |
| |||
#23 | interdiff-19-23.txt | 939 bytes | smustgrave |
Comments
Comment #8
volegerComment #9
vacho CreditAttribution: vacho at Skilld commentedChanges on /core/scripts/run-tests.sh
And changes on /core/modules/system/lib/Drupal/system/Tests/Theme/EntityFilteringThemeTest.php
These only restring the list for test. I think that it is not a real enhancement on performance.
BTW the file EntityFilteringThemeTest.php now is at /core/modules/system/tests/src/Functional/Theme/EntityFilteringThemeTest.php
Comment #10
kostyashupenkoComment #16
quietone CreditAttribution: quietone at PreviousNext commentedAdding issue summary and re-parenting.
Comment #17
smustgrave CreditAttribution: smustgrave at Mobomo commentedThis may fail but we will see.
Comment #19
smustgrave CreditAttribution: smustgrave at Mobomo commentedIgnore #17 completely wrong patch.
Comment #21
smustgrave CreditAttribution: smustgrave at Mobomo commentedNot sure why it said it failed.
Comment #22
smustgrave CreditAttribution: smustgrave at Mobomo commentedInstalling olivero_test because it ships with an article content type with body, comment, tags, and image field, teaser view mode, etc which were being tested here.
Comment #23
smustgrave CreditAttribution: smustgrave at Mobomo commentedUpdated patch for 10.1.x
Comment #24
smustgrave CreditAttribution: smustgrave at Mobomo commentedShould move away from olivero_test and create what we need.
Comment #25
jungleComment #26
smustgrave CreditAttribution: smustgrave at Mobomo commentedChanges look good!
Comment #28
catchCommitted 8c9110d and pushed to 10.1.x. Thanks!