Reorg'ed tests so test classes share a single BrowsingContext and parser

Author: egil

src/CompareResult.cs src/Core/CompareResult.cssrc/CompareResult.cs renamed to src/Core/CompareResult.cs

@@ -1,4 +1,4 @@
-namespace Egil.AngleSharp.Diffing
+namespace Egil.AngleSharp.Diffing.Core
 {
     public enum CompareResult
     {

src/Strategies/AttributeStrategies/AttributeComparer.cs

@@ -3,7 +3,7 @@
 using Egil.AngleSharp.Diffing.Core;
 
 namespace Egil.AngleSharp.Diffing.Strategies.AttributeStrategies
-{    
+{
     public static class AttributeComparer
     {
         private const string IGNORE_CASE_POSTFIX = ":ignorecase";

tests/Core/ComparisonSourceTest.cs

@@ -5,6 +5,10 @@ namespace Egil.AngleSharp.Diffing.Core
 {
     public class ComparisonSourceTest : DiffingTestBase
     {
+        public ComparisonSourceTest(DiffingTestFixture fixture) : base(fixture)
+        {
+        }
+
         [Fact(DisplayName = "Two sources are equal if all their properties are equal")]
         public void Test1()
         {

tests/Core/DiffingEngineTestBase.cs

@@ -5,6 +5,10 @@ namespace Egil.AngleSharp.Diffing.Core
 {
     public abstract class DiffingEngineTestBase : DiffingTestBase
     {
+        public DiffingEngineTestBase(DiffingTestFixture fixture) : base(fixture)
+        {
+        }
+
         protected static HtmlDifferenceEngine CreateHtmlDiffEngine(
                 Func<DiffContext, SourceCollection, SourceCollection, IEnumerable<Comparison>>? nodeMatcher = null,
                 Func<DiffContext, SourceMap, SourceMap, IEnumerable<AttributeComparison>>? attrMatcher = null,

tests/Core/HtmlDifferenceEngineTest.cs

@@ -8,6 +8,10 @@ namespace Egil.AngleSharp.Diffing.Core
 {
     public class HtmlDifferenceEngineTest : DiffingEngineTestBase
     {
+        public HtmlDifferenceEngineTest(DiffingTestFixture fixture) : base(fixture)
+        {
+        }
+
         [Fact(DisplayName = "Unmatched nodes in control/test are returned as missing/unexpected diffs")]
         public void UnmatchedNodesBecomesMissingUnexpectedDiffs()
         {

tests/Core/SourceCollectionTest.cs

@@ -7,6 +7,10 @@ namespace Egil.AngleSharp.Diffing.Core
 {
     public class SourceCollectionTest : DiffingTestBase
     {
+        public SourceCollectionTest(DiffingTestFixture fixture) : base(fixture)
+        {
+        }
+
         [Theory(DisplayName = "Source type is correct saved in the collection")]
         [InlineData(ComparisonSourceType.Control)]
         [InlineData(ComparisonSourceType.Test)]

tests/Core/SourceMapTest.cs

@@ -7,6 +7,10 @@ namespace Egil.AngleSharp.Diffing.Core
 {
     public class SourceMapTest : DiffingTestBase
     {
+        public SourceMapTest(DiffingTestFixture fixture) : base(fixture)
+        {
+        }
+
         [Fact(DisplayName = "When initialized with a non-element an exception is thrown")]
         public void Test0()
         {

tests/DiffingStrategyPipelineTest.cs

@@ -10,6 +10,10 @@ public class DiffingStrategyPipelineTest : DiffingTestBase
     {
         private DiffContext _dummyContext = new DiffContext(null, null);
 
+        public DiffingStrategyPipelineTest(DiffingTestFixture fixture) : base(fixture)
+        {
+        }
+
         private FilterDecision NegateDecision(FilterDecision decision) => decision switch
         {
             FilterDecision.Keep => FilterDecision.Exclude,

tests/Core/DiffingTestBase.cs tests/DiffingTestBase.cstests/Core/DiffingTestBase.cs renamed to tests/DiffingTestBase.cs

@@ -2,28 +2,25 @@
 using AngleSharp;
 using AngleSharp.Dom;
 using AngleSharp.Html.Parser;
+using Egil.AngleSharp.Diffing.Core;
+using Xunit;
 
-namespace Egil.AngleSharp.Diffing.Core
+namespace Egil.AngleSharp.Diffing
 {
-    public abstract class DiffingTestBase
+    public abstract class DiffingTestBase : IClassFixture<DiffingTestFixture>
     {
-        private readonly IBrowsingContext _context;
-        private readonly IHtmlParser _htmlParser;
-        private readonly IDocument _document;
+        private readonly DiffingTestFixture _testFixture;
 
         protected INodeList EmptyNodeList => ToNodeList("");
 
-        protected DiffingTestBase()
+        public DiffingTestBase(DiffingTestFixture fixture)
         {
-            var config = Configuration.Default.WithCss();
-            _context = BrowsingContext.New(config);
-            _htmlParser = _context.GetService<IHtmlParser>();
-            _document = _context.OpenNewAsync().Result;
+            _testFixture = fixture;
         }
 
         protected INodeList ToNodeList(string? htmlsnippet)
         {
-            var fragment = _htmlParser.ParseFragment(htmlsnippet, _document.Body);
+            var fragment = _testFixture.Parse(htmlsnippet);
             return fragment;
         }
 
@@ -34,7 +31,7 @@ protected IEnumerable<ComparisonSource> ToComparisonSourceList(string html)
 
         protected INode ToNode(string htmlsnippet)
         {
-            var fragment = _htmlParser.ParseFragment(htmlsnippet, _document.Body);
+            var fragment = _testFixture.Parse(htmlsnippet);
             return fragment[0];
         }
 

tests/DiffingTestFixture.cs

@@ -0,0 +1,26 @@
+using AngleSharp;
+using AngleSharp.Dom;
+using AngleSharp.Html.Parser;
+
+namespace Egil.AngleSharp.Diffing
+{
+    public class DiffingTestFixture
+    {
+        private readonly IBrowsingContext _context;
+        private readonly IHtmlParser _htmlParser;
+        private readonly IDocument _document;
+
+        public DiffingTestFixture()
+        {
+            var config = Configuration.Default.WithCss();
+            _context = BrowsingContext.New(config);
+            _htmlParser = _context.GetService<IHtmlParser>();
+            _document = _context.OpenNewAsync().Result;
+        }
+
+        public INodeList Parse(string? html)
+        {
+            return _htmlParser.ParseFragment(html, _document.Body);
+        }
+    }
+}