Skip to content

chore(Localizer): update localizer check unit test#7544

Merged
ArgoZhang merged 16 commits intomainfrom
doc-comment
Jan 20, 2026
Merged

chore(Localizer): update localizer check unit test#7544
ArgoZhang merged 16 commits intomainfrom
doc-comment

Conversation

@ArgoZhang
Copy link
Copy Markdown
Member

Link issues

fixes #7543

Summary By Copilot

Regression?

  • Yes
  • No

Risk

  • High
  • Medium
  • Low

Verification

  • Manual (required)
  • Automated

Packaging changes reviewed?

  • Yes
  • No
  • N/A

☑️ Self Check before Merge

⚠️ Please check all items below before review. ⚠️

  • Doc is updated/provided or not needed
  • Demo is updated/provided or not needed
  • Merge the latest code from the main branch

Copilot AI review requested due to automatic review settings January 20, 2026 02:47
@bb-auto bb-auto Bot added the chore This are tasks or bot action label Jan 20, 2026
@bb-auto bb-auto Bot added this to the v10.2.0 milestone Jan 20, 2026
Copy link
Copy Markdown
Contributor

@sourcery-ai sourcery-ai Bot left a comment

Choose a reason for hiding this comment

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

Sorry @ArgoZhang, your pull request is larger than the review limit of 150000 diff characters

@codecov
Copy link
Copy Markdown

codecov Bot commented Jan 20, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 100.00%. Comparing base (55ac5b9) to head (5079d20).
⚠️ Report is 1 commits behind head on main.

Additional details and impacted files
@@            Coverage Diff            @@
##              main     #7544   +/-   ##
=========================================
  Coverage   100.00%   100.00%           
=========================================
  Files          748       748           
  Lines        33000     33000           
  Branches      4588      4588           
=========================================
  Hits         33000     33000           
Flag Coverage Δ
BB 100.00% <ø> (?)

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

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

This pull request updates the localizer check unit test infrastructure. The main purpose is to refactor how component attribute information is retrieved and displayed in documentation samples, moving from manual GetAttributes() methods to automated retrieval through AttributeTable components.

Changes:

  • Refactored test infrastructure to use extension methods for cleaner code organization
  • Removed manual GetAttributes() methods from 100+ sample components, relying on automatic attribute retrieval
  • Enhanced documentation with version tags and improved English descriptions
  • Updated project references to support the new testing approach

Reviewed changes

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

Show a summary per file
File Description
test/UnitTestDocs/MenuTest.cs Refactored to use IsComponentLayout extension method
test/UnitTest.Localization/UnitTest.Localization.csproj Updated project reference and removed unused Foo.cs link
test/UnitTest.Localization/JsonTest.cs New test file for validating localization JSON files
src/BootstrapBlazor.Server/Extensions/TypExtensions.cs New extension method to check component layout types
src/BootstrapBlazor/Components/ErrorLogger/* Updated documentation with version tags and improved English
src/BootstrapBlazor/Components/CountUp/CountUpOption.cs Improved English documentation and added version tags
src/BootstrapBlazor.Server/Services/ComponentAttributeCacheService.cs Enhanced to handle non-IComponent types
src/BootstrapBlazor.Server/Components/Samples/* Removed 100+ manual GetAttributes methods

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread src/BootstrapBlazor.Server/Extensions/TypExtensions.cs
@ArgoZhang ArgoZhang merged commit 9ffc3d4 into main Jan 20, 2026
4 checks passed
@ArgoZhang ArgoZhang deleted the doc-comment branch January 20, 2026 03:15
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

chore This are tasks or bot action

Projects

None yet

Development

Successfully merging this pull request may close these issues.

chore(Localizer): update localizer check unit test

3 participants