mirror of
https://github.com/GoodOlClint/PSProxmoxVE.git
synced 2026-07-26 07:58:14 +00:00
fix: address PR #69 review feedback
- VmService.ExecuteGuestCommand: guard against null elements in -Args. The old JSON-serialization tolerated nulls (as "null"); the repeated-key path would NRE in EncodeFormValue. Throw a clear ArgumentException instead. - findings.json: refresh the stale counters block (untouched since F085) to the actual ledger state — next_id 92, resolved 83 — and bump last_updated to 2026-05-22. last_scan_date stays 2026-03-26 (F086–F091 came from issue triage, not a formal review scan). - VmServiceTests: add empty-array (single command entry) and null-element (throws) cases. Note: F091 is the correct next ID — F086–F090 already exist from prior merged PRs (#60/#61/#66/#67); only the counters were lagging. Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
This commit is contained in:
@@ -1,3 +1,4 @@
|
||||
using System;
|
||||
using System.Collections.Generic;
|
||||
using System.Linq;
|
||||
using Moq;
|
||||
@@ -77,5 +78,36 @@ namespace PSProxmoxVE.Core.Tests.Services
|
||||
Assert.Equal("command", only.Key);
|
||||
Assert.Equal("whoami", only.Value);
|
||||
}
|
||||
|
||||
[Fact]
|
||||
public void ExecuteGuestCommand_EmptyArgs_SendsSingleCommandEntry()
|
||||
{
|
||||
List<KeyValuePair<string, string>>? captured = null;
|
||||
var mockClient = new Mock<IPveHttpClient>();
|
||||
mockClient
|
||||
.Setup(c => c.PostAsync(It.IsAny<string>(), It.IsAny<IEnumerable<KeyValuePair<string, string>>>()))
|
||||
.Callback<string, IEnumerable<KeyValuePair<string, string>>>((_, data) => captured = data.ToList())
|
||||
.ReturnsAsync("{\"data\":{\"pid\":9}}");
|
||||
|
||||
var service = new VmService(mockClient.Object);
|
||||
service.ExecuteGuestCommand(CreateSession(), TestNode, TestVmId, "whoami", new string[0]);
|
||||
|
||||
Assert.NotNull(captured);
|
||||
var only = Assert.Single(captured!);
|
||||
Assert.Equal("command", only.Key);
|
||||
Assert.Equal("whoami", only.Value);
|
||||
}
|
||||
|
||||
[Fact]
|
||||
public void ExecuteGuestCommand_NullArgElement_ThrowsArgumentException()
|
||||
{
|
||||
var mockClient = new Mock<IPveHttpClient>();
|
||||
var service = new VmService(mockClient.Object);
|
||||
|
||||
var ex = Assert.Throws<ArgumentException>(() =>
|
||||
service.ExecuteGuestCommand(CreateSession(), TestNode, TestVmId,
|
||||
"cmd.exe", new[] { "/c", null!, "echo" }));
|
||||
Assert.Equal("args", ex.ParamName);
|
||||
}
|
||||
}
|
||||
}
|
||||
|
||||
Reference in New Issue
Block a user