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.
https://www.drupal.org/pift-ci-job/838384 tests JSON API against PHP 7.1 + the next Drupal 8 minor (8.5 currently).
- Nov 30: "299 pass"
- Dec 1: "No tests found"
- All daily test runs since then: "No tests found"
I dug into the DrupalCI console output. It contains this:
00:01:47.042 PHP Fatal error: Class Drupal\Tests\jsonapi\Functional\RestJsonApiUnsupported contains 1 abstract method and must therefore be declared abstract or implement the remaining methods (Drupal\Tests\rest\Functional\ResourceTestBase::getExpectedUnauthorizedAccessCacheability) in /var/www/html/modules/contrib/jsonapi/tests/src/Functional/RestJsonApiUnsupported.php on line 15
Comment | File | Size | Author |
---|---|---|---|
#4 | 2931831-4.patch | 636 bytes | Wim Leers |
Comments
Comment #2
Wim Leers#2765959: Make 4xx REST responses cacheable by (Dynamic) Page Cache + comprehensive cacheability test coverage introduced a new abstract method on a base class that this test is using. We need to update JSON API's test. This will work fine on Drupal 8.4 too.
Comment #3
Wim LeersComment #4
Wim LeersComment #5
Wim LeersApparently something changed in 8.5 that causes tests to fail. But since A) daily tests haven't been running, B) JSON API isn't tested against 8.5 in issues, we've been unaware!
Committing this fix, because it's better to be aware about a breakage than continuing to ignore it.
Comment #6
Wim LeersYou may wonder "why the hell do we have to implement this?"
The reason is that REST test coverage should think about this. It's just that this particular REST test doesn't need to care.
Comment #8
Wim LeersComment #9
e0ipsoThanks for taking care of this!