Skip to content

Add unit tests for PropertyGrid.RefreshTabs(PropertyTabScope) in PropertyGridTests.cs.#14627

Closed
ricardobossan wants to merge 1 commit into
dotnet:mainfrom
ricardobossan:Add_Coverrage_For_PropertyGrid_RefreshTabs
Closed

Add unit tests for PropertyGrid.RefreshTabs(PropertyTabScope) in PropertyGridTests.cs.#14627
ricardobossan wants to merge 1 commit into
dotnet:mainfrom
ricardobossan:Add_Coverrage_For_PropertyGrid_RefreshTabs

Conversation

@ricardobossan

@ricardobossan ricardobossan commented Jun 10, 2026

Copy link
Copy Markdown
Member

Related #12055

Proposed changes

  • Add unit tests for PropertyGrid.RefreshTabs(PropertyTabScope) in PropertyGridTests.cs.

Customer Impact

  • None

Regression?

  • No

Risk

  • Minimal

Test methodology

  • Unit tests

Test environment(s)

  • 11.0.100-preview.5.26227.104
Microsoft Reviewers: Open in CodeFlow

- Add unit tests for `PropertyGrid.RefreshTabs(PropertyTabScope)` in `PropertyGridTests.cs`.

- None

- No

- Minimal

- Unit tests

- 11.0.100-preview.5.26227.104
@ricardobossan ricardobossan requested a review from a team as a code owner June 10, 2026 00:27
@ricardobossan ricardobossan requested review from LeafShi1 and removed request for a team June 10, 2026 00:27
@ricardobossan ricardobossan self-assigned this Jun 10, 2026

@SimonZhao888 SimonZhao888 left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good to me!

out _);

using TestComponent component = new();
ComponentCollection components = new(new IComponent[] { component });

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Collection initialization can be simplified

@ricardobossan

Copy link
Copy Markdown
Member Author

Closing due to #12055 (comment)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants