diff --git a/core/Azure.Mcp.Core/src/Areas/Subscription/Commands/SubscriptionListCommand.cs b/core/Azure.Mcp.Core/src/Areas/Subscription/Commands/SubscriptionListCommand.cs index 3b5c6daba..aff9706c4 100644 --- a/core/Azure.Mcp.Core/src/Areas/Subscription/Commands/SubscriptionListCommand.cs +++ b/core/Azure.Mcp.Core/src/Areas/Subscription/Commands/SubscriptionListCommand.cs @@ -49,11 +49,9 @@ public override async Task ExecuteAsync(CommandContext context, var subscriptionService = context.GetService(); var subscriptions = await subscriptionService.GetSubscriptions(options.Tenant, options.RetryPolicy); - context.Response.Results = subscriptions?.Count > 0 - ? ResponseResult.Create( + context.Response.Results = ResponseResult.Create( new SubscriptionListCommandResult(subscriptions), - SubscriptionJsonContext.Default.SubscriptionListCommandResult) - : null; + SubscriptionJsonContext.Default.SubscriptionListCommandResult); } catch (Exception ex) { diff --git a/core/Azure.Mcp.Core/tests/Azure.Mcp.Core.UnitTests/Areas/Subscription/SubscriptionListCommandTests.cs b/core/Azure.Mcp.Core/tests/Azure.Mcp.Core.UnitTests/Areas/Subscription/SubscriptionListCommandTests.cs index 92aed978e..452558daa 100644 --- a/core/Azure.Mcp.Core/tests/Azure.Mcp.Core.UnitTests/Areas/Subscription/SubscriptionListCommandTests.cs +++ b/core/Azure.Mcp.Core/tests/Azure.Mcp.Core.UnitTests/Areas/Subscription/SubscriptionListCommandTests.cs @@ -105,7 +105,7 @@ await _subscriptionService.Received(1).GetSubscriptions( } [Fact] - public async Task ExecuteAsync_EmptySubscriptionList_ReturnsNullResults() + public async Task ExecuteAsync_EmptySubscriptionList_ReturnsNotNullResults() { // Arrange _subscriptionService @@ -120,7 +120,7 @@ public async Task ExecuteAsync_EmptySubscriptionList_ReturnsNullResults() // Assert Assert.NotNull(result); Assert.Equal(200, result.Status); - Assert.Null(result.Results); + Assert.NotNull(result.Results); } [Fact]