test(analyzer): add unit test coverage for NetEvolve.Defaults.Analyzer - #475
#475test(analyzer): add unit test coverage for NetEvolve.Defaults.Analyzer#475samtrion merged 1 commit intomaindailydevops/defaults:mainfrom feat/analyzer-unit-testsdailydevops/defaults:feat/analyzer-unit-testsCopy head branch name to clipboard
Conversation
Add a new NetEvolve.Defaults.Analyzer.Tests.Unit TUnit project covering IsPackableProjectDiagnosticAnalyzer, OldProjectSupportDiagnosticAnalyzer, and Helper/RuleCategory, including edge cases (missing/whitespace values, boundary years, invalid input) beyond the happy path. fix(analyzer): stop OLD0001 from being reported twice OldProjectSupportDiagnosticAnalyzer reported the same OLD0001 descriptor once per configured key, so a project setting both direngineering and direngineeringsettings received two duplicate diagnostics. Report once when either key is configured. Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_013cR8rWVwvVY2bfM4QyKvtw
|
Important Review skippedAuto reviews are limited based on label configuration. 🏷️ Required labels (at least one) (1)
Please check the settings in the CodeRabbit UI or the ⚙️ Run configurationConfiguration used: Organization UI Review profile: CHILL Plan: Pro Run ID: You can disable this status message by setting the Use the checkbox below for a quick retry:
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## main #475 +/- ##
===========================================
+ Coverage 0 100.00% +100.00%
===========================================
Files 0 1 +1
Lines 0 16 +16
Branches 0 1 +1
===========================================
+ Hits 0 16 +16 ☔ View full report in Codecov by Harness. 🚀 New features to boost your workflow:
|
Summary
NetEvolve.Defaults.Analyzer.Tests.UnitTUnit project coveringIsPackableProjectDiagnosticAnalyzer,OldProjectSupportDiagnosticAnalyzer, andHelper/RuleCategory, including edge cases (missing/whitespace values, boundary years, invalid input) beyond the happy path.OldProjectSupportDiagnosticAnalyzerreportedOLD0001twice when bothdirengineeringanddirengineeringsettingswere configured. It now reports once.Test plan
dotnet build— cleandotnet testintests/NetEvolve.Defaults.Analyzer.Tests.Unit— 55/55 passed🤖 Generated with Claude Code
https://claude.ai/code/session_013cR8rWVwvVY2bfM4QyKvtw