Add unit tests for ToolStripAdornerWindowService

[Nora-Zhou01]

Related #10773

Proposed changes

• Add unit test ToolStripAdornerWindowServiceTests.cs for public properties and method of the ToolStripAdornerWindowService
• Enable nullability in ToolStripAdornerWindowServiceTests.cs

Microsoft Reviewers: Open in CodeFlow

[codecov]

Codecov Report

Attention: Patch coverage is 91.53846% with 11 lines in your changes missing coverage. Please review.

Project coverage is 76.06432%. Comparing base (e91e3bc) to head (02c7043).
Report is 325 commits behind head on main.

Additional details and impacted files

@@                 Coverage Diff                 @@
##                main      #13029         +/-   ##
===================================================
+ Coverage   75.75455%   76.06432%   +0.30977%     
===================================================
  Files           3159        3254         +95     
  Lines         635942      643320       +7378     
  Branches       46987       47395        +408     
===================================================
+ Hits          481755      489337       +7582     
+ Misses        150726      150425        -301     
- Partials        3461        3558         +97     

Flag	Coverage Δ
Debug	76.06432% <91.53846%> (+0.30977%)	⬆️
integration	18.01386% <ø> (-0.16092%)	⬇️
production	50.02463% <ø> (+0.70708%)	⬆️
test	96.93532% <91.53846%> (-0.10975%)	⬇️
unit	47.47903% <ø> (+0.94733%)	⬆️

Flags with carried forward coverage won't be shown. Click here to find out more.

[Copilot]

PR Overview

This pull request adds unit tests for the ToolStripAdornerWindowService to validate its public properties and methods along with enabling nullability in the tests.

• Introduces multiple tests covering graphics, coordinate translation, and resource disposal.
• Provides STA-thread execution for tests that interact with UI components.

Reviewed Changes

File	Description
src/System.Windows.Forms.Design/tests/UnitTests/System/Windows/Forms/Design/ToolStripAdornerWindowServiceTests.cs	Contains new unit tests for verifying ToolStripAdornerWindowService behavior and proper handling of designer overlays and graphics.

Copilot reviewed 1 out of 1 changed files in this pull request and generated 1 comment.

[Copilot]

The test creates a Control in a using statement, causing it to be disposed immediately after the tool strip service is initialized. Consider instantiating the control without a using block to ensure it remains available for the lifetime of the service during testing.

Suggested change

	public ToolStripAdornerWindowServiceTests() => (_behaviorServiceWrapper, _service) = Initialize();
	private (BehaviorServiceWrapper, ToolStripAdornerWindowService) Initialize()
	{
	BehaviorServiceWrapper behaviorServiceWrapper = new();
	using Control control = new();
	var service = new ToolStripAdornerWindowService(behaviorServiceWrapper.ServiceProviderMock.Object, control);
	return (behaviorServiceWrapper, service);
	private readonly Control _control;
	public ToolStripAdornerWindowServiceTests() => (_behaviorServiceWrapper, _service, _control) = Initialize();
	private (BehaviorServiceWrapper, ToolStripAdornerWindowService, Control) Initialize()
	{
	BehaviorServiceWrapper behaviorServiceWrapper = new();
	Control control = new();
	var service = new ToolStripAdornerWindowService(behaviorServiceWrapper.ServiceProviderMock.Object, control);
	return (behaviorServiceWrapper, service, control);