Incremental commit

- Added default values for CaptureOptions parameter of Profile methods
waxtell · Feb 27, 2019 · 470b48f · 470b48f
1 parent 472d30a
commit 470b48f
Show file tree

Hide file tree

Showing 2 changed files with 29 additions and 4 deletions.
diff --git a/Aop.Profiler.Unit.Tests/PerMethodAdapterTests.cs b/Aop.Profiler.Unit.Tests/PerMethodAdapterTests.cs
@@ -159,6 +159,31 @@ void EventProcessor(IDictionary<string, object> @event)
             }
         }
 
+        [Fact]
+        public void DefaultOptionsYieldExpectedDictionarySize()
+        {
+            var eventCount = 0;
+            var itemCount = 0;
+
+            var proxy = new PerMethodAdapter<ForTestingPurposes>
+                (
+                    Process.Lean(EventProcessor)
+                )
+                .Profile(x => x.VirtualMethodCall(It.IsAny<int>(), It.IsAny<string>()))
+                .Adapt(new ForTestingPurposes());
+
+            proxy.VirtualMethodCall(0, "zero");
+
+            Assert.Equal(1, eventCount);
+            Assert.Equal(BitFunctions.CountSet((int)CaptureOptions.Default), itemCount);
+
+            void EventProcessor(IDictionary<string, object> @event)
+            {
+                eventCount++;
+                itemCount = @event.Count;
+            }
+        }
+
         [Fact]
         public async Task AsyncActionDoesNotAllowSerializedResultOption()
         {

diff --git a/Aop.Profiler/PerMethodAdapter.cs b/Aop.Profiler/PerMethodAdapter.cs
@@ -9,7 +9,7 @@ namespace Aop.Profiler
 {
     public class PerMethodAdapter<T> : BaseAdapter<T>, IPerMethodAdapter<T> where T : class
     {
-        public IPerMethodAdapter<T> Profile<TReturn>(Expression<Func<T, Task<TReturn>>> target, CaptureOptions captureOptions)
+        public IPerMethodAdapter<T> Profile<TReturn>(Expression<Func<T, Task<TReturn>>> target, CaptureOptions captureOptions = CaptureOptions.Default)
         {
             return
                 Profile
@@ -20,7 +20,7 @@ public IPerMethodAdapter<T> Profile<TReturn>(Expression<Func<T, Task<TReturn>>>
                 );
         }
 
-        public IPerMethodAdapter<T> Profile(Expression<Func<T, Task>> target, CaptureOptions captureOptions)
+        public IPerMethodAdapter<T> Profile(Expression<Func<T, Task>> target, CaptureOptions captureOptions = CaptureOptions.Default)
         {
             return
                 Profile
@@ -31,7 +31,7 @@ public IPerMethodAdapter<T> Profile(Expression<Func<T, Task>> target, CaptureOpt
                 );
         }
 
-        public IPerMethodAdapter<T> Profile(Expression<Action<T>> target, CaptureOptions captureOptions)
+        public IPerMethodAdapter<T> Profile(Expression<Action<T>> target, CaptureOptions captureOptions = CaptureOptions.Default)
         {
             Profile
             (
@@ -43,7 +43,7 @@ public IPerMethodAdapter<T> Profile(Expression<Action<T>> target, CaptureOptions
             return this;
         }
 
-        public IPerMethodAdapter<T> Profile<TReturn>(Expression<Func<T, TReturn>> target, CaptureOptions captureOptions)
+        public IPerMethodAdapter<T> Profile<TReturn>(Expression<Func<T, TReturn>> target, CaptureOptions captureOptions = CaptureOptions.Default)
         {
             return
                 Profile