Merge pull request #176 from JSkimming/use-method-groups

Use method groups

Author: JSkimming

.dockerignore

@@ -1,8 +1,24 @@
-**/obj
+**/.classpath
+**/.dockerignore
+**/.env
+**/.git
+**/.github
+**/.gitignore
+**/.circleci
+**/.project
+**/.settings
+**/.toolstarget
+**/.vs
+**/.vscode
+**/*.*proj.user
+**/*.dbmdl
+**/*.jfm
+**/azds.yaml
 **/bin
-.vs
-.git
-.gitignore
-.circleci
-Dockerfile
+**/charts
+**/docker-compose*
+**/Dockerfile*
+**/obj
+**/*.md
+LICENSE
 **/TestResults

castle.core.asyncinterceptor.sln.DotSettings

@@ -9,4 +9,6 @@
 	<s:String x:Key="/Default/CodeStyle/CSharpUsing/MandatoryImports/=System_002ELinq/@EntryIndexedValue">System.Linq</s:String>
 	<s:Boolean x:Key="/Default/CodeStyle/CSharpUsing/QualifiedUsingAtNestedScope/@EntryValue">True</s:Boolean>
 	<s:String x:Key="/Default/CodeStyle/Naming/JavaScriptNaming/UserRules/=JS_005FBLOCK_005FSCOPE_005FFUNCTION/@EntryIndexedValue">&lt;Policy Inspect="True" Prefix="" Suffix="" Style="aaBb" /&gt;</s:String>
-	<s:String x:Key="/Default/CodeStyle/Naming/JavaScriptNaming/UserRules/=JS_005FCLASS/@EntryIndexedValue">&lt;Policy Inspect="True" Prefix="" Suffix="" Style="aaBb" /&gt;</s:String></wpf:ResourceDictionary>
+	<s:String x:Key="/Default/CodeStyle/Naming/JavaScriptNaming/UserRules/=JS_005FCLASS/@EntryIndexedValue">&lt;Policy Inspect="True" Prefix="" Suffix="" Style="aaBb" /&gt;</s:String>
+	<s:Boolean x:Key="/Default/Environment/Filtering/ExcludeCoverageFilters/=_002A_002ETests_003B_002A_003B_002A_003B_002A/@EntryIndexedValue">True</s:Boolean>
+	<s:Boolean x:Key="/Default/Environment/Filtering/ExcludeCoverageFilters/=_002A_003B_002A_003B_002A_002ENoCoverage_002E_002A_003B_002A/@EntryIndexedValue">True</s:Boolean></wpf:ResourceDictionary>

test/Castle.Core.AsyncInterceptor.Tests/AsyncExceptionInterceptorShould.cs

@@ -58,7 +58,7 @@ public void ShouldLog3Entries()
     {
         // Act
         InvalidOperationException ex =
-            Assert.Throws<InvalidOperationException>(() => _proxy.SynchronousVoidExceptionMethod());
+            Assert.Throws<InvalidOperationException>(_proxy.SynchronousVoidExceptionMethod);
 
         // Assert
         Assert.Equal(3, _log.Count);
@@ -71,7 +71,7 @@ public void ShouldLog3Entries()
     public void ShouldAllowProcessingPriorToInvocation()
     {
         // Act
-        Assert.Throws<InvalidOperationException>(() => _proxy.SynchronousVoidExceptionMethod());
+        Assert.Throws<InvalidOperationException>(_proxy.SynchronousVoidExceptionMethod);
 
         // Assert
         Assert.Equal(MethodName + ":StartingVoidInvocation", _log[0]);
@@ -82,7 +82,7 @@ public void ShouldAllowExceptionHandling()
     {
         // Act
         InvalidOperationException ex =
-            Assert.Throws<InvalidOperationException>(() => _proxy.SynchronousVoidExceptionMethod());
+            Assert.Throws<InvalidOperationException>(_proxy.SynchronousVoidExceptionMethod);
 
         // Assert
         Assert.Equal(MethodName + ":VoidExceptionThrown:" + ex.Message, _log[2]);
@@ -96,13 +96,13 @@ public void ShouldPreserveTheStackTrace()
 
         // Act
         InvalidOperationException interceptedException =
-            Assert.Throws<InvalidOperationException>(() => _proxy.SynchronousVoidExceptionMethod());
+            Assert.Throws<InvalidOperationException>(_proxy.SynchronousVoidExceptionMethod);
 
         // Assert
 
         // Get the exception without being intercepted, this is used to compare against the intercepted exception.
         InvalidOperationException noneInterceptedException =
-            Assert.Throws<InvalidOperationException>(() => _target.SynchronousVoidExceptionMethod());
+            Assert.Throws<InvalidOperationException>(_target.SynchronousVoidExceptionMethod);
 
         CompareStackTrace(noneInterceptedException, interceptedException);
     }
@@ -509,7 +509,7 @@ public async Task ShouldReturnAFaultedTask()
         Assert.IsType<ArgumentOutOfRangeException>(result.Exception?.InnerException);
 
         ArgumentOutOfRangeException noneInterceptedException =
-            await Assert.ThrowsAsync<ArgumentOutOfRangeException>(() => target.Test1()).ConfigureAwait(false);
+            await Assert.ThrowsAsync<ArgumentOutOfRangeException>(target.Test1).ConfigureAwait(false);
 
         CompareStackTrace(noneInterceptedException, result.Exception?.InnerException);
     }
@@ -529,7 +529,7 @@ public async Task ShouldReturnAFaultedTaskResult()
         Assert.IsType<ArgumentOutOfRangeException>(result.Exception?.InnerException);
 
         ArgumentOutOfRangeException noneInterceptedException =
-            await Assert.ThrowsAsync<ArgumentOutOfRangeException>(() => target.Test2()).ConfigureAwait(false);
+            await Assert.ThrowsAsync<ArgumentOutOfRangeException>(target.Test2).ConfigureAwait(false);
 
         CompareStackTrace(noneInterceptedException, result.Exception?.InnerException);
     }