From d432d3d86d3adfad73e7cce596cc318365fe2231 Mon Sep 17 00:00:00 2001
From: Luke Bakken <luke@bakken.io>
Date: Tue, 12 Sep 2023 14:56:31 -0700
Subject: [PATCH] Ensure that the order of arguments is `expected`, `actual`,
 take two!

Fix yolo-copy from 6.x branch
---
 .../TestOAuth2ClientCredentialsProvider.cs    | 28 ++++++++++---------
 .../Unit/TestTimerBasedCredentialRefresher.cs | 12 ++++----
 2 files changed, 22 insertions(+), 18 deletions(-)

diff --git a/projects/Unit/TestOAuth2ClientCredentialsProvider.cs b/projects/Unit/TestOAuth2ClientCredentialsProvider.cs
index fbc1647739..bf2bee6eb6 100644
--- a/projects/Unit/TestOAuth2ClientCredentialsProvider.cs
+++ b/projects/Unit/TestOAuth2ClientCredentialsProvider.cs
@@ -30,7 +30,7 @@
 //---------------------------------------------------------------------------
 
 using System;
-
+using System.Threading;
 using Moq;
 using RabbitMQ.Client.OAuth2;
 using Xunit;
@@ -67,8 +67,8 @@ public void ShouldRequestTokenWhenGettingPasswordOrValidUntilForFirstTimeAccess(
         {
             IToken firstToken = newToken("the_access_token", "the_refresh_token", TimeSpan.FromSeconds(1));
             _oAuth2Client.Setup(p => p.RequestToken()).Returns(firstToken);
-            Assert.Equal(_provider.Password, firstToken.AccessToken);
-            Assert.Equal(_provider.ValidUntil.Value, firstToken.ExpiresIn);
+            Assert.Equal(firstToken.AccessToken, _provider.Password);
+            Assert.Equal(firstToken.ExpiresIn, _provider.ValidUntil.Value);
         }
 
         [Fact]
@@ -78,16 +78,16 @@ public void ShouldRefreshTokenUsingRefreshTokenWhenAvailable()
             IToken refreshedToken = newToken("the_access_token2", "the_refresh_token_2", TimeSpan.FromSeconds(60));
             _oAuth2Client.Setup(p => p.RequestToken()).Returns(firstToken);
             _provider.Refresh();
-            Assert.Equal(_provider.Password, firstToken.AccessToken);
-            Assert.Equal(_provider.ValidUntil.Value, firstToken.ExpiresIn);
-            _oAuth2Client.Reset();
-            System.Threading.Thread.Sleep(1000);
+            Assert.Equal(firstToken.AccessToken, _provider.Password);
+            Assert.Equal(firstToken.ExpiresIn, _provider.ValidUntil.Value);
 
+            _oAuth2Client.Reset();
+            Thread.Sleep(1000);
             _oAuth2Client.Setup(p => p.RefreshToken(firstToken)).Returns(refreshedToken);
             _provider.Refresh();
-            Assert.Equal(_provider.Password, refreshedToken.AccessToken);
-            Assert.Equal(_provider.ValidUntil.Value, refreshedToken.ExpiresIn);
 
+            Assert.Equal(refreshedToken.AccessToken, _provider.Password);
+            Assert.Equal(refreshedToken.ExpiresIn, _provider.ValidUntil.Value);
         }
 
         [Fact]
@@ -99,12 +99,14 @@ public void ShouldRequestTokenWhenRefreshTokenNotAvailable()
                 .Returns(firstToken)
                 .Returns(refreshedToken);
             _provider.Refresh();
-            Assert.Equal(_provider.Password, firstToken.AccessToken);
-            Assert.Equal(_provider.ValidUntil.Value, firstToken.ExpiresIn);
+
+            Assert.Equal(firstToken.AccessToken, _provider.Password);
+            Assert.Equal(firstToken.ExpiresIn, _provider.ValidUntil.Value);
 
             _provider.Refresh();
-            Assert.Equal(_provider.Password, refreshedToken.AccessToken);
-            Assert.Equal(_provider.ValidUntil.Value, refreshedToken.ExpiresIn);
+
+            Assert.Equal(refreshedToken.AccessToken, _provider.Password);
+            Assert.Equal(refreshedToken.ExpiresIn, _provider.ValidUntil.Value);
             Mock.Verify(_oAuth2Client);
         }
 
diff --git a/projects/Unit/TestTimerBasedCredentialRefresher.cs b/projects/Unit/TestTimerBasedCredentialRefresher.cs
index ce5047ba37..9dbaccbb58 100644
--- a/projects/Unit/TestTimerBasedCredentialRefresher.cs
+++ b/projects/Unit/TestTimerBasedCredentialRefresher.cs
@@ -30,7 +30,7 @@
 //---------------------------------------------------------------------------
 
 using System;
-using System.Threading.Tasks;
+using System.Threading;
 using Moq;
 using Xunit;
 
@@ -67,20 +67,21 @@ public void TestDoNotRegisterWhenHasNoExpiry()
         }
 
         [Fact]
-        public async void TestRefreshToken()
+        public void TestRefreshToken()
         {
             _credentialsProvider.Setup(p => p.ValidUntil).Returns(TimeSpan.FromSeconds(1));
             _credentialsProvider.Setup(p => p.Password).Returns("the-token").Verifiable();
             _callback.Setup(p => p.Invoke(true));
             _refresher.Register(_credentialsProvider.Object, _callback.Object);
-            await Task.Delay(1 * 1000);
+
+            Thread.Sleep(TimeSpan.FromSeconds(1));
 
             _credentialsProvider.Verify();
             _callback.Verify();
         }
 
         [Fact]
-        public async void TestRefreshTokenFailed()
+        public void TestRefreshTokenFailed()
         {
             _credentialsProvider.Setup(p => p.ValidUntil).Returns(TimeSpan.FromSeconds(1));
             _credentialsProvider.SetupSequence(p => p.Password)
@@ -88,7 +89,8 @@ public async void TestRefreshTokenFailed()
                 .Throws(new Exception());
             _callback.Setup(p => p.Invoke(false));
             _refresher.Register(_credentialsProvider.Object, _callback.Object);
-            await Task.Delay(1 * 1000);
+
+            Thread.Sleep(TimeSpan.FromSeconds(1));
 
             _credentialsProvider.Verify();
             _callback.Verify();