Add parsing tests for atomic extension

Author: Art4

tests/Functional/ParsingTest.php

@@ -65,6 +65,7 @@ public function testParseSimpleResourceWithDifferentParser(callable $parser): vo
         $this->assertFalse($document->has('meta'));
         $this->assertFalse($document->has('jsonapi'));
         $this->assertFalse($document->has('links'));
+        $this->assertFalse($document->has('relationships'));
         $this->assertFalse($document->has('included'));
         $this->assertTrue($document->has('data'));
 
@@ -76,9 +77,12 @@ public function testParseSimpleResourceWithDifferentParser(callable $parser): vo
         $this->assertTrue($resource->has('id'));
         $this->assertSame($resource->get('id'), '1');
         $this->assertTrue($resource->has('attributes'));
-        $this->assertInstanceOf(Attributes::class, $resource->get('attributes'));
-        $this->assertTrue($resource->has('relationships'));
-        $this->assertInstanceOf(RelationshipCollection::class, $resource->get('relationships'));
+
+        $attributes = $resource->get('attributes');
+
+        $this->assertInstanceOf(Attributes::class, $attributes);
+        $this->assertTrue($attributes->has('title'));
+        $this->assertSame($attributes->get('title'), 'JSON API paints my bikeshed!');
     }
 
     public function testParseSimpleResourceIdentifier(): void

tests/Functional/SerializerTest.php

@@ -47,7 +47,7 @@ public static function jsonapiDataProvider(): array
         foreach ($filenames as $file) {
             $filename = str_replace($path, '', $file);
 
-            $files[] = [
+            $files[$filename] = [
                 $filename,
                 in_array($filename, $requestFiles),
             ];
@@ -71,12 +71,10 @@ public function testParseJsonapiDataWithErrorAbortManager(string $filename, bool
 
         $document = $manager->parse($input);
 
-        $expected = json_decode($string, true);
-
         // Test full array
-        $this->assertEquals(
-            $expected,
-            (new ArraySerializer(['recursive' => true]))->serialize($document),
+        $this->assertJsonStringEqualsJsonString(
+            $string,
+            json_encode((new ArraySerializer(['recursive' => true]))->serialize($document), JSON_PRETTY_PRINT),
             $filename
         );
     }

tests/files/01_simple_resource.json

@@ -3,8 +3,7 @@
     "type": "articles",
     "id": "1",
     "attributes": {
-    },
-    "relationships": {
+      "title": "JSON API paints my bikeshed!"
     }
   }
-}
+}

tests/files/02_simple_resource_identifier.json

@@ -3,4 +3,4 @@
     "type": "articles",
     "id": "1"
   }
-}
+}

tests/files/18_jsonapi_object_with_ext_profile.json

@@ -1,13 +1,15 @@
 {
-    "meta": {},
-    "jsonapi": {
-      "version": "1.1",
-      "ext": [
-        "https://jsonapi.org/ext/atomic"
-      ],
-      "profile": [
-        "http://example.com/profiles/flexible-pagination",
-        "http://example.com/profiles/resource-versioning"
-      ]
-    }
+  "meta": {
+    "note": "BC 1.0: A document MUST contain at least one of the following top-level members: data, errors, meta"
+  },
+  "jsonapi": {
+    "version": "1.1",
+    "ext": [
+      "https://jsonapi.org/ext/atomic"
+    ],
+    "profile": [
+      "http://example.com/profiles/flexible-pagination",
+      "http://example.com/profiles/resource-versioning"
+    ]
   }
+}

tests/files/21_ext_atomic_operations.json

@@ -0,0 +1,17 @@
+{
+  "atomic:operations": [
+    {
+      "op": "add",
+      "href": "/blogPosts",
+      "data": {
+        "type": "articles",
+        "attributes": {
+          "title": "JSON API paints my bikeshed!"
+        }
+      }
+    }
+  ],
+  "meta": {
+    "note": "BC 1.0: A document MUST contain at least one of the following top-level members: data, errors, meta"
+  }
+}

tests/files/22_ext_atomic_results.json

@@ -0,0 +1,19 @@
+{
+  "atomic:results": [
+    {
+      "data": {
+        "links": {
+          "self": "http://example.com/blogPosts/13"
+        },
+        "type": "articles",
+        "id": "13",
+        "attributes": {
+          "title": "JSON API paints my bikeshed!"
+        }
+      }
+    }
+  ],
+  "meta": {
+    "note": "BC 1.0: A document MUST contain at least one of the following top-level members: data, errors, meta"
+  }
+}