chore(Localizer): update localizer check unit test#7544
Conversation
There was a problem hiding this comment.
Sorry @ArgoZhang, your pull request is larger than the review limit of 150000 diff characters
Codecov Report✅ All modified and coverable lines are covered by tests. 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
Flags with carried forward coverage won't be shown. Click here to find out more. ☔ View full report in Codecov by Sentry. 🚀 New features to boost your workflow:
|
There was a problem hiding this comment.
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.
Link issues
fixes #7543
Summary By Copilot
Regression?
Risk
Verification
Packaging changes reviewed?
☑️ Self Check before Merge