Change patch to post

Author: Bart Koelman

src/JsonApiDotNetCore/Controllers/JsonApiAtomicOperationsController.cs

@@ -31,7 +31,7 @@ public JsonApiAtomicOperationsController(IAtomicOperationsProcessor atomicOperat
         /// <param name="cancellationToken">Propagates notification that request handling should be canceled.</param>
         /// <example>
         /// <code>
-        /// PATCH /api/v1/operations HTTP/1.1
+        /// POST /api/v1/operations HTTP/1.1
         /// Content-Type: application/vnd.api+json
         /// 
         /// {
@@ -50,8 +50,8 @@ public JsonApiAtomicOperationsController(IAtomicOperationsProcessor atomicOperat
         /// }
         /// </code>
         /// </example>
-        [HttpPatch]
-        public virtual async Task<IActionResult> PatchOperationsAsync([FromBody] AtomicOperationsDocument doc, CancellationToken cancellationToken)
+        [HttpPost]
+        public virtual async Task<IActionResult> PostOperationsAsync([FromBody] AtomicOperationsDocument doc, CancellationToken cancellationToken)
         {
             if (doc == null) return new StatusCodeResult(422);
 

src/JsonApiDotNetCore/Middleware/JsonApiMiddleware.cs

@@ -72,7 +72,7 @@ public async Task Invoke(HttpContext httpContext,
         private static bool PathIsBulk(RouteValueDictionary routeValues)
         {
             var actionName = (string)routeValues["action"];
-            return actionName == "PatchOperations";
+            return actionName == "PostOperations";
         }
 
         private static ResourceContext CreatePrimaryResourceContext(RouteValueDictionary routeValues,

test/OperationsExampleTests/Add/AddTests.cs

@@ -43,7 +43,7 @@ public async Task Can_Create_Author()
             };
 
             // act
-            var (response, data) = await _fixture.PatchAsync<AtomicOperationsDocument>("api/v1/operations", content);
+            var (response, data) = await _fixture.PostAsync<AtomicOperationsDocument>("api/v1/operations", content);
 
             // assert
             Assert.NotNull(response);
@@ -84,7 +84,7 @@ public async Task Can_Create_Authors()
             }
 
             // act
-            var (response, data) = await _fixture.PatchAsync<AtomicOperationsDocument>("api/v1/operations", content);
+            var (response, data) = await _fixture.PostAsync<AtomicOperationsDocument>("api/v1/operations", content);
 
             // assert
             Assert.NotNull(response);
@@ -137,7 +137,7 @@ public async Task Can_Create_Article_With_Existing_Author()
             };
 
             // act
-            var (response, data) = await _fixture.PatchAsync<AtomicOperationsDocument>("api/v1/operations", content);
+            var (response, data) = await _fixture.PostAsync<AtomicOperationsDocument>("api/v1/operations", content);
 
             // assert
             Assert.NotNull(response);
@@ -205,7 +205,7 @@ public async Task Can_Create_Articles_With_Existing_Author()
             }
 
             // act
-            var (response, data) = await _fixture.PatchAsync<AtomicOperationsDocument>("api/v1/operations", content);
+            var (response, data) = await _fixture.PostAsync<AtomicOperationsDocument>("api/v1/operations", content);
 
             // assert
             Assert.NotNull(response);
@@ -268,7 +268,7 @@ public async Task Can_Create_Author_With_Article_Using_LocalId()
             };
 
             // act
-            var (response, data) = await _fixture.PatchAsync<AtomicOperationsDocument>("api/v1/operations", content);
+            var (response, data) = await _fixture.PostAsync<AtomicOperationsDocument>("api/v1/operations", content);
 
             // assert
             Assert.NotNull(response);

test/OperationsExampleTests/Remove/RemoveTests.cs

@@ -40,7 +40,7 @@ public async Task Can_Remove_Author()
             };
 
             // act
-            var (response, data) = await _fixture.PatchAsync<AtomicOperationsDocument>("api/v1/operations", content);
+            var (response, data) = await _fixture.PostAsync<AtomicOperationsDocument>("api/v1/operations", content);
 
             // assert
             Assert.NotNull(response);
@@ -74,7 +74,7 @@ public async Task Can_Remove_Authors()
                 );
 
             // act
-            var (response, data) = await _fixture.PatchAsync<AtomicOperationsDocument>("api/v1/operations", content);
+            var (response, data) = await _fixture.PostAsync<AtomicOperationsDocument>("api/v1/operations", content);
 
             // assert
             Assert.NotNull(response);

test/OperationsExampleTests/TestFixture.cs

@@ -52,17 +52,17 @@ public void Dispose()
             }
         }
 
-        public async Task<(HttpResponseMessage response, T data)> PatchAsync<T>(string route, object data)
+        public async Task<(HttpResponseMessage response, T data)> PostAsync<T>(string route, object data)
         {
-            var response = await PatchAsync(route, data);
+            var response = await PostAsync(route, data);
             var json = await response.Content.ReadAsStringAsync();
             var obj = JsonConvert.DeserializeObject<T>(json);
             return (response, obj);
         }
 
-        private async Task<HttpResponseMessage> PatchAsync(string route, object data)
+        private async Task<HttpResponseMessage> PostAsync(string route, object data)
         {
-            var httpMethod = new HttpMethod("PATCH");
+            var httpMethod = HttpMethod.Post;
             var request = new HttpRequestMessage(httpMethod, route);
             request.Content = new StringContent(JsonConvert.SerializeObject(data));
             request.Content.Headers.ContentLength = 1;

test/OperationsExampleTests/Transactions/TransactionFailureTests.cs

@@ -66,7 +66,7 @@ public async Task Cannot_Create_Author_If_Article_Creation_Fails()
             };
 
             // act
-            var (response, data) = await _fixture.PatchAsync<ErrorDocument>("api/v1/operations", content);
+            var (response, data) = await _fixture.PostAsync<ErrorDocument>("api/v1/operations", content);
 
             // assert
             Assert.NotNull(response);

test/OperationsExampleTests/Update/UpdateTests.cs

@@ -52,7 +52,7 @@ public async Task Can_Update_Author()
             };
 
             // act
-            var (response, data) = await _fixture.PatchAsync<AtomicOperationsDocument>("api/v1/operations", content);
+            var (response, data) = await _fixture.PostAsync<AtomicOperationsDocument>("api/v1/operations", content);
 
             // assert
             Assert.NotNull(response);
@@ -99,7 +99,7 @@ public async Task Can_Update_Authors()
                 });
 
             // act
-            var (response, data) = await _fixture.PatchAsync<AtomicOperationsDocument>("api/v1/operations", content);
+            var (response, data) = await _fixture.PostAsync<AtomicOperationsDocument>("api/v1/operations", content);
 
             // assert
             Assert.NotNull(response);