fix: suppress Moq1203 for setup wrapped in extension methods #1086

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Open

rjmurillo wants to merge 14 commits into main from fix/1067-wrapped-setup-false-positive

Open

fix: suppress Moq1203 for setup wrapped in extension methods #1086

Merge remote-tracking branch 'origin/main' into fix/1067-wrapped-setu…

Codacy Production / Codacy Static Code Analysis required action Mar 31, 2026 in 0s

1 new issue (0 max.) of at least medium severity.

Here is an overview of what got changed by this pull request:

Issues
======
- Added 3

See the complete overview on Codacy

Annotations

Check notice on line 347 in src/Common/ISymbolExtensions.Moq.cs

codacy-production / Codacy Static Code Analysis

src/Common/ISymbolExtensions.Moq.cs#L347

Reduce the number of conditional operators (6) used in the expression (maximum allowed 3).

Check warning on line 664 in tests/Moq.Analyzers.Test/MethodSetupShouldSpecifyReturnValueAnalyzerTests.cs

codacy-production / Codacy Static Code Analysis

tests/Moq.Analyzers.Test/MethodSetupShouldSpecifyReturnValueAnalyzerTests.cs#L664

Method MethodSetupShouldSpecifyReturnValueAnalyzerTests::BuildWrappedSetupSource has 102 lines of code (limit is 50)

Check notice on line 798 in tests/Moq.Analyzers.Test/MethodSetupShouldSpecifyReturnValueAnalyzerTests.cs

codacy-production / Codacy Static Code Analysis

tests/Moq.Analyzers.Test/MethodSetupShouldSpecifyReturnValueAnalyzerTests.cs#L798

Remove the 'Async' suffix to the name of this method.

View more details on Codacy Production