From 003e7971808b64551b5147c0bae86748f80a741d Mon Sep 17 00:00:00 2001 From: "dependabot[bot]" <49699333+dependabot[bot]@users.noreply.github.com> Date: Thu, 12 Oct 2023 16:15:15 +0000 Subject: [PATCH 1/3] Bump SonarAnalyzer.CSharp from 9.11.0.78383 to 9.12.0.78982 Bumps [SonarAnalyzer.CSharp](https://github.com/SonarSource/sonar-dotnet) from 9.11.0.78383 to 9.12.0.78982. - [Release notes](https://github.com/SonarSource/sonar-dotnet/releases) - [Commits](https://github.com/SonarSource/sonar-dotnet/compare/9.11.0.78383...9.12.0.78982) --- updated-dependencies: - dependency-name: SonarAnalyzer.CSharp dependency-type: direct:production update-type: version-update:semver-minor ... Signed-off-by: dependabot[bot] --- Directory.Packages.props | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/Directory.Packages.props b/Directory.Packages.props index 399fd208f6e..1b5c81ede0e 100644 --- a/Directory.Packages.props +++ b/Directory.Packages.props @@ -31,7 +31,7 @@ - + From cedcd7d07c71565da65921b4039e162831765f81 Mon Sep 17 00:00:00 2001 From: Martin Costello Date: Thu, 12 Oct 2023 17:40:10 +0100 Subject: [PATCH 2/3] Fix S3257 warnings Fix two S3257 warnings. --- .../Simmy/Outcomes/OutcomePipelineBuilderExtensions.TResult.cs | 2 +- .../Outcomes/OutcomeChaosPipelineBuilderExtensionsTests.cs | 2 +- 2 files changed, 2 insertions(+), 2 deletions(-) diff --git a/src/Polly.Core/Simmy/Outcomes/OutcomePipelineBuilderExtensions.TResult.cs b/src/Polly.Core/Simmy/Outcomes/OutcomePipelineBuilderExtensions.TResult.cs index 7a038719d2a..30cb350dea5 100644 --- a/src/Polly.Core/Simmy/Outcomes/OutcomePipelineBuilderExtensions.TResult.cs +++ b/src/Polly.Core/Simmy/Outcomes/OutcomePipelineBuilderExtensions.TResult.cs @@ -86,7 +86,7 @@ public static ResiliencePipelineBuilder AddChaosResult(this Re { Enabled = enabled, InjectionRate = injectionRate, - Outcome = new(result) + Outcome = result }); return builder; } diff --git a/test/Polly.Core.Tests/Simmy/Outcomes/OutcomeChaosPipelineBuilderExtensionsTests.cs b/test/Polly.Core.Tests/Simmy/Outcomes/OutcomeChaosPipelineBuilderExtensionsTests.cs index cb3f5ea3d2a..0288f71a55a 100644 --- a/test/Polly.Core.Tests/Simmy/Outcomes/OutcomeChaosPipelineBuilderExtensionsTests.cs +++ b/test/Polly.Core.Tests/Simmy/Outcomes/OutcomeChaosPipelineBuilderExtensionsTests.cs @@ -16,7 +16,7 @@ public class OutcomeChaosPipelineBuilderExtensionsTests InjectionRate = 0.6, Enabled = true, Randomizer = () => 0.5, - Outcome = new(100) + Outcome = 100 }); AssertResultStrategy(builder, true, 0.6, new(100)) From 06da74750ccb08537d5f5f117ea22b303ff16189 Mon Sep 17 00:00:00 2001 From: martincostello Date: Fri, 13 Oct 2023 09:32:50 +0100 Subject: [PATCH 3/3] Fix S3257 warnings Actually fix the S3257 warnings. --- .../Outcomes/OutcomePipelineBuilderExtensions.TResult.cs | 2 +- .../Outcomes/OutcomeChaosPipelineBuilderExtensionsTests.cs | 4 ++-- 2 files changed, 3 insertions(+), 3 deletions(-) diff --git a/src/Polly.Core/Simmy/Outcomes/OutcomePipelineBuilderExtensions.TResult.cs b/src/Polly.Core/Simmy/Outcomes/OutcomePipelineBuilderExtensions.TResult.cs index 30cb350dea5..e68873980b0 100644 --- a/src/Polly.Core/Simmy/Outcomes/OutcomePipelineBuilderExtensions.TResult.cs +++ b/src/Polly.Core/Simmy/Outcomes/OutcomePipelineBuilderExtensions.TResult.cs @@ -86,7 +86,7 @@ public static ResiliencePipelineBuilder AddChaosResult(this Re { Enabled = enabled, InjectionRate = injectionRate, - Outcome = result + Outcome = new Outcome(result) }); return builder; } diff --git a/test/Polly.Core.Tests/Simmy/Outcomes/OutcomeChaosPipelineBuilderExtensionsTests.cs b/test/Polly.Core.Tests/Simmy/Outcomes/OutcomeChaosPipelineBuilderExtensionsTests.cs index 0288f71a55a..1dbb2fe5a18 100644 --- a/test/Polly.Core.Tests/Simmy/Outcomes/OutcomeChaosPipelineBuilderExtensionsTests.cs +++ b/test/Polly.Core.Tests/Simmy/Outcomes/OutcomeChaosPipelineBuilderExtensionsTests.cs @@ -16,7 +16,7 @@ public class OutcomeChaosPipelineBuilderExtensionsTests InjectionRate = 0.6, Enabled = true, Randomizer = () => 0.5, - Outcome = 100 + Outcome = new Outcome(100) }); AssertResultStrategy(builder, true, 0.6, new(100)) @@ -28,7 +28,7 @@ public class OutcomeChaosPipelineBuilderExtensionsTests { builder => { - builder.AddChaosFault(new FaultStrategyOptions + builder.AddChaosFault(new FaultStrategyOptions { InjectionRate = 0.6, Enabled = true,