Skip to content

Latest commit

 

History

History
80 lines (56 loc) · 4.64 KB

contributing-analyzer.md

File metadata and controls

80 lines (56 loc) · 4.64 KB

Building, Testing and Debugging the Analyzer

All C# and VB .NET analyzers present in SonarLint for Visual Studio, in the SonarQube SonarC# Plugin or in the SonarQube SonarVB plugin are being developed here. These analyzers rely on Roslyn 1.3.2 API.

Working with the code

  1. Clone this repository
  2. Download sub-modules git submodule update --init --recursive
  3. Run .\scripts\build\dev-build.ps1 -restore -build -test -metadata

Developing with Visual Studio 2017

  1. Visual Studio 2017 Version 15.7
    • Ensure C#, VB, MSBuild, .NET Core and Visual Studio Extensibility are included in the selected work loads
    • Ensure Visual Studio is on Version "15.7" or greater
  2. .NET Core SDK 2.1.402 (the current previews are: Windows x64 installer, Windows x86 installer)
  3. Open SonarAnalyzer.sln in the sonaranalyzer-dotnet subfolder

The following environment variables must be set:

  • MSBUILD_PATH - path to MSBuild.exe from the Visual Studio installation folder
  • ORCHESTRATOR_CONFIG_URL - path to orchestrator.properties (for integration tests)
  • rule_api_path - path to folder containing the rule api jar
  • Path - the Path must contain the path to the dotnet core installation folder

Running Tests

Unit Tests

You can run the Unit Tests via the Test Explorer of Visual Studio or using .\scripts\build\dev-build.ps1 test

Integration Tests

Running the tests

To run the ITs you will need to follow this pattern:

  1. Make sure the project is built: Integration tests don't build the analyzer, but use the results of the latest build (debug or release)
  2. Open the Developer Command Prompt for VS2017 from the start menu
  3. Go to PATH_TO_CLONED_REPOSITORY/sonaranalyzer-dotnet/its
  4. Run powershell
  5. Run .\regression-test.ps1

Note: you can run a single rule using the -ruleId parameter (e.g. .\regression-test.ps1 -ruleId S1234)

If the script ends with SUCCESS: No differences were found! (or exit code 0), this means the changes you have made haven't impacted any rule.

If the script ends with ERROR: There are differences between the actual and the expected issues. (or exit code 1), the changes you have made have impacted one or many issues raised by the rules.

Manual differences review

You can visualize the differences using:

  1. cd actual
  2. git diff --cached

Semi-automated differences review

If you run the project ReviewDiffs in debug mode from Visual Studio, it will print in the output windows all places where a difference has been found. From there you can easily navigate between differences (double-click, F4...).

Disclaimer: This program is still very new and the paint is still very wet. It may not work in all situations, but it's easy to update it ??.

PS: This program outputs message in a way that is compatible with the VsColorOutput extension, so that errors are colored in red, and differences are colored like warnings.

Updating the references

Please review all added/removed/updated issue to confirm they are wanted. When this is done run .\update-expected.ps1 to update the list of expected issues. Note: Integration tests build the code to analyze. If you have an antivirus program on your computer, this may fail with some error messages about an executable file that cannot be open for writing. If this happens, look at the antivirus logs for confirmation, and contact IT team to add an rule to your antivirus program...

Manual Tests

From Visual Studio, make sure SonarAnalyzer.Vsix.csproj is selected as startup project. And then do the following:

  1. Make sure SonarLint for Visual Studio is uninstalled
  2. Hit F5 to launch the experimental instance of Visual Studio
  3. Open one of the following solutions from the experimental instance:
  4. Turn on your new rule in Validation Ruleset, review the results, improve, and setup the regression test once you are satisfied.
    • Note: the solutions have been pre-configured to use this ruleset on all their projects.

Contributing

Please see Contributing Code for details on contributing changes back to the code.