From 9f9f88aab78410bc672cfcd04b8814855a7f1cd3 Mon Sep 17 00:00:00 2001 From: Nicolas Constant Date: Mon, 7 Feb 2022 19:37:28 -0500 Subject: [PATCH] clean up --- .../RefreshTwitterUserStatusProcessorTests.cs | 98 ++----------------- 1 file changed, 7 insertions(+), 91 deletions(-) diff --git a/src/Tests/BirdsiteLive.Pipeline.Tests/Processors/RefreshTwitterUserStatusProcessorTests.cs b/src/Tests/BirdsiteLive.Pipeline.Tests/Processors/RefreshTwitterUserStatusProcessorTests.cs index be342c0..cd2d116 100644 --- a/src/Tests/BirdsiteLive.Pipeline.Tests/Processors/RefreshTwitterUserStatusProcessorTests.cs +++ b/src/Tests/BirdsiteLive.Pipeline.Tests/Processors/RefreshTwitterUserStatusProcessorTests.cs @@ -289,10 +289,7 @@ namespace BirdsiteLive.Pipeline.Tests.Processors twitterUserServiceMock .Setup(x => x.GetUser(It.Is(y => y == acct2))) .Throws(new Exception()); - - //twitterUserServiceMock - // .Setup(x => x.PurgeUser(It.Is(y => y == acct2))); - + var twitterUserDalMock = new Mock(MockBehavior.Strict); twitterUserDalMock .Setup(x => x.GetTwitterUserAsync(It.Is(y => y == acct2))) @@ -364,10 +361,7 @@ namespace BirdsiteLive.Pipeline.Tests.Processors twitterUserServiceMock .Setup(x => x.GetUser(It.Is(y => y == acct2))) .Returns((TwitterUser)null); - - //twitterUserServiceMock - // .Setup(x => x.PurgeUser(It.Is(y => y == acct2))); - + var twitterUserDalMock = new Mock(MockBehavior.Strict); twitterUserDalMock .Setup(x => x.GetTwitterUserAsync(It.Is(y => y == acct2))) @@ -439,10 +433,7 @@ namespace BirdsiteLive.Pipeline.Tests.Processors twitterUserServiceMock .Setup(x => x.GetUser(It.Is(y => y == acct2))) .Returns((TwitterUser)null); - - //twitterUserServiceMock - // .Setup(x => x.PurgeUser(It.Is(y => y == acct2))); - + var twitterUserDalMock = new Mock(MockBehavior.Strict); twitterUserDalMock .Setup(x => x.GetTwitterUserAsync(It.Is(y => y == acct2))) @@ -516,10 +507,7 @@ namespace BirdsiteLive.Pipeline.Tests.Processors { Protected = true }); - - //twitterUserServiceMock - // .Setup(x => x.PurgeUser(It.Is(y => y == acct2))); - + var twitterUserDalMock = new Mock(MockBehavior.Strict); twitterUserDalMock .Setup(x => x.GetTwitterUserAsync(It.Is(y => y == acct2))) @@ -594,10 +582,7 @@ namespace BirdsiteLive.Pipeline.Tests.Processors { Protected = true }); - - //twitterUserServiceMock - // .Setup(x => x.PurgeUser(It.Is(y => y == acct2))); - + var twitterUserDalMock = new Mock(MockBehavior.Strict); twitterUserDalMock .Setup(x => x.GetTwitterUserAsync(It.Is(y => y == acct2))) @@ -625,70 +610,7 @@ namespace BirdsiteLive.Pipeline.Tests.Processors removeTwitterAccountActionMock.VerifyAll(); #endregion } - - //[TestMethod] - //public async Task ProcessAsync_Protected_Test() - //{ - // #region Stubs - // var userId1 = 1; - // var acct1 = "user1"; - - // var userId2 = 2; - // var acct2 = "user2"; - - // var users = new List - // { - // new SyncTwitterUser - // { - // Id = userId1, - // Acct = acct1 - // }, - // new SyncTwitterUser - // { - // Id = userId2, - // Acct = acct2 - // } - // }; - - // var settings = new InstanceSettings - // { - // FailingTwitterUserCleanUpThreshold = 300 - // }; - // #endregion - - // #region Mocks - // var twitterUserServiceMock = new Mock(MockBehavior.Strict); - // twitterUserServiceMock - // .Setup(x => x.GetUser(It.Is(y => y == acct1))) - // .Returns(new TwitterUser - // { - // Protected = false - // }); - - // twitterUserServiceMock - // .Setup(x => x.GetUser(It.Is(y => y == acct2))) - // .Returns(new TwitterUser - // { - // Protected = true - // }); - - // var twitterUserDalMock = new Mock(MockBehavior.Strict); - // var removeTwitterAccountActionMock = new Mock(MockBehavior.Strict); - // #endregion - - // var processor = new RefreshTwitterUserStatusProcessor(twitterUserServiceMock.Object, twitterUserDalMock.Object, removeTwitterAccountActionMock.Object, settings); - // var result = await processor.ProcessAsync(users.ToArray(), CancellationToken.None); - - // #region Validations - // Assert.AreEqual(1, result.Length); - // Assert.IsTrue(result.Any(x => x.User.Id == userId1)); - - // twitterUserServiceMock.VerifyAll(); - // twitterUserDalMock.VerifyAll(); - // removeTwitterAccountActionMock.VerifyAll(); - // #endregion - //} - + [TestMethod] public async Task ProcessAsync_Error_NotInit_Test() { @@ -720,9 +642,6 @@ namespace BirdsiteLive.Pipeline.Tests.Processors .Setup(x => x.GetUser(It.Is(y => y == acct1))) .Returns((TwitterUser)null); - //twitterUserServiceMock - // .Setup(x => x.PurgeUser(It.Is(y => y == acct1))); - var twitterUserDalMock = new Mock(MockBehavior.Strict); twitterUserDalMock .Setup(x => x.GetTwitterUserAsync(It.Is(y => y == acct1))) @@ -790,10 +709,7 @@ namespace BirdsiteLive.Pipeline.Tests.Processors twitterUserServiceMock .Setup(x => x.GetUser(It.Is(y => y == acct2))) .Throws(new RateLimitExceededException()); - - //twitterUserServiceMock - // .Setup(x => x.PurgeUser(It.Is(y => y == acct2))); - + var twitterUserDalMock = new Mock(MockBehavior.Strict); twitterUserDalMock .Setup(x => x.GetTwitterUserAsync(It.Is(y => y == acct2)))