Answer label creation with 201 without Location

docs/sdd/sdd-005-api-konvence.md

@@ -25,9 +25,9 @@ Tvary API a klienta.
 - Přihlášení nese cookie, nebo API token v hlavičce `Authorization: Bearer` (SDD-006). OpenAPI
   dokument deklaruje schéma Bearer.
 - Založení odpovídá 201 s hlavičkou `Location` na detail nového záznamu; u souboru a verze míří na
-  stažení obsahu, detail souboru není. Založení API tokenu odpovídá 201 bez `Location`, protože
-  token nemá detail. Upload, který nic neuloží, protože se obsah shoduje s aktuální verzí,
-  odpovídá 200. Komentář odpovídá 204 i na založení. Editace a smazání odpovídají 204.
+  stažení obsahu, detail souboru není. Založení záznamu, který nemá detail, odpovídá 201 bez
+  `Location`: API token a štítek. Upload, který nic neuloží, protože se obsah shoduje s aktuální
+  verzí, odpovídá 200. Komentář odpovídá 204 i na založení. Editace a smazání odpovídají 204.
 - V těle jde hodnota výčtu jako název; číslo ani neznámý název se nepřijme (400). V query
   stringu ji váže framework, který číslo bere.
 - Každý výpis bere stránku, směr řazení, svůj řadicí klíč a hledaný text; k tomu zúžení své

src/Api/EvilCase.Api/Controllers/LabelsController.cs

@@ -31,7 +31,7 @@ public async Task<ActionResult<LabelItem>> CreateLabel([FromServices] ILabelWrit
 
         return result.Outcome switch
         {
-            LabelCreateOutcome.Created => this.CreatedAtAction(nameof(this.ListLabels), routeValues: null, result.Label),
+            LabelCreateOutcome.Created => this.StatusCode(StatusCodes.Status201Created, result.Label),
             LabelCreateOutcome.NameTaken => this.NameTakenProblem(),
             _ => throw new UnreachableException(),
         };

src/Tests/EvilCase.Tests/Hosting/CreateLocationTests.cs

@@ -2,10 +2,13 @@
 using System.Net.Http.Json;
 using EvilBrains.EvilCase.Api.Contract.Cases;
 using EvilBrains.EvilCase.Api.Contract.Contacts;
+using EvilBrains.EvilCase.Api.Contract.Labels;
 using EvilBrains.EvilCase.Business.Cases;
 using EvilBrains.EvilCase.Business.Contacts;
+using EvilBrains.EvilCase.Business.Labels;
 using EvilBrains.EvilCase.Domain.Cases;
 using EvilBrains.EvilCase.Domain.Contacts;
+using EvilBrains.EvilCase.Domain.Labels;
 using Microsoft.Extensions.DependencyInjection;
 
 namespace EvilBrains.EvilCase.Tests.Hosting;
@@ -56,6 +59,21 @@ public void SetUp()
                     Name = call.Arg<ContactEditRequest>().Name,
                 });
             services.AddSingleton(contactWriter);
+
+            var labelWriter = Substitute.For<ILabelWriter>();
+            labelWriter
+                .CreateLabel(Arg.Any<LabelEditRequest>(), Arg.Any<CancellationToken>())
+                .Returns(static call => new LabelCreateResult
+                {
+                    Outcome = LabelCreateOutcome.Created,
+                    Label = new LabelItem
+                    {
+                        LabelId = Guid.CreateVersion7(),
+                        Name = call.Arg<LabelEditRequest>().Name,
+                        Color = call.Arg<LabelEditRequest>().Color,
+                    },
+                });
+            services.AddSingleton(labelWriter);
         });
         this.client = this.host.CreateClient();
     }
@@ -108,4 +126,22 @@ public async Task AFiledContactCarriesTheLocationOfItsDetail()
                 "the Location names the detail route of the contact that was filed");
         }
     }
+
+    [Test]
+    public async Task ACreatedLabelCarriesNoLocation()
+    {
+        using var request = new HttpRequestMessage(HttpMethod.Post, new Uri("/api/labels", UriKind.Relative))
+        {
+            Headers = { Authorization = TestTokens.BearerFrom(this.host) },
+            Content = JsonContent.Create(new LabelEditRequest { Name = "Nový štítek", Color = LabelColor.Blue }),
+        };
+
+        using var response = await this.client.SendAsync(request);
+
+        using (Assert.EnterMultipleScope())
+        {
+            Assert.That(response.StatusCode, Is.EqualTo(HttpStatusCode.Created));
+            Assert.That(response.Headers.Location, Is.Null, "a label has no detail route for a Location to name");
+        }
+    }
 }