Consistent POST/PUT behaviour

src/IIIFPresentation/API.Tests/Converters/FastJsonPropertyReadTests.cs

@@ -0,0 +1,106 @@
+using API.Converters;
+
+namespace API.Tests.Converters;
+
+public class FastJsonPropertyReadTests
+{
+    [Fact]
+    public void FindAtLevel_FindsValue_DefaultTopLevel()
+    {
+        FastJsonPropertyRead.FindAtLevel(Json, "searchedProperty").Should()
+            .Be("fnord", "it's the value of 'searchedProperty' in sample JSON");
+    }
+
+    [Fact]
+    public void FindAtLevel_FindsValue_Deeper()
+    {
+        FastJsonPropertyRead.FindAtLevel(Json, "priority", 3).Should()
+            .Be("high", "it's the value of first instance of 'priority' property in sample JSON");
+    }
+
+    [Fact]
+    public void FindAtLevel_ReturnsNull_IfNotFound()
+    {
+      FastJsonPropertyRead.FindAtLevel(Json, "i don't exist in the json").Should()
+        .BeNull("no such property in the JSON");
+    }
+
+    // just some arbitrary JSON - the method under test is not IIIF specific
+    private const string Json =
+
+        #region <sample json>
+
+        """
+        {
+          "id": 42,
+          "name": "Sample Root",
+          "enabled": true,
+          "tags": ["alpha", "beta", "gamma"],
+          "metadata": {
+            "created": "2025-02-15T12:34:56Z",
+            "updated": "2025-03-01T08:12:45Z",
+            "attributes": {
+              "priority": "high",
+              "flags": {
+                "archived": false,
+                "requiresReview": true,
+                "internal": {
+                  "level": 3,
+                  "notes": ["check format", "verify fields"]
+                }
+              }
+            }
+          },
+          "items": [
+            {
+              "id": "a1",
+              "quantity": 10,
+              "details": {
+                "manufacturer": "Acme Corp",
+                "dimensions": {
+                  "width": 10.5,
+                  "height": 20.0,
+                  "depth": 5.75
+                }
+              }
+            },
+            {
+              "id": "b2",
+              "quantity": 4,
+              "details": {
+                "manufacturer": "Globex",
+                "dimensions": {
+                  "width": 5.0,
+                  "height": 8.0,
+                  "depth": 1.25,
+                  "history": [
+                    {
+                      "revision": 1,
+                      "notes": { "editor": "system", "timestamp": "2025-01-01T00:00:00Z" }
+                    },
+                    {
+                      "revision": 2,
+                      "notes": { "editor": "qa", "timestamp": "2025-02-01T00:00:00Z" }
+                    }
+                  ]
+                }
+              }
+            }
+          ],
+          "config": {
+            "mode": "full",
+            "retry": 3,
+            "options": {
+              "validate": true,
+              "paths": [
+                { "name": "input", "value": "/var/data/in" },
+                { "name": "output", "value": "/var/data/out" }
+              ]
+            }
+          },
+          "searchedProperty": "fnord"
+        }
+        """;
+
+    #endregion
+}

src/IIIFPresentation/API.Tests/Converters/PresentationIIIFCleanerTests.cs

@@ -0,0 +1,87 @@
+using API.Converters;
+using IIIF.Presentation.V3;
+using IIIF.Presentation.V3.Content;
+using IIIF.Presentation.V3.Strings;
+using Models.API.Collection;
+using Models.API.Manifest;
+
+namespace API.Tests.Converters;
+
+public class PresentationIIIFCleanerTests
+{
+    [Fact]
+    public void TestCleanManifest()
+    {
+        var manifest = new PresentationManifest
+        {
+            // From IIIF
+            Id = "this/is/some/Id",
+            Label = new LanguageMap("en", "some label"),
+            Thumbnail =
+            [
+                new Image
+                {
+                    Id = "https://localhost/thumbs/12/23/blabla/full/143,200/0/default.jpg"
+                }
+            ],
+            Rights = "https://creativecommons.org/licenses/by/4.0/",
+            Items =
+            [
+                new Canvas
+                {
+                    Id = "some id"
+                }
+            ]
+            // outside IIIF
+            ,
+            Slug = "some slug",
+            Parent = "some parent",
+            PublicId = "some public id"
+        };
+
+        var clean = PresentationIIIFCleaner.OnlyIIIFProperties(manifest);
+
+        clean.Slug.Should().BeNull("not in IIIF");
+        clean.Parent.Should().BeNull("not in IIIF");
+        clean.PublicId.Should().BeNull("not in IIIF");
+
+        clean.Rights.Should().Be(manifest.Rights, "in the IIIF");
+        clean.Id.Should().Be(manifest.Id, "in the IIIF");
+        clean.Thumbnail.Should().BeEquivalentTo(manifest.Thumbnail, "in the IIIF");
+        clean.Items.Should().BeEquivalentTo(manifest.Items, "in the IIIF");
+    }
+
+    [Fact]
+    public void TestCleanCollection()
+    {
+        var collection = new PresentationCollection
+        {
+            // From IIIF
+            Id = "this/is/some/Id",
+            Label = new LanguageMap("en", "some label"),
+            Thumbnail =
+            [
+                new Image
+                {
+                    Id = "https://localhost/thumbs/12/23/blabla/full/143,200/0/default.jpg"
+                }
+            ],
+            Rights = "https://creativecommons.org/licenses/by/4.0/"
+            // outside IIIF
+            ,
+            Slug = "some slug",
+            Parent = "some parent",
+            PublicId = "some public id"
+        };
+
+        var clean = PresentationIIIFCleaner.OnlyIIIFProperties(collection);
+
+        clean.Slug.Should().BeNull("not in IIIF");
+        clean.Parent.Should().BeNull("not in IIIF");
+        clean.PublicId.Should().BeNull("not in IIIF");
+
+        clean.Rights.Should().Be(collection.Rights, "in the IIIF");
+        clean.Id.Should().Be(collection.Id, "in the IIIF");
+        clean.Thumbnail.Should().BeEquivalentTo(collection.Thumbnail, "in the IIIF");
+    }
+}

src/IIIFPresentation/API.Tests/Integration/Infrastructure/CollectionDefinition.cs

@@ -15,4 +15,10 @@ public class StorageCollection : ICollectionFixture<StorageFixture>
     {
         public const string CollectionName = "Storage Collection";
     }
-}
+
+    [CollectionDefinition(CollectionName, DisableParallelization = true)]
+    public class RestCollection : ICollectionFixture<StorageFixture>
+    {
+        public const string CollectionName = "Rest Collection";
+    }
+}

src/IIIFPresentation/API.Tests/Integration/Infrastructure/HttpRequestMessageBuilder.cs

@@ -15,6 +15,16 @@ public static HttpRequestMessage GetPrivateRequest(HttpMethod method, string pat
 
         return requestMessage;
     }
+
+    public static HttpRequestMessage GetPlainRequest(HttpMethod method, string path, string content,
+        Guid? etag = null)
+    {
+        var requestMessage = new HttpRequestMessage(method, path).WithJsonContent(content);
+        if (etag is not null)
+            requestMessage.Headers.IfMatch.Add(new EntityTagHeaderValue($"\"{etag:N}\""));
+
+        return requestMessage;
+    }
 
     public static HttpRequestMessage GetPrivateRequest(HttpMethod method, string path, Guid? etag = null)
     {