Fix #958 (#960)

CGI-FR · Jul 22, 2022 · 2981b3d · 2981b3d
1 parent 5229998
commit 2981b3d
Show file tree

Hide file tree

Showing 4 changed files with 12 additions and 12 deletions.
diff --git a/src/AzureIoTHub.Portal.Server.Tests.Unit/Services/DeviceModelsClientServiceTests.cs b/src/AzureIoTHub.Portal.Server.Tests.Unit/Services/DeviceModelsClientServiceTests.cs
@@ -201,12 +201,10 @@ public async Task ChangeAvatarPropertiesShouldChangeAvatar()
             var deviceModel = Fixture.Create<DeviceModel>();
             var content = new MultipartFormDataContent();
 
-            _ = MockHttpClient.When(HttpMethod.Get, $"/api/models/{deviceModel.ModelId}/properties")
+            _ = MockHttpClient.When(HttpMethod.Post, $"/api/models/{deviceModel.ModelId}/avatar")
                 .With(m =>
                 {
-                    _ = m.Content.Should().BeAssignableTo<ObjectContent<MultipartFormDataContent>>();
-                    var body = m.Content as ObjectContent<MultipartFormDataContent>;
-                    _ = body.Value.Should().BeEquivalentTo(content);
+                    _ = m.Content.Should().BeEquivalentTo(content);
                     return true;
                 })
                 .Respond(HttpStatusCode.Created);

diff --git a/src/AzureIoTHub.Portal.Server.Tests.Unit/Services/LoRaWanDeviceModelsClientServiceTests.cs b/src/AzureIoTHub.Portal.Server.Tests.Unit/Services/LoRaWanDeviceModelsClientServiceTests.cs
@@ -190,12 +190,10 @@ public async Task ChangeAvatarPropertiesShouldChangeAvatar()
             };
             var content = new MultipartFormDataContent();
 
-            _ = MockHttpClient.When(HttpMethod.Get, $"/api/lorawan/models/{deviceModel.ModelId}/properties")
+            _ = MockHttpClient.When(HttpMethod.Post, $"/api/lorawan/models/{deviceModel.ModelId}/avatar")
                 .With(m =>
                 {
-                    _ = m.Content.Should().BeAssignableTo<ObjectContent<MultipartFormDataContent>>();
-                    var body = m.Content as ObjectContent<MultipartFormDataContent>;
-                    _ = body.Value.Should().BeEquivalentTo(content);
+                    _ = m.Content.Should().BeEquivalentTo(content);
                     return true;
                 })
                 .Respond(HttpStatusCode.Created);

diff --git a/src/AzureIoTHub.Portal/Client/Services/DeviceModelsClientService.cs b/src/AzureIoTHub.Portal/Client/Services/DeviceModelsClientService.cs
@@ -58,9 +58,11 @@ public Task<string> GetAvatarUrl(string deviceModelId)
             return this.http.GetStringAsync($"api/models/{deviceModelId}/avatar");
         }
 
-        public Task ChangeAvatar(string deviceModelId, MultipartFormDataContent avatar)
+        public async Task ChangeAvatar(string deviceModelId, MultipartFormDataContent avatar)
         {
-            return this.http.PostAsJsonAsync($"api/models/{deviceModelId}/avatar", avatar);
+            var result = await this.http.PostAsync($"api/models/{deviceModelId}/avatar", avatar);
+
+            _ = result.EnsureSuccessStatusCode();
         }
     }
 }
diff --git a/src/AzureIoTHub.Portal/Client/Services/LoRaWanDeviceModelsClientService.cs b/src/AzureIoTHub.Portal/Client/Services/LoRaWanDeviceModelsClientService.cs
@@ -48,9 +48,11 @@ public Task<string> GetAvatarUrl(string deviceModelId)
             return this.http.GetStringAsync($"api/lorawan/models/{deviceModelId}/avatar");
         }
 
-        public Task ChangeAvatar(string deviceModelId, MultipartFormDataContent avatar)
+        public async Task ChangeAvatar(string deviceModelId, MultipartFormDataContent avatar)
         {
-            return this.http.PostAsJsonAsync($"api/lorawan/models/{deviceModelId}/avatar", avatar);
+            var result = await this.http.PostAsync($"api/lorawan/models/{deviceModelId}/avatar", avatar);
+
+            _ = result.EnsureSuccessStatusCode();
         }
     }
 }