rjmurillo / moq.analyzers

Set of analyzers for Moq mocking library
BSD 3-Clause "New" or "Revised" License
22 stars 2 forks source link

Update ConstructorArgumentsShouldMatchAnalyzer test cases to make sure all positive and negative cases are covered #124

Closed rjmurillo closed 1 week ago

rjmurillo commented 1 week ago

This is comprehensive and did not change the code coverage numbers.

Resolves #123

There are still blocks that are not covered:

https://github.com/rjmurillo/moq.analyzers/blob/e3e5dec23cf7660fc531006755cff62748d3c22b/src/Moq.Analyzers/ConstructorArgumentsShouldMatchAnalyzer.cs#L208-L213

The return null cases

https://github.com/rjmurillo/moq.analyzers/blob/e3e5dec23cf7660fc531006755cff62748d3c22b/src/Moq.Analyzers/ConstructorArgumentsShouldMatchAnalyzer.cs#L166-L175

codacy-production[bot] commented 1 week ago

Coverage summary from Codacy

See diff coverage on Codacy

Coverage variation Diff coverage
:white_check_mark: +0.00% (target: -1.00%) :white_check_mark: (target: 95.00%)
Coverage variation details | | Coverable lines | Covered lines | Coverage | | ------------- | ------------- | ------------- | ------------- | | Common ancestor commit (e3e5dec23cf7660fc531006755cff62748d3c22b) | 423 | 392 | 92.67% | | | Head commit (dbf13f9335ce46ec263ff1f65f892061e335a705) | 423 (+0) | 392 (+0) | 92.67% (**+0.00%**) | **Coverage variation** is the difference between the coverage for the head and common ancestor commits of the pull request branch: ` - `
Diff coverage details | | Coverable lines | Covered lines | Diff coverage | | ------------- | ------------- | ------------- | ------------- | | Pull request (#124) | 0 | 0 | **∅ (not applicable)** | **Diff coverage** is the percentage of lines that are covered by tests out of the coverable lines that the pull request added or modified: `/ * 100%`

See your quality gate settings    Change summary preferences


: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