refactor: fix or ignore sonar issues #922

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

Merged

vbreuss merged 2 commits into main from topic/fix-sonar-issues

Jan 21, 2026

Merged

refactor: fix or ignore sonar issues #922

Disable complexity warning in `FileSsystemWatcherMcok`

Annotations

10 warnings

Run sonarcloud analysis

/home/runner/work/Testably.Abstractions/Testably.Abstractions/Source/Testably.Abstractions.Testing/FileSystem/FileSystemWatcherMock.cs(377,4): warning S1066: Merge this if statement with the enclosing one. (https://rules.sonarsource.com/csharp/RSPEC-1066) [/home/runner/work/Testably.Abstractions/Testably.Abstractions/Source/Testably.Abstractions.Testing/Testably.Abstractions.Testing.csproj::TargetFramework=net6.0]

Run sonarcloud analysis

/home/runner/work/Testably.Abstractions/Testably.Abstractions/Source/Testably.Abstractions.Testing/FileSystem/FileSystemWatcherMock.cs(886,54): warning S3011: Make sure that this accessibility bypass is safe here. (https://rules.sonarsource.com/csharp/RSPEC-3011) [/home/runner/work/Testably.Abstractions/Testably.Abstractions/Source/Testably.Abstractions.Testing/Testably.Abstractions.Testing.csproj::TargetFramework=net10.0]

Run sonarcloud analysis

/home/runner/work/Testably.Abstractions/Testably.Abstractions/Source/Testably.Abstractions.Testing/FileSystem/FileSystemWatcherMock.cs(864,51): warning S3011: Make sure that this accessibility bypass is safe here. (https://rules.sonarsource.com/csharp/RSPEC-3011) [/home/runner/work/Testably.Abstractions/Testably.Abstractions/Source/Testably.Abstractions.Testing/Testably.Abstractions.Testing.csproj::TargetFramework=net10.0]

Run sonarcloud analysis

/home/runner/work/Testably.Abstractions/Testably.Abstractions/Source/Testably.Abstractions.Testing/FileSystem/FileSystemWatcherMock.cs(377,4): warning S1066: Merge this if statement with the enclosing one. (https://rules.sonarsource.com/csharp/RSPEC-1066) [/home/runner/work/Testably.Abstractions/Testably.Abstractions/Source/Testably.Abstractions.Testing/Testably.Abstractions.Testing.csproj::TargetFramework=net10.0]

Run sonarcloud analysis

/home/runner/work/Testably.Abstractions/Testably.Abstractions/Source/Testably.Abstractions.Testing/FileSystem/FileSystemWatcherMock.cs(886,54): warning S3011: Make sure that this accessibility bypass is safe here. (https://rules.sonarsource.com/csharp/RSPEC-3011) [/home/runner/work/Testably.Abstractions/Testably.Abstractions/Source/Testably.Abstractions.Testing/Testably.Abstractions.Testing.csproj::TargetFramework=net9.0]

Run sonarcloud analysis

/home/runner/work/Testably.Abstractions/Testably.Abstractions/Source/Testably.Abstractions.Testing/FileSystem/FileSystemWatcherMock.cs(864,51): warning S3011: Make sure that this accessibility bypass is safe here. (https://rules.sonarsource.com/csharp/RSPEC-3011) [/home/runner/work/Testably.Abstractions/Testably.Abstractions/Source/Testably.Abstractions.Testing/Testably.Abstractions.Testing.csproj::TargetFramework=net9.0]

Run sonarcloud analysis

/home/runner/work/Testably.Abstractions/Testably.Abstractions/Source/Testably.Abstractions.Testing/FileSystem/FileSystemWatcherMock.cs(377,4): warning S1066: Merge this if statement with the enclosing one. (https://rules.sonarsource.com/csharp/RSPEC-1066) [/home/runner/work/Testably.Abstractions/Testably.Abstractions/Source/Testably.Abstractions.Testing/Testably.Abstractions.Testing.csproj::TargetFramework=net9.0]

Run sonarcloud analysis

/home/runner/work/Testably.Abstractions/Testably.Abstractions/Source/Testably.Abstractions.Testing/FileSystem/FileSystemWatcherMock.cs(886,54): warning S3011: Make sure that this accessibility bypass is safe here. (https://rules.sonarsource.com/csharp/RSPEC-3011) [/home/runner/work/Testably.Abstractions/Testably.Abstractions/Source/Testably.Abstractions.Testing/Testably.Abstractions.Testing.csproj::TargetFramework=net8.0]

Run sonarcloud analysis

/home/runner/work/Testably.Abstractions/Testably.Abstractions/Source/Testably.Abstractions.Testing/FileSystem/FileSystemWatcherMock.cs(864,51): warning S3011: Make sure that this accessibility bypass is safe here. (https://rules.sonarsource.com/csharp/RSPEC-3011) [/home/runner/work/Testably.Abstractions/Testably.Abstractions/Source/Testably.Abstractions.Testing/Testably.Abstractions.Testing.csproj::TargetFramework=net8.0]

Run sonarcloud analysis

/home/runner/work/Testably.Abstractions/Testably.Abstractions/Source/Testably.Abstractions.Testing/FileSystem/FileSystemWatcherMock.cs(377,4): warning S1066: Merge this if statement with the enclosing one. (https://rules.sonarsource.com/csharp/RSPEC-1066) [/home/runner/work/Testably.Abstractions/Testably.Abstractions/Source/Testably.Abstractions.Testing/Testably.Abstractions.Testing.csproj::TargetFramework=net8.0]