From d93fb9464da1dd5a89330165cf031056413967cf Mon Sep 17 00:00:00 2001 From: Jake Meiergerd Date: Tue, 5 Dec 2023 22:33:03 -0600 Subject: [PATCH] Removed dead testing code that was disabled previously, and is now superseded by fd9083f90eef3301cb78961886054598ab2d4931 from #760. (#782) --- ...bservableChangeSetFixtureWithCompletion.cs | 46 ------------------- 1 file changed, 46 deletions(-) delete mode 100644 src/DynamicData.Tests/Cache/ToObservableChangeSetFixtureWithCompletion.cs diff --git a/src/DynamicData.Tests/Cache/ToObservableChangeSetFixtureWithCompletion.cs b/src/DynamicData.Tests/Cache/ToObservableChangeSetFixtureWithCompletion.cs deleted file mode 100644 index 1d594f990..000000000 --- a/src/DynamicData.Tests/Cache/ToObservableChangeSetFixtureWithCompletion.cs +++ /dev/null @@ -1,46 +0,0 @@ -using System; -using System.Collections.Generic; -using System.Reactive.Subjects; - -using DynamicData.Tests.Domain; - -using FluentAssertions; - -namespace DynamicData.Tests.Cache; - -public class ToObservableChangeSetFixtureWithCompletion : IDisposable -{ - private readonly IDisposable _disposable; - - private readonly ISubject _observable; - - private readonly List _target; - - private bool _hasCompleted = false; - - public ToObservableChangeSetFixtureWithCompletion() - { - _observable = new Subject(); - - _target = new List(); - - _disposable = _observable.ToObservableChangeSet(p => p.Key).Clone(_target).Subscribe(x => { }, () => _hasCompleted = true); - } - - // [Fact] - disabled as it's questionable whether the completion should be invoked - public void ShouldReceiveUpdatesThenComplete() - { - _observable.OnNext(new Person("One", 1)); - _observable.OnNext(new Person("Two", 2)); - - _target.Count.Should().Be(2); - - _observable.OnCompleted(); - _hasCompleted.Should().Be(true); - - _observable.OnNext(new Person("Three", 3)); - _target.Count.Should().Be(2); - } - - public void Dispose() => _disposable.Dispose(); -}