Closed rjmurillo closed 4 months ago
Coverage variation | Diff coverage |
---|---|
:white_check_mark: +0.03% (target: -1.00%) | :x: 93.67% (target: 95.00%) |
:rocket: Don’t miss a bit, follow what’s new on Codacy.
Codacy stopped sending the deprecated coverage status on June 5th, 2024. Learn more
Code Climate has analyzed commit 70cd7a80 and detected 37 issues on this pull request.
Here's the issue category breakdown:
Category | Count |
---|---|
Complexity | 15 |
Duplication | 2 |
Style | 20 |
View more on Code Climate.
The majority of this change is mechanical (moving code from one location to another, replacing magic strings with constants, etc.). There is a functional update that completely changes the
ConstructorArgumentsShouldMatchAnalyzer
:Mock.Of<T>
andMockRepository
features ofMoq
Baseline is an empty analyzer, control is the old interface constructor parameter analyzer, treatment is the new combined analyzer.
Changes
Misc:
Common
folder and namespaceDocumentation:
Analyzers:
Diagnostic
ConstructorArgumentsShouldMatchAnalyzer
now supportsMock.Of<T>
syntax andMockRepository
patternsBenchmarks:
ConstructorArgumentsShouldMatchAnalyzer
Test:
ConstructorArgumentsShouldMatchAnalyzer
test collateral (it's the highest volume) into partial filesNote: while code coverage is improved, it's still below 95%. This is due to some guard cases that need coverage. For example, #141
Resolves #122 For Moq1001 and Moq1002, #85