Suppress CA2012 warning for ChannelClosed method

electricessence · electricessence · commit 1453eb9a411e · 2024-11-18T08:15:12.000-08:00
Add suppression for CA2012 in ExceptionTests.ChannelClosed to ensure synchronous operation. Wrap with IDE0079 disable/enable to prevent unnecessary suppression warning.
diff --git a/Open.ChannelExtensions.Tests/ExceptionTests.cs b/Open.ChannelExtensions.Tests/ExceptionTests.cs
@@ -96,7 +96,9 @@ await range
 	}
 
 	[Fact]
+#pragma warning disable IDE0079 // Remove unnecessary suppression
 	[SuppressMessage("Reliability", "CA2012:Use ValueTasks correctly", Justification = "Needs to happen synchronously")]
+#pragma warning restore IDE0079 // Remove unnecessary suppression
 	public static void ChannelClosed()
 	{
 		var channel = Channel.CreateBounded<int>(new BoundedChannelOptions(1000)

Original file line number	Diff line number	Diff line change
`@@ -96,7 +96,9 @@ await range`
`96`	`96`	`}`
`97`	`97`
`98`	`98`	`[Fact]`
	`99`	`+#pragma warning disable IDE0079 // Remove unnecessary suppression`
`99`	`100`	`[SuppressMessage("Reliability", "CA2012:Use ValueTasks correctly", Justification = "Needs to happen synchronously")]`
	`101`	`+#pragma warning restore IDE0079 // Remove unnecessary suppression`
`100`	`102`	`public static void ChannelClosed()`
`101`	`103`	`{`
`102`	`104`	`var channel = Channel.CreateBounded<int>(new BoundedChannelOptions(1000)`