- Issue created by @vensires
- First commit to issue fork.
- Merge request !7555Issue #3439898: Fix Media tests that rely on UID1's super user behavior β (Closed) created by simonbaese
- Status changed to Needs review
11 months ago 8:38pm 17 April 2024 - π©πͺGermany simonbaese Berlin
The
ContentModerationTest
made a distinction between the user 1 and an admin user. Therefore, all tests forrootUser
were simply removed. - Status changed to RTBC
11 months ago 1:47pm 18 April 2024 - Status changed to Needs work
11 months ago 3:01pm 18 April 2024 - π¬π§United Kingdom alexpott πͺπΊπ
I'm not sure about the changes to core/modules/media_library/tests/src/FunctionalJavascript/ContentModerationTest.php it seems to be testing user 1 specific stuff. I think we need to carefully think about why the user 1 tests are here because we have the admin user here already.
- πΊπΈUnited States smustgrave
But with the addition of π Add a container parameter that can remove the special behavior of UID#1 Fixed should we be testing user1 stuff?
- First commit to issue fork.
- π³πΏNew Zealand quietone
I asked in Slack, #media, about why the test was specifically testing user #1. Two media maintainers, marcoscano and seanb responded saying that they saw no reason for it. seanb also said that they probably copied the test from somewhere else and then made it work for media.
So, I think it is OK to remove the specific assertions with user #1. Therefor restoring the RTBC.
- First commit to issue fork.
- π«π·France nod_ Lille
test failure that seems legit
Drupal\Tests\standard\FunctionalJavascript\StandardPerformanceTest::testStandardPerformance Failed asserting that two arrays are identical. --- Expected +++ Actual @@ @@ ], 'CacheSetCount' => 45, 'CacheDeleteCount' => 0, - 'CacheTagChecksumCount' => 38, - 'CacheTagIsValidCount' => 43, + 'CacheTagChecksumCount' => 37, + 'CacheTagIsValidCount' => 42, 'CacheTagInvalidationCount' => 0, 'CacheTagLookupQueryCount' => 21, 'CacheTagGroupedLookups' => Array &2 [ core/tests/Drupal/Tests/PerformanceTestTrait.php:679 core/profiles/standard/tests/src/FunctionalJavascript/StandardPerformanceTest.php:175 core/profiles/standard/tests/src/FunctionalJavascript/StandardPerformanceTest.php:57
- π«π·France nod_ Lille
appears random π Remove cache tag checksum assertions from performance tests Active