DI Refactor: Azure.Mcp.Tools.LoadTesting

[Copilot]

Removes context.GetService<ILoadTestingService>() calls from ExecuteAsync across all LoadTesting commands, replacing them with constructor-injected dependencies per the pattern established in #158.

Commands updated

• TestGetCommand
• TestCreateCommand
• TestResourceCreateCommand
• TestResourceListCommand
• TestRunCreateOrUpdateCommand
• TestRunGetCommand

Pattern applied

// Before
public sealed class TestGetCommand(ILogger<TestGetCommand> logger) : BaseLoadTestingCommand<TestGetOptions>
{
    public override async Task<CommandResponse> ExecuteAsync(...)
    {
        var service = context.GetService<ILoadTestingService>(); // resolved at call time
        var results = await service.GetTestAsync(...);
    }
}

// After
public sealed class TestGetCommand(ILogger<TestGetCommand> logger, ILoadTestingService loadTestingService) : BaseLoadTestingCommand<TestGetOptions>
{
    private readonly ILoadTestingService _loadTestingService = loadTestingService;

    public override async Task<CommandResponse> ExecuteAsync(...)
    {
        var results = await _loadTestingService.GetTestAsync(...);
    }
}

Tests updated

All unit tests now pass the mock ILoadTestingService directly to the command constructor instead of registering it in the ServiceCollection, removing the indirect DI container dependency from test setup.

Original prompt

---

This section details on the original issue you should resolve

<issue_title>DI Refactor: Azure.Mcp.Tools.LoadTesting</issue_title>
<issue_description>## Summary

Refactor Azure.Mcp.Tools.LoadTesting to use constructor dependency injection instead of resolving services via context.GetService<T>() in ExecuteAsync.

This is a sub-issue of #158.

Instructions

Follow the algorithm described in tools/di-refactor-plan.md with {ToolArea} = LoadTesting.

An example PR is: #1815
</issue_description>

Comments on the Issue (you are @copilot in this section)

• Fixes DI Refactor: Azure.Mcp.Tools.LoadTesting #1859

---

💬 Send tasks to Copilot coding agent from Slack and Teams to turn conversations into code. Copilot posts an update in your thread when it's finished.

[Copilot]

Pull request overview

This PR refactors the Azure.Mcp.Tools.LoadTesting command implementations to use constructor-injected ILoadTestingService rather than resolving it from CommandContext during ExecuteAsync, aligning LoadTesting with the DI pattern referenced in #158.

Changes:

• Updated LoadTesting commands to accept ILoadTestingService via constructor injection and to call the injected instance inside ExecuteAsync.
• Updated unit tests to construct commands with a mocked ILoadTestingService directly instead of registering it in a ServiceCollection.

Reviewed changes

Copilot reviewed 12 out of 12 changed files in this pull request and generated no comments.

Show a summary per file

File	Description
tools/Azure.Mcp.Tools.LoadTesting/src/Commands/LoadTestRun/TestRunGetCommand.cs	Switches from context.GetService<ILoadTestingService>() to constructor-injected service.
tools/Azure.Mcp.Tools.LoadTesting/src/Commands/LoadTestRun/TestRunCreateOrUpdateCommand.cs	Same DI refactor for create/update test run flow.
tools/Azure.Mcp.Tools.LoadTesting/src/Commands/LoadTestResource/TestResourceListCommand.cs	Same DI refactor for listing load test resources.
tools/Azure.Mcp.Tools.LoadTesting/src/Commands/LoadTestResource/TestResourceCreateCommand.cs	Same DI refactor for creating load test resources.
tools/Azure.Mcp.Tools.LoadTesting/src/Commands/LoadTest/TestGetCommand.cs	Same DI refactor for getting a test definition.
tools/Azure.Mcp.Tools.LoadTesting/src/Commands/LoadTest/TestCreateCommand.cs	Same DI refactor for creating a test definition.
tools/Azure.Mcp.Tools.LoadTesting/tests/Azure.Mcp.Tools.LoadTesting.UnitTests/TestRunGetCommandTests.cs	Updates tests to pass mocked service directly to command constructor.
tools/Azure.Mcp.Tools.LoadTesting/tests/Azure.Mcp.Tools.LoadTesting.UnitTests/TestRunCreateOrUpdateCommandTests.cs	Updates tests to pass mocked service directly to command constructor.
tools/Azure.Mcp.Tools.LoadTesting/tests/Azure.Mcp.Tools.LoadTesting.UnitTests/TestResourcesListCommandTests.cs	Updates tests to pass mocked service directly to command constructor.
tools/Azure.Mcp.Tools.LoadTesting/tests/Azure.Mcp.Tools.LoadTesting.UnitTests/TestResourceCreateTests.cs	Updates tests to pass mocked service directly to command constructor.
tools/Azure.Mcp.Tools.LoadTesting/tests/Azure.Mcp.Tools.LoadTesting.UnitTests/TestGetCommandTests.cs	Updates tests to pass mocked service directly to command constructor.
tools/Azure.Mcp.Tools.LoadTesting/tests/Azure.Mcp.Tools.LoadTesting.UnitTests/TestCreateCommandTests.cs	Updates tests to pass mocked service directly to command constructor.

---

You can also share your feedback on Copilot code review. Take the survey.