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.
Comment | File | Size | Author |
---|---|---|---|
#6 | 2865413-6.patch | 916 bytes | jofitz |
#6 | interdiff-4-6.txt | 665 bytes | jofitz |
#4 | 2865413-4.patch | 864 bytes | jofitz |
#4 | interdiff-2-4.txt | 1009 bytes | jofitz |
#2 | convert_web_tests_to-2865413-2.patch | 2.24 KB | GoZ |
Comments
Comment #2
GoZ CreditAttribution: GoZ at Barbe-Rousse, Centarro commentedComment #3
klausithis is a base class and must not be moved. It is already deprecated. We can just leave it alone.
lol, this file is in the completely wrong place. This is a unit test, not a functional test, so must be moved to tests/src/Unit. This test was introduced in #1996130: REST views: not adding dependencies on Serializer providers.
So we can repurpose this issue to just move the unit test into the correct location.
Comment #4
jofitz CreditAttribution: jofitz at ComputerMinds commentedComment #5
klausiBut we have nice subdirectories in the Unit folder, so I would suggest to move this into a CompilerPass directory there.
Comment #6
jofitz CreditAttribution: jofitz at ComputerMinds commentedMoved the file and updated the namespace.
Comment #7
dawehnerGood idea to put it into a folder!
Comment #8
alexpottCommitted and pushed e2554ce to 8.4.x and e82e3eb to 8.3.x. Thanks!
Test only change - committed to 8.3.x too.