Reset test settings before clearing the data-directory override

[Gabriel Dufresne (GabrielDuf)]

This pull request makes a minor refactor to the test cleanup logic in the Dispose methods of two test classes. The main change is reordering the call to Settings.ResetSettings() to ensure it occurs before resetting other test overrides, which may improve test isolation and reliability.

Test cleanup improvements:

• src/UniGetUI.PackageEngine.Tests/OperationCallArgsWiringTests.cs: Moved the call to Settings.ResetSettings() to the start of the Dispose method for more predictable test state reset.
• src/UniGetUI.PackageEngine.Tests/ShimStandardInputTests.cs: Likewise, reordered Settings.ResetSettings() to be the first statement in the Dispose method.

[Copilot]

🟢 Approval recommended

The focused changes correctly preserve test isolation without introducing unresolved issues.

Pull request overview

Reorders test cleanup so settings are reset while temporary directory overrides remain active, preventing accidental interaction with real user settings.

Changes:

• Moves Settings.ResetSettings() before clearing test overrides in two test fixtures.

File summaries

File	Description
OperationCallArgsWiringTests.cs	Corrects cleanup ordering.
ShimStandardInputTests.cs	Corrects cleanup ordering.

Review details

• Files reviewed: 2/2 changed files
• Comments generated: 0
• Review effort level: Balanced

---

💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.

[randy-but-a-ro]

🤖 Pull request was approved automatically: the AI review is complete and all its review threads are resolved. 🎉

Integration Details

{
	"deliveryId": "4c28def0-ad4d-11f1-8903-9dbbbb8c6581",
	"headSha": "8166918ff4b3243dcdfcfc3961e3099e81af435c",
	"reviewer": "copilot-pull-request-reviewer[bot]"
}