refactor: fix or ignore sonar issues (#922) #764
build.yml
on: push
Deployment protection rules
Reviewers, timers, and other rules protecting deployments in this run
| Event | Environments |
|---|---|
|
vbreuss
approved
|
production |
Annotations
13 warnings
|
Static code analysis
/home/runner/work/Testably.Abstractions/Testably.Abstractions/Source/Testably.Abstractions.Testing/Statistics/ParameterDescription.cs(67,6): warning S3060: Offload the code that's conditional on this type test to the appropriate subclass and remove the condition. (https://rules.sonarsource.com/csharp/RSPEC-3060) [/home/runner/work/Testably.Abstractions/Testably.Abstractions/Source/Testably.Abstractions.Testing/Testably.Abstractions.Testing.csproj::TargetFramework=net9.0]
|
|
Static code analysis
/home/runner/work/Testably.Abstractions/Testably.Abstractions/Source/Testably.Abstractions.Testing/Storage/InMemoryStorage.cs(276,71): warning S3267: Loops should be simplified using the "Where" LINQ method (https://rules.sonarsource.com/csharp/RSPEC-3267) [/home/runner/work/Testably.Abstractions/Testably.Abstractions/Source/Testably.Abstractions.Testing/Testably.Abstractions.Testing.csproj::TargetFramework=net9.0]
|
|
Static code analysis
/home/runner/work/Testably.Abstractions/Testably.Abstractions/Source/Testably.Abstractions.Testing/Storage/InMemoryStorage.cs(216,40): warning S3776: Refactor this method to reduce its Cognitive Complexity from 26 to the 15 allowed. (https://rules.sonarsource.com/csharp/RSPEC-3776) [/home/runner/work/Testably.Abstractions/Testably.Abstractions/Source/Testably.Abstractions.Testing/Testably.Abstractions.Testing.csproj::TargetFramework=net9.0]
|
|
Static code 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]
|
|
Static code analysis
/home/runner/work/Testably.Abstractions/Testably.Abstractions/Source/Testably.Abstractions.Testing/Storage/InMemoryStorage.cs(55,27): warning S3776: Refactor this method to reduce its Cognitive Complexity from 16 to the 15 allowed. (https://rules.sonarsource.com/csharp/RSPEC-3776) [/home/runner/work/Testably.Abstractions/Testably.Abstractions/Source/Testably.Abstractions.Testing/Testably.Abstractions.Testing.csproj::TargetFramework=net9.0]
|
|
Static code analysis
/home/runner/work/Testably.Abstractions/Testably.Abstractions/Source/Testably.Abstractions.Testing/FileSystemInitializerExtensions.cs(41,21): warning S3776: Refactor this method to reduce its Cognitive Complexity from 16 to the 15 allowed. (https://rules.sonarsource.com/csharp/RSPEC-3776) [/home/runner/work/Testably.Abstractions/Testably.Abstractions/Source/Testably.Abstractions.Testing/Testably.Abstractions.Testing.csproj::TargetFramework=net9.0]
|
|
Static code analysis
/home/runner/work/Testably.Abstractions/Testably.Abstractions/Source/Testably.Abstractions.Testing/Helpers/PathHelper.cs(87,23): warning S3267: Loops should be simplified using the "Where" LINQ method (https://rules.sonarsource.com/csharp/RSPEC-3267) [/home/runner/work/Testably.Abstractions/Testably.Abstractions/Source/Testably.Abstractions.Testing/Testably.Abstractions.Testing.csproj::TargetFramework=net9.0]
|
|
Static code analysis
/home/runner/work/Testably.Abstractions/Testably.Abstractions/Source/Testably.Abstractions.Testing/Helpers/RandomFactory.cs(27,28): warning S2245: Make sure that using this pseudorandom number generator is safe here. (https://rules.sonarsource.com/csharp/RSPEC-2245) [/home/runner/work/Testably.Abstractions/Testably.Abstractions/Source/Testably.Abstractions.Testing/Testably.Abstractions.Testing.csproj::TargetFramework=net9.0]
|
|
Static code analysis
/home/runner/work/Testably.Abstractions/Testably.Abstractions/Source/Testably.Abstractions.Testing/RandomSystem/RandomMock.cs(55,32): warning S2245: Make sure that using this pseudorandom number generator is safe here. (https://rules.sonarsource.com/csharp/RSPEC-2245) [/home/runner/work/Testably.Abstractions/Testably.Abstractions/Source/Testably.Abstractions.Testing/Testably.Abstractions.Testing.csproj::TargetFramework=net9.0]
|
|
Static code analysis
/home/runner/work/Testably.Abstractions/Testably.Abstractions/Source/Testably.Abstractions.Testing/Statistics/IStatistics.cs(23,30): warning S2326: 'TType' is not used in the interface. (https://rules.sonarsource.com/csharp/RSPEC-2326) [/home/runner/work/Testably.Abstractions/Testably.Abstractions/Source/Testably.Abstractions.Testing/Testably.Abstractions.Testing.csproj::TargetFramework=net9.0]
|
|
Unit tests (windows-latest)
D:\a\Testably.Abstractions\Testably.Abstractions\Tests\Testably.Abstractions.Testing.Tests\obj\Release\net48\aweXpect.Frameworks\aweXpect.Frameworks.FrameworkGenerator\Xunit3.g.cs(23,35): warning CS0436: The type 'DoesNotReturnAttribute' in 'C:\Users\runneradmin\.nuget\packages\nullable\1.3.1\contentFiles\cs\net40\Nullable\DoesNotReturnAttribute.cs' conflicts with the imported type 'DoesNotReturnAttribute' in 'Testably.Abstractions.Testing, Version=1.0.0.0, Culture=neutral, PublicKeyToken=f24346c8579fcb48'. Using the type defined in 'C:\Users\runneradmin\.nuget\packages\nullable\1.3.1\contentFiles\cs\net40\Nullable\DoesNotReturnAttribute.cs'. [D:\a\Testably.Abstractions\Testably.Abstractions\Tests\Testably.Abstractions.Testing.Tests\Testably.Abstractions.Testing.Tests.csproj::TargetFramework=net48]
|
|
Unit tests (windows-latest)
D:\a\Testably.Abstractions\Testably.Abstractions\Tests\Testably.Abstractions.Testing.Tests\obj\Release\net48\aweXpect.Frameworks\aweXpect.Frameworks.FrameworkGenerator\Xunit3.g.cs(18,35): warning CS0436: The type 'DoesNotReturnAttribute' in 'C:\Users\runneradmin\.nuget\packages\nullable\1.3.1\contentFiles\cs\net40\Nullable\DoesNotReturnAttribute.cs' conflicts with the imported type 'DoesNotReturnAttribute' in 'Testably.Abstractions.Testing, Version=1.0.0.0, Culture=neutral, PublicKeyToken=f24346c8579fcb48'. Using the type defined in 'C:\Users\runneradmin\.nuget\packages\nullable\1.3.1\contentFiles\cs\net40\Nullable\DoesNotReturnAttribute.cs'. [D:\a\Testably.Abstractions\Testably.Abstractions\Tests\Testably.Abstractions.Testing.Tests\Testably.Abstractions.Testing.Tests.csproj::TargetFramework=net48]
|
|
Unit tests (windows-latest)
D:\a\Testably.Abstractions\Testably.Abstractions\Tests\Testably.Abstractions.Testing.Tests\obj\Release\net48\aweXpect.Frameworks\aweXpect.Frameworks.FrameworkGenerator\Xunit3.g.cs(13,35): warning CS0436: The type 'DoesNotReturnAttribute' in 'C:\Users\runneradmin\.nuget\packages\nullable\1.3.1\contentFiles\cs\net40\Nullable\DoesNotReturnAttribute.cs' conflicts with the imported type 'DoesNotReturnAttribute' in 'Testably.Abstractions.Testing, Version=1.0.0.0, Culture=neutral, PublicKeyToken=f24346c8579fcb48'. Using the type defined in 'C:\Users\runneradmin\.nuget\packages\nullable\1.3.1\contentFiles\cs\net40\Nullable\DoesNotReturnAttribute.cs'. [D:\a\Testably.Abstractions\Testably.Abstractions\Tests\Testably.Abstractions.Testing.Tests\Testably.Abstractions.Testing.Tests.csproj::TargetFramework=net48]
|
Artifacts
Produced during runtime
| Name | Size | Digest | |
|---|---|---|---|
|
API-tests
|
10.2 KB |
sha256:3c23b9b348356192179dd8b141d705bb384c8f4fc5b0466a914c48442661128c
|
|
|
Linux-artifacts
|
9.34 MB |
sha256:36db34ce8056d3fb7765242ce8e3b1cceba0e5cd36a66d9c8389ad2fc7264afb
|
|
|
Packages
|
1.82 MB |
sha256:1e94cb0085b71fc0472862630b3acceafe4a5c3f8d222f2829cfa3cc8a290479
|
|
|
Windows-artifacts
|
10.4 MB |
sha256:3353130885cb1aba2fc833e32ce09898eac9dbc84a216444d2d9ecbaa0eaf756
|
|
|
macOS-artifacts
|
9.38 MB |
sha256:f7f99dcd5a8b19b40d1cf13a53af78b1c66037fd97dbff1cb27724da118f1489
|
|