COnverted TestBed class into a non-abstract one to make overriding Clear() and DisposeAsync() optional #107

Author: Arash-Sabet

examples/Xunit.Microsoft.DependencyInjection.ExampleTests/IntegrationTests.cs

@@ -1,42 +1,34 @@
-using Microsoft.Extensions.Options;
+using Microsoft.Extensions.Options;
 using Options = Xunit.Microsoft.DependencyInjection.ExampleTests.Services.Options;
 
-namespace Xunit.Microsoft.DependencyInjection.ExampleTests
+namespace Xunit.Microsoft.DependencyInjection.ExampleTests;
+
+public class IntegrationTests : TestBed<TestFixture>
 {
-    public class IntegrationTests : TestBed<TestFixture>
+    public IntegrationTests(ITestOutputHelper testOutputHelper, TestFixture fixture)
+        : base(testOutputHelper, fixture)
     {
-        public IntegrationTests(ITestOutputHelper testOutputHelper, TestFixture fixture)
-            : base(testOutputHelper, fixture)
-        {
-        }
-
-        [Theory]
-        [InlineData(1, 2)]
-        public void Test1(int x, int y)
-        {
-            var calculator = _fixture.GetService<ICalculator>(_testOutputHelper);
-            var option = _fixture.GetService<IOptions<Options>>(_testOutputHelper);
-            var calculated = calculator?.Add(x, y);
-            var expected = option?.Value.Rate * (x + y);
-            Assert.True(expected == calculated);
-        }
-
-        [Theory]
-        [InlineData(1, 2)]
-        public void Test2(int x, int y)
-        {
-            var calculator = _fixture.GetScopedService<ICalculator>(_testOutputHelper);
-            var option = _fixture.GetScopedService<IOptions<Options>>(_testOutputHelper);
-            var calculated = calculator?.Add(x, y);
-            var expected = option?.Value.Rate * (x + y);
-            Assert.True(expected == calculated);
-        }
+    }
 
-        protected override void Clear()
-        {
-        }
+    [Theory]
+    [InlineData(1, 2)]
+    public void Test1(int x, int y)
+    {
+        var calculator = _fixture.GetService<ICalculator>(_testOutputHelper);
+        var option = _fixture.GetService<IOptions<Options>>(_testOutputHelper);
+        var calculated = calculator?.Add(x, y);
+        var expected = option?.Value.Rate * (x + y);
+        Assert.True(expected == calculated);
+    }
 
-        protected override ValueTask DisposeAsyncCore()
-            => new();
+    [Theory]
+    [InlineData(1, 2)]
+    public void Test2(int x, int y)
+    {
+        var calculator = _fixture.GetScopedService<ICalculator>(_testOutputHelper);
+        var option = _fixture.GetScopedService<IOptions<Options>>(_testOutputHelper);
+        var calculated = calculator?.Add(x, y);
+        var expected = option?.Value.Rate * (x + y);
+        Assert.True(expected == calculated);
     }
 }

src/Abstracts/TestBed.cs

@@ -1,6 +1,6 @@
 namespace Xunit.Microsoft.DependencyInjection.Abstracts;
 
-public abstract class TestBed<TFixture> : IDisposable, IClassFixture<TFixture>, IAsyncDisposable
+public class TestBed<TFixture> : IDisposable, IClassFixture<TFixture>, IAsyncDisposable
 	where TFixture : class
 {
 	protected readonly ITestOutputHelper _testOutputHelper;
@@ -41,8 +41,6 @@ public void Dispose()
 		GC.SuppressFinalize(this);
 	}
 
-	protected abstract void Clear();
-
 	public async ValueTask DisposeAsync()
 	{
 		if (!_disposedAsync)
@@ -53,5 +51,6 @@ public async ValueTask DisposeAsync()
 		}
 	}
 
-	protected abstract ValueTask DisposeAsyncCore();
+	protected virtual void Clear() { }
+	protected virtual ValueTask DisposeAsyncCore() => new();
 }