Skip to content
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.

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

S1200 provides different results for Automapper IT on MSBuild16 #2854

Open
pavel-mikula-sonarsource opened this issue Dec 2, 2019 · 0 comments
Labels
Area: C# C# rules related issues. Type: UX Improve any kind of user experience

Comments

@pavel-mikula-sonarsource
Copy link
Contributor

Description

Rule S1200 contains expected json file for ITs of AutoMapper project:
\sonaranalyzer-dotnet\its\actual\Automapper\AutoMapper-025B9F93-7953-454B-AF29-E6A55B2AA58F-S1200.json

This file declares expected error message:

Split this class into smaller and more specialized ones to reduce its dependencies on other classes from 33 to the maximum authorized 30 or less.

Using VS 2019 with MSBuild 16 produces different error message:

Split this class into smaller and more specialized ones to reduce its dependencies on other classes from 34 to the maximum authorized 30 or less.

Repro steps

Run ITs in VS2019 with MSBuild 16:

.\regression-test.ps1 -ruleId S1200 -project Automapper

and compare actual ITs results with expected.

Expected behavior

Both versions should provide the same result.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Area: C# C# rules related issues. Type: UX Improve any kind of user experience
Projects
None yet
Development

No branches or pull requests

1 participant