Skip to content

Add tests

Add tests #353

Triggered via push December 11, 2023 12:49
Status Success
Total duration 1m 51s
Artifacts 2

ci.yml

on: push
Matrix: Build & Test
Publish Tests Results
8s
Publish Tests Results
Fit to window
Zoom out
Zoom in

Annotations

43 warnings
Build & Test (6.0)
The following actions uses node12 which is deprecated and will be forced to run on node16: actions/checkout@v2. For more info: https://github.blog/changelog/2023-06-13-github-actions-all-actions-will-run-on-node16-instead-of-node12-by-default/
Build & Test (6.0): src/LeanCode.Contracts/Binary.cs#L30
Provide a method named 'ToByteArray' or 'FromNullable' as an alternate for operator op_Implicit (https://learn.microsoft.com/dotnet/fundamentals/code-analysis/quality-rules/ca2225)
Build & Test (6.0): src/LeanCode.Contracts/Binary.cs#L8
Properties should not return arrays (https://learn.microsoft.com/dotnet/fundamentals/code-analysis/quality-rules/ca1819)
Build & Test (6.0): src/LeanCode.Contracts/Security/AuthorizeWhenHasAnyOfAttribute.cs#L10
Add a public read-only property accessor for positional argument permissions of Attribute AuthorizeWhenHasAnyOfAttribute (https://learn.microsoft.com/dotnet/fundamentals/code-analysis/quality-rules/ca1019)
Build & Test (6.0): src/LeanCode.Contracts/CommandResult.cs#L20
In externally visible method 'CommandResult CommandResult.NotValid(ValidationResult validationResult)', validate parameter 'validationResult' is non-null before using it. If appropriate, throw an 'ArgumentNullException' when the argument is 'null'. (https://learn.microsoft.com/dotnet/fundamentals/code-analysis/quality-rules/ca1062)
Build & Test (6.0): src/LeanCode.Contracts/Security/AuthorizeWhenAttribute.cs#L59
Change 'List<AuthorizeWhenAttribute.AuthorizerDefinition>' in 'AuthorizeWhenAttribute<T>.GetCustomAuthorizers()' to use 'Collection<T>', 'ReadOnlyCollection<T>' or 'KeyedCollection<K,V>' (https://learn.microsoft.com/dotnet/fundamentals/code-analysis/quality-rules/ca1002)
Build & Test (6.0): src/LeanCode.Contracts/Security/AuthorizeWhenAttribute.cs#L22
Change 'List<AuthorizeWhenAttribute.AuthorizerDefinition>' in 'AuthorizeWhenAttribute.GetCustomAuthorizers<T>()' to use 'Collection<T>', 'ReadOnlyCollection<T>' or 'KeyedCollection<K,V>' (https://learn.microsoft.com/dotnet/fundamentals/code-analysis/quality-rules/ca1002)
Build & Test (6.0): src/LeanCode.Contracts/NotificationEnvelope.cs#L16
In externally visible method 'NotificationEnvelope.NotificationEnvelope(Guid id, ITopic topic, object notification)', validate parameter 'topic' is non-null before using it. If appropriate, throw an 'ArgumentNullException' when the argument is 'null'. (https://learn.microsoft.com/dotnet/fundamentals/code-analysis/quality-rules/ca1062)
Build & Test (6.0): src/LeanCode.Contracts/NotificationEnvelope.cs#L17
In externally visible method 'NotificationEnvelope.NotificationEnvelope(Guid id, ITopic topic, object notification)', validate parameter 'notification' is non-null before using it. If appropriate, throw an 'ArgumentNullException' when the argument is 'null'. (https://learn.microsoft.com/dotnet/fundamentals/code-analysis/quality-rules/ca1062)
Build & Test (6.0): src/LeanCode.Contracts/Security/AuthorizeWhenAttribute.cs#L25
Change 'List<AuthorizeWhenAttribute.AuthorizerDefinition>' in 'AuthorizeWhenAttribute.GetCustomAuthorizers(Type)' to use 'Collection<T>', 'ReadOnlyCollection<T>' or 'KeyedCollection<K,V>' (https://learn.microsoft.com/dotnet/fundamentals/code-analysis/quality-rules/ca1002)
Build & Test (6.0): src/LeanCode.Contracts/NotificationTagGenerator.cs#L32
'string.IndexOf(char)' has a method overload that takes a 'StringComparison' parameter. Replace this call in 'LeanCode.Contracts.NotificationTagGenerator.GetSimpleName(System.Type)' with a call to 'string.IndexOf(char, System.StringComparison)' for clarity of intent. (https://learn.microsoft.com/dotnet/fundamentals/code-analysis/quality-rules/ca1307)
Build & Test (6.0): src/LeanCode.Contracts/Binary.cs#L30
Provide a method named 'ToByteArray' or 'FromNullable' as an alternate for operator op_Implicit (https://learn.microsoft.com/dotnet/fundamentals/code-analysis/quality-rules/ca2225)
Build & Test (6.0): src/LeanCode.Contracts/Security/AuthorizeWhenHasAnyOfAttribute.cs#L10
Add a public read-only property accessor for positional argument permissions of Attribute AuthorizeWhenHasAnyOfAttribute (https://learn.microsoft.com/dotnet/fundamentals/code-analysis/quality-rules/ca1019)
Build & Test (6.0): src/LeanCode.Contracts/Binary.cs#L8
Properties should not return arrays (https://learn.microsoft.com/dotnet/fundamentals/code-analysis/quality-rules/ca1819)
Build & Test (6.0): src/LeanCode.Contracts/CommandResult.cs#L20
In externally visible method 'CommandResult CommandResult.NotValid(ValidationResult validationResult)', validate parameter 'validationResult' is non-null before using it. If appropriate, throw an 'ArgumentNullException' when the argument is 'null'. (https://learn.microsoft.com/dotnet/fundamentals/code-analysis/quality-rules/ca1062)
Build & Test (6.0): src/LeanCode.Contracts/Security/AuthorizeWhenAttribute.cs#L59
Change 'List<AuthorizeWhenAttribute.AuthorizerDefinition>' in 'AuthorizeWhenAttribute<T>.GetCustomAuthorizers()' to use 'Collection<T>', 'ReadOnlyCollection<T>' or 'KeyedCollection<K,V>' (https://learn.microsoft.com/dotnet/fundamentals/code-analysis/quality-rules/ca1002)
Build & Test (6.0): src/LeanCode.Contracts/Security/AuthorizeWhenAttribute.cs#L22
Change 'List<AuthorizeWhenAttribute.AuthorizerDefinition>' in 'AuthorizeWhenAttribute.GetCustomAuthorizers<T>()' to use 'Collection<T>', 'ReadOnlyCollection<T>' or 'KeyedCollection<K,V>' (https://learn.microsoft.com/dotnet/fundamentals/code-analysis/quality-rules/ca1002)
Build & Test (6.0): src/LeanCode.Contracts/Security/AuthorizeWhenAttribute.cs#L25
Change 'List<AuthorizeWhenAttribute.AuthorizerDefinition>' in 'AuthorizeWhenAttribute.GetCustomAuthorizers(Type)' to use 'Collection<T>', 'ReadOnlyCollection<T>' or 'KeyedCollection<K,V>' (https://learn.microsoft.com/dotnet/fundamentals/code-analysis/quality-rules/ca1002)
Build & Test (6.0): src/LeanCode.Contracts/NotificationEnvelope.cs#L16
In externally visible method 'NotificationEnvelope.NotificationEnvelope(Guid id, ITopic topic, object notification)', validate parameter 'topic' is non-null before using it. If appropriate, throw an 'ArgumentNullException' when the argument is 'null'. (https://learn.microsoft.com/dotnet/fundamentals/code-analysis/quality-rules/ca1062)
Build & Test (6.0): src/LeanCode.Contracts/NotificationEnvelope.cs#L17
In externally visible method 'NotificationEnvelope.NotificationEnvelope(Guid id, ITopic topic, object notification)', validate parameter 'notification' is non-null before using it. If appropriate, throw an 'ArgumentNullException' when the argument is 'null'. (https://learn.microsoft.com/dotnet/fundamentals/code-analysis/quality-rules/ca1062)
Build & Test (6.0): src/LeanCode.Contracts/NotificationTagGenerator.cs#L32
'string.IndexOf(char)' has a method overload that takes a 'StringComparison' parameter. Replace this call in 'LeanCode.Contracts.NotificationTagGenerator.GetSimpleName(System.Type)' with a call to 'string.IndexOf(char, System.StringComparison)' for clarity of intent. (https://learn.microsoft.com/dotnet/fundamentals/code-analysis/quality-rules/ca1307)
Build & Test (7.0)
The following actions uses node12 which is deprecated and will be forced to run on node16: actions/checkout@v2. For more info: https://github.blog/changelog/2023-06-13-github-actions-all-actions-will-run-on-node16-instead-of-node12-by-default/
Build & Test (7.0): src/LeanCode.Contracts/Binary.cs#L30
Provide a method named 'ToByteArray' or 'FromNullable' as an alternate for operator op_Implicit (https://learn.microsoft.com/dotnet/fundamentals/code-analysis/quality-rules/ca2225)
Build & Test (7.0): src/LeanCode.Contracts/Binary.cs#L8
Properties should not return arrays (https://learn.microsoft.com/dotnet/fundamentals/code-analysis/quality-rules/ca1819)
Build & Test (7.0): src/LeanCode.Contracts/NotificationTagGenerator.cs#L32
'string.IndexOf(char)' has a method overload that takes a 'StringComparison' parameter. Replace this call in 'LeanCode.Contracts.NotificationTagGenerator.GetSimpleName(System.Type)' with a call to 'string.IndexOf(char, System.StringComparison)' for clarity of intent. (https://learn.microsoft.com/dotnet/fundamentals/code-analysis/quality-rules/ca1307)
Build & Test (7.0): src/LeanCode.Contracts/NotificationTagGenerator.cs#L46
Prefer comparing 'Length' to 0 rather than using 'Any()', both for clarity and for performance (https://learn.microsoft.com/dotnet/fundamentals/code-analysis/quality-rules/ca1860)
Build & Test (7.0): src/LeanCode.Contracts/Converters/BinaryJsonConverter.cs#L6
Type 'BinaryJsonConverter' can be sealed because it has no subtypes in its containing assembly and is not externally visible (https://learn.microsoft.com/dotnet/fundamentals/code-analysis/quality-rules/ca1852)
Build & Test (7.0): src/LeanCode.ContractsGenerator/Analyzers/KnownTypeCheck.cs#L5
Change type of field 'ValidKnownTypeValues' from 'System.Collections.Generic.IReadOnlySet<LeanCode.ContractsGenerator.KnownType>' to 'System.Collections.Generic.HashSet<LeanCode.ContractsGenerator.KnownType>' for improved performance (https://learn.microsoft.com/dotnet/fundamentals/code-analysis/quality-rules/ca1859)
Build & Test (7.0): src/LeanCode.ContractsGenerator/Compilation/CompilationFailedException.cs#L28
The behavior of 'StringBuilder.AppendLine(ref StringBuilder.AppendInterpolatedStringHandler)' could vary based on the current user's locale settings. Replace this call in 'CompilationFailedException.GetDiagnosticsMessage(ImmutableArray<Diagnostic>)' with a call to 'StringBuilder.AppendLine(IFormatProvider, ref StringBuilder.AppendInterpolatedStringHandler)'. (https://learn.microsoft.com/dotnet/fundamentals/code-analysis/quality-rules/ca1305)
Build & Test (7.0): src/LeanCode.ContractsGenerator/Compilation/CompilationFailedException.cs#L28
The behavior of 'Diagnostic.GetMessage([IFormatProvider])' could vary based on the current user's locale settings. Provide a value for the 'IFormatProvider' argument. (https://learn.microsoft.com/dotnet/fundamentals/code-analysis/quality-rules/ca1305)
Build & Test (7.0): src/LeanCode.ContractsGenerator/Generation/NotificationTagGenerator.cs#L29
Possible multiple enumerations of 'IEnumerable' collection. Consider using an implementation that avoids multiple enumerations. (https://learn.microsoft.com/dotnet/fundamentals/code-analysis/quality-rules/ca1851)
Build & Test (7.0): src/LeanCode.ContractsGenerator/Generation/NotificationTagGenerator.cs#L33
Possible multiple enumerations of 'IEnumerable' collection. Consider using an implementation that avoids multiple enumerations. (https://learn.microsoft.com/dotnet/fundamentals/code-analysis/quality-rules/ca1851)
Build & Test (7.0): src/LeanCode.Contracts/Binary.cs#L30
Provide a method named 'ToByteArray' or 'FromNullable' as an alternate for operator op_Implicit (https://learn.microsoft.com/dotnet/fundamentals/code-analysis/quality-rules/ca2225)
Build & Test (7.0): src/LeanCode.Contracts/Binary.cs#L8
Properties should not return arrays (https://learn.microsoft.com/dotnet/fundamentals/code-analysis/quality-rules/ca1819)
Build & Test (7.0): src/LeanCode.Contracts/NotificationTagGenerator.cs#L32
'string.IndexOf(char)' has a method overload that takes a 'StringComparison' parameter. Replace this call in 'LeanCode.Contracts.NotificationTagGenerator.GetSimpleName(System.Type)' with a call to 'string.IndexOf(char, System.StringComparison)' for clarity of intent. (https://learn.microsoft.com/dotnet/fundamentals/code-analysis/quality-rules/ca1307)
Build & Test (7.0): src/LeanCode.Contracts/NotificationTagGenerator.cs#L46
Prefer comparing 'Length' to 0 rather than using 'Any()', both for clarity and for performance (https://learn.microsoft.com/dotnet/fundamentals/code-analysis/quality-rules/ca1860)
Build & Test (7.0): src/LeanCode.Contracts/Converters/BinaryJsonConverter.cs#L6
Type 'BinaryJsonConverter' can be sealed because it has no subtypes in its containing assembly and is not externally visible (https://learn.microsoft.com/dotnet/fundamentals/code-analysis/quality-rules/ca1852)
Build & Test (7.0): src/LeanCode.Contracts/Security/AuthorizeWhenHasAnyOfAttribute.cs#L10
Add a public read-only property accessor for positional argument permissions of Attribute AuthorizeWhenHasAnyOfAttribute (https://learn.microsoft.com/dotnet/fundamentals/code-analysis/quality-rules/ca1019)
Build & Test (7.0): src/LeanCode.Contracts/Binary.cs#L8
Properties should not return arrays (https://learn.microsoft.com/dotnet/fundamentals/code-analysis/quality-rules/ca1819)
Build & Test (7.0): src/LeanCode.Contracts/Binary.cs#L30
Provide a method named 'ToByteArray' or 'FromNullable' as an alternate for operator op_Implicit (https://learn.microsoft.com/dotnet/fundamentals/code-analysis/quality-rules/ca2225)
Build & Test (7.0): src/LeanCode.Contracts/Security/AuthorizeWhenAttribute.cs#L59
Change 'List<AuthorizeWhenAttribute.AuthorizerDefinition>' in 'AuthorizeWhenAttribute<T>.GetCustomAuthorizers()' to use 'Collection<T>', 'ReadOnlyCollection<T>' or 'KeyedCollection<K,V>' (https://learn.microsoft.com/dotnet/fundamentals/code-analysis/quality-rules/ca1002)
Build & Test (7.0): src/LeanCode.Contracts/Security/AuthorizeWhenAttribute.cs#L22
Change 'List<AuthorizeWhenAttribute.AuthorizerDefinition>' in 'AuthorizeWhenAttribute.GetCustomAuthorizers<T>()' to use 'Collection<T>', 'ReadOnlyCollection<T>' or 'KeyedCollection<K,V>' (https://learn.microsoft.com/dotnet/fundamentals/code-analysis/quality-rules/ca1002)
Publish Tests Results
The following actions uses node12 which is deprecated and will be forced to run on node16: actions/download-artifact@v2. For more info: https://github.blog/changelog/2023-06-13-github-actions-all-actions-will-run-on-node16-instead-of-node12-by-default/

Artifacts

Produced during runtime
Name Size
test_results_6.0.zip Expired
129 KB
test_results_7.0.zip Expired
129 KB