When running the tests on the new patches we get this deprecation notice:

  4x: Drupal\Tests\BrowserTestBase::$defaultTheme is required in drupal:9.0.0 when using an install profile that does not set a default theme. See https://www.drupal.org/node/2352949, which includes recommendations on which theme to use.
    1x in JsonapiResourceTest::testAddReminderResource from Drupal\Tests\jsonapi_resources\Functional
    1x in JsonapiResourceTest::testFeaturedNodesResource from Drupal\Tests\jsonapi_resources\Functional
    1x in JsonapiResourceTest::testAuthorContentResource from Drupal\Tests\jsonapi_resources\Functional
    1x in JsonapiResourceTest::testCurrentUserInfoResource from Drupal\Tests\jsonapi_resources\Functional
CommentFileSizeAuthor
#2 3094974-2.patch481 bytessam711

Comments

sam711 created an issue. See original summary.

sam711’s picture

StatusFileSize
new481 bytes
sam711’s picture

Status: Active » Needs review
mglaman’s picture

Status: Needs review » Reviewed & tested by the community

Thanks! re-running HEAD tests to verify it's broken.

  • mglaman committed a40f3a0 on 8.x-1.x authored by sam711
    Issue #3094974 by sam711: Add defaultTheme to JsonapiResourceTest
    
mglaman’s picture

Status: Reviewed & tested by the community » Fixed

Thank you! HEAD will be green once more!

kristen pol’s picture

Issue tags: +Drupal 9 compatibility

Per a Slack discussion with Gábor Hojtsy regarding usage of D9 tags (Drupal 9, Drupal 9 compatibility, Drupal 9 readiness, etc.), "Drupal 9 compatibility" should be used for contributed projects that need updating and "Drupal 9" was the old tag for D8 issues before the D9 branch was ready. Doing tag cleanup here based on that discussion.

Status: Fixed » Closed (fixed)

Automatically closed - issue fixed for 2 weeks with no activity.