From a241714125531f415a4141dd13b6e86d56d2f010 Mon Sep 17 00:00:00 2001 From: Clint Branham Date: Wed, 25 Mar 2026 15:32:25 -0500 Subject: [PATCH] fix: update CloudInit regenerate test for PUT endpoint Test was still expecting GetAsync on cloudinit/dump but the service now calls PutAsync on cloudinit (regenerate endpoint). Updated mock setup and assertions to match. Co-Authored-By: Claude Opus 4.6 (1M context) --- .../Services/CloudInitServiceTests.cs | 11 +++++------ 1 file changed, 5 insertions(+), 6 deletions(-) diff --git a/tests/PSProxmoxVE.Core.Tests/Services/CloudInitServiceTests.cs b/tests/PSProxmoxVE.Core.Tests/Services/CloudInitServiceTests.cs index 50e2c76..fd898bc 100644 --- a/tests/PSProxmoxVE.Core.Tests/Services/CloudInitServiceTests.cs +++ b/tests/PSProxmoxVE.Core.Tests/Services/CloudInitServiceTests.cs @@ -100,12 +100,12 @@ namespace PSProxmoxVE.Core.Tests.Services } [Fact] - public void RegenerateCloudInitImage_CallsGetAsyncAndReturnsData() + public void RegenerateCloudInitImage_CallsPutAsyncAndReturnsUpid() { // Arrange - var json = @"{""data"": ""#cloud-config\nuser: ubuntu\npassword: secret\n""}"; + var json = @"{""data"": ""UPID:pve1:00001234:00005678:12345678:cloudinit:100:root@pam:""}"; var mockClient = new Mock(); - mockClient.Setup(c => c.GetAsync("nodes/pve1/qemu/100/cloudinit/dump?type=user")) + mockClient.Setup(c => c.PutAsync("nodes/pve1/qemu/100/cloudinit", null)) .ReturnsAsync(json); var service = new CloudInitService(mockClient.Object); @@ -113,9 +113,8 @@ namespace PSProxmoxVE.Core.Tests.Services var result = service.RegenerateCloudInitImage(CreateSession(), "pve1", 100); // Assert - Assert.Contains("#cloud-config", result); - Assert.Contains("ubuntu", result); - mockClient.Verify(c => c.GetAsync("nodes/pve1/qemu/100/cloudinit/dump?type=user"), Times.Once); + Assert.Contains("UPID:", result); + mockClient.Verify(c => c.PutAsync("nodes/pve1/qemu/100/cloudinit", null), Times.Once); } [Fact]