tests/src/Functional/CommentTest.php | 9 +++++++-- tests/src/Functional/ResourceTestBase.php | 16 ++++++++++++---- tests/src/Functional/TermTest.php | 16 +++++++++++++--- tests/src/Functional/UserTest.php | 13 ++++++++++--- 4 files changed, 42 insertions(+), 12 deletions(-) diff --git a/tests/src/Functional/CommentTest.php b/tests/src/Functional/CommentTest.php index faa5f40..2ae8661 100644 --- a/tests/src/Functional/CommentTest.php +++ b/tests/src/Functional/CommentTest.php @@ -6,6 +6,7 @@ use Drupal\comment\Entity\Comment; use Drupal\comment\Entity\CommentType; use Drupal\comment\Tests\CommentTestTrait; use Drupal\Component\Serialization\Json; +use Drupal\Component\Utility\NestedArray; use Drupal\Core\Cache\Cache; use Drupal\Core\Url; use Drupal\entity_test\Entity\EntityTest; @@ -294,7 +295,9 @@ class CommentTest extends ResourceTestBase { $this->setUpAuthorization('POST'); $url = Url::fromRoute(sprintf('jsonapi.%s.collection', static::$resourceTypeName)); - $request_options = $this->getAuthenticationRequestOptions('POST'); + $request_options = []; + $request_options[RequestOptions::HEADERS]['Accept'] = 'application/vnd.api+json'; + $request_options = NestedArray::mergeDeep($request_options, $this->getAuthenticationRequestOptions('POST')); $remove_field = function(array $normalization, $type, $attribute_name) { unset($normalization['data'][$type][$attribute_name]); @@ -340,7 +343,9 @@ class CommentTest extends ResourceTestBase { $this->setUpAuthorization('POST'); // Create request. - $request_options = $this->getAuthenticationRequestOptions('POST'); + $request_options = []; + $request_options[RequestOptions::HEADERS]['Accept'] = 'application/vnd.api+json'; + $request_options = NestedArray::mergeDeep($request_options, $this->getAuthenticationRequestOptions('POST')); $request_options[RequestOptions::BODY] = Json::encode($this->getNormalizedPostEntity()); $url = Url::fromRoute('jsonapi.comment--comment.collection'); diff --git a/tests/src/Functional/ResourceTestBase.php b/tests/src/Functional/ResourceTestBase.php index ab47175..dda450e 100644 --- a/tests/src/Functional/ResourceTestBase.php +++ b/tests/src/Functional/ResourceTestBase.php @@ -582,7 +582,9 @@ abstract class ResourceTestBase extends BrowserTestBase { // @todo Remove line below in favor of commented line in https://www.drupal.org/project/jsonapi/issues/2878463. $url = Url::fromRoute(sprintf('jsonapi.%s.individual', static::$resourceTypeName), [static::$entityTypeId => $this->entity->uuid()]); /* $url = $this->entity->toUrl('jsonapi'); */ - $request_options = $this->getAuthenticationRequestOptions('GET'); + $request_options = []; + $request_options[RequestOptions::HEADERS]['Accept'] = 'application/vnd.api+json'; + $request_options = NestedArray::mergeDeep($request_options, $this->getAuthenticationRequestOptions('GET')); // DX: 403 when unauthorized. $response = $this->request('GET', $url, $request_options); @@ -778,7 +780,9 @@ abstract class ResourceTestBase extends BrowserTestBase { // error responses provide a good DX // - to eventually result in a well-formed request that succeeds. $url = Url::fromRoute(sprintf('jsonapi.%s.collection', static::$resourceTypeName)); - $request_options = $this->getAuthenticationRequestOptions('POST'); + $request_options = []; + $request_options[RequestOptions::HEADERS]['Accept'] = 'application/vnd.api+json'; + $request_options = NestedArray::mergeDeep($request_options, $this->getAuthenticationRequestOptions('POST')); // @todo Uncomment in https://www.drupal.org/project/jsonapi/issues/2943170. // @codingStandardsIgnoreStart @@ -984,7 +988,9 @@ abstract class ResourceTestBase extends BrowserTestBase { // @todo Remove line below in favor of commented line in https://www.drupal.org/project/jsonapi/issues/2878463. $url = Url::fromRoute(sprintf('jsonapi.%s.individual', static::$resourceTypeName), [static::$entityTypeId => $this->entity->uuid()]); /* $url = $this->entity->toUrl('jsonapi'); */ - $request_options = $this->getAuthenticationRequestOptions('PATCH'); + $request_options = []; + $request_options[RequestOptions::HEADERS]['Accept'] = 'application/vnd.api+json'; + $request_options = NestedArray::mergeDeep($request_options, $this->getAuthenticationRequestOptions('PATCH')); // @todo Uncomment in https://www.drupal.org/project/jsonapi/issues/2943170. // @codingStandardsIgnoreStart @@ -1244,7 +1250,9 @@ abstract class ResourceTestBase extends BrowserTestBase { // @todo Remove line below in favor of commented line in https://www.drupal.org/project/jsonapi/issues/2878463. $url = Url::fromRoute(sprintf('jsonapi.%s.individual', static::$resourceTypeName), [static::$entityTypeId => $this->entity->uuid()]); /* $url = $this->entity->toUrl('jsonapi'); */ - $request_options = $this->getAuthenticationRequestOptions('PATCH'); + $request_options = []; + $request_options[RequestOptions::HEADERS]['Accept'] = 'application/vnd.api+json'; + $request_options = NestedArray::mergeDeep($request_options, $this->getAuthenticationRequestOptions('DELETE')); // DX: 403 when unauthorized. $response = $this->request('DELETE', $url, $request_options); diff --git a/tests/src/Functional/TermTest.php b/tests/src/Functional/TermTest.php index 745421a..2e0034a 100644 --- a/tests/src/Functional/TermTest.php +++ b/tests/src/Functional/TermTest.php @@ -3,6 +3,7 @@ namespace Drupal\Tests\jsonapi\Functional; use Drupal\Component\Serialization\Json; +use Drupal\Component\Utility\NestedArray; use Drupal\Core\Url; use Drupal\taxonomy\Entity\Term; use Drupal\taxonomy\Entity\Vocabulary; @@ -286,16 +287,18 @@ class TermTest extends ResourceTestBase { // @todo Remove line below in favor of commented line in https://www.drupal.org/project/jsonapi/issues/2878463. $url = Url::fromRoute(sprintf('jsonapi.%s.individual', static::$resourceTypeName), [static::$entityTypeId => $this->entity->uuid()]); /* $url = $this->entity->toUrl('jsonapi'); */ + $request_options = []; + $request_options[RequestOptions::HEADERS]['Accept'] = 'application/vnd.api+json'; + $request_options = NestedArray::mergeDeep($request_options, $this->getAuthenticationRequestOptions('GET')); // GET term's current normalization. - $response = $this->request('GET', $url, $this->getAuthenticationRequestOptions('GET')); + $response = $this->request('GET', $url, $request_options); $normalization = Json::decode((string) $response->getBody()); // Change term's path alias. $normalization['data']['attributes']['path']['alias'] .= 's-rule-the-world'; // Create term PATCH request. - $request_options = $this->getAuthenticationRequestOptions('PATCH'); $request_options[RequestOptions::BODY] = Json::encode($normalization); // PATCH request: 200. @@ -335,6 +338,11 @@ class TermTest extends ResourceTestBase { * @dataProvider providerTestGetIndividualTermWithParent */ public function testGetIndividualTermWithParent(array $parent_term_ids) { + if (floatval(\Drupal::VERSION) < 8.6) { + $this->markTestSkipped('The "parent" field on terms is only available for normalization in Drupal 8.6 and later.'); + return; + } + // Create all possible parent terms. Term::create(['vid' => Vocabulary::load('camelids')->id()]) ->setName('Lamoids') @@ -349,7 +357,9 @@ class TermTest extends ResourceTestBase { // @todo Remove line below in favor of commented line in https://www.drupal.org/project/jsonapi/issues/2878463. $url = Url::fromRoute(sprintf('jsonapi.%s.individual', static::$resourceTypeName), [static::$entityTypeId => $this->entity->uuid()]); /* $url = $this->entity->toUrl('jsonapi'); */ - $request_options = $this->getAuthenticationRequestOptions('GET'); + $request_options = []; + $request_options[RequestOptions::HEADERS]['Accept'] = 'application/vnd.api+json'; + $request_options = NestedArray::mergeDeep($request_options, $this->getAuthenticationRequestOptions('GET')); $this->setUpAuthorization('GET'); $response = $this->request('GET', $url, $request_options); $expected = $this->getExpectedNormalizedEntity(); diff --git a/tests/src/Functional/UserTest.php b/tests/src/Functional/UserTest.php index 67d2af4..8567e00 100644 --- a/tests/src/Functional/UserTest.php +++ b/tests/src/Functional/UserTest.php @@ -3,6 +3,7 @@ namespace Drupal\Tests\jsonapi\Functional; use Drupal\Component\Serialization\Json; +use Drupal\Component\Utility\NestedArray; use Drupal\Core\Url; use Drupal\jsonapi\Normalizer\HttpExceptionNormalizer; use Drupal\Tests\rest\Functional\BcTimestampNormalizerUnixTestTrait; @@ -183,7 +184,9 @@ class UserTest extends ResourceTestBase { // @todo Remove line below in favor of commented line in https://www.drupal.org/project/jsonapi/issues/2878463. $url = Url::fromRoute(sprintf('jsonapi.user--user.individual'), ['user' => $this->account->uuid()]); /* $url = $this->account->toUrl('jsonapi'); */ - $request_options = $this->getAuthenticationRequestOptions('PATCH'); + $request_options = []; + $request_options[RequestOptions::HEADERS]['Accept'] = 'application/vnd.api+json'; + $request_options = NestedArray::mergeDeep($request_options, $this->getAuthenticationRequestOptions('PATCH')); // Test case 1: changing email. $normalization = $original_normalization; @@ -261,7 +264,9 @@ class UserTest extends ResourceTestBase { // Update password in $this->account, prepare for future requests. $this->account->passRaw = $new_password; - $request_options = $this->getAuthenticationRequestOptions('PATCH'); + $request_options = []; + $request_options[RequestOptions::HEADERS]['Accept'] = 'application/vnd.api+json'; + $request_options = NestedArray::mergeDeep($request_options, $this->getAuthenticationRequestOptions('PATCH')); // Test case 3: changing name. $normalization = $original_normalization; @@ -335,7 +340,9 @@ class UserTest extends ResourceTestBase { // @todo Remove line below in favor of commented line in https://www.drupal.org/project/jsonapi/issues/2878463. $url = Url::fromRoute(sprintf('jsonapi.user--user.individual'), ['user' => $this->account->uuid()]); /* $url = $this->account->toUrl('jsonapi'); */ - $request_options = $this->getAuthenticationRequestOptions('PATCH'); + $request_options = []; + $request_options[RequestOptions::HEADERS]['Accept'] = 'application/vnd.api+json'; + $request_options = NestedArray::mergeDeep($request_options, $this->getAuthenticationRequestOptions('PATCH')); $normalization = $original_normalization; $normalization['data']['attributes']['mail'] = 'new-email@example.com';