(GH-367) SenderAuthentication.GetAllDomainsAsync must return Paginate…

…dResponseWithLinks
Jericho · Aug 4, 2024 · cae620c · cae620c
1 parent 706f0d8
commit cae620c
Show file tree

Hide file tree

Showing 3 changed files with 7 additions and 7 deletions.
diff --git a/Source/StrongGrid.IntegrationTests/Tests/SenderAuthentication.cs b/Source/StrongGrid.IntegrationTests/Tests/SenderAuthentication.cs
@@ -14,10 +14,10 @@ public async Task RunAsync(IBaseClient client, TextWriter log, CancellationToken
 			await log.WriteLineAsync("\n***** SENDER AUTHENTICATION: DOMAINS *****\n").ConfigureAwait(false);
 
 			var domains = await client.SenderAuthentication.GetAllDomainsAsync(50, 0, false, null, null, null, cancellationToken).ConfigureAwait(false);
-			await log.WriteLineAsync($"All AuthenticatedSender domains retrieved. There are {domains.Length} domains").ConfigureAwait(false);
+			await log.WriteLineAsync($"All AuthenticatedSender domains retrieved. There are {domains.Records.Length} domains").ConfigureAwait(false);
 
 			var fictitiousDomain = "StrongGridIntegrationTesting.com";
-			var cleanUpTasks = domains
+			var cleanUpTasks = domains.Records
 				.Where(d => d.Domain == fictitiousDomain)
 				.Select(async oldDomain =>
 				{

diff --git a/Source/StrongGrid/Resources/ISenderAuthentication.cs b/Source/StrongGrid/Resources/ISenderAuthentication.cs
@@ -25,9 +25,9 @@ public interface ISenderAuthentication
 		/// <param name="onBehalfOf">The user to impersonate.</param>
 		/// <param name="cancellationToken">The cancellation token.</param>
 		/// <returns>
-		/// An array of <see cref="AuthenticatedDomain" />.
+		/// The <see cref="PaginatedResponseWithLinks{AuthenticatedDomain}" />.
 		/// </returns>
-		Task<AuthenticatedDomain[]> GetAllDomainsAsync(int limit = 50, int offset = 0, bool excludeSubusers = false, string username = null, string domain = null, string onBehalfOf = null, CancellationToken cancellationToken = default);
+		Task<PaginatedResponseWithLinks<AuthenticatedDomain>> GetAllDomainsAsync(int limit = 50, int offset = 0, bool excludeSubusers = false, string username = null, string domain = null, string onBehalfOf = null, CancellationToken cancellationToken = default);
 
 		/// <summary>
 		/// Get a specific authenticated domain.

diff --git a/Source/StrongGrid/Resources/SenderAuthentication.cs b/Source/StrongGrid/Resources/SenderAuthentication.cs
@@ -41,9 +41,9 @@ internal SenderAuthentication(Pathoschild.Http.Client.IClient client)
 		/// <param name="onBehalfOf">The user to impersonate.</param>
 		/// <param name="cancellationToken">The cancellation token.</param>
 		/// <returns>
-		/// An array of <see cref="AuthenticatedDomain" />.
+		/// The <see cref="PaginatedResponseWithLinks{AuthenticatedDomain}" />.
 		/// </returns>
-		public Task<AuthenticatedDomain[]> GetAllDomainsAsync(int limit = 50, int offset = 0, bool excludeSubusers = false, string username = null, string domain = null, string onBehalfOf = null, CancellationToken cancellationToken = default)
+		public Task<PaginatedResponseWithLinks<AuthenticatedDomain>> GetAllDomainsAsync(int limit = 50, int offset = 0, bool excludeSubusers = false, string username = null, string domain = null, string onBehalfOf = null, CancellationToken cancellationToken = default)
 		{
 			return _client
 				.GetAsync($"{_endpoint}/domains")
@@ -54,7 +54,7 @@ public Task<AuthenticatedDomain[]> GetAllDomainsAsync(int limit = 50, int offset
 				.WithArgument("username", username)
 				.WithArgument("domain", domain)
 				.WithCancellationToken(cancellationToken)
-				.AsObject<AuthenticatedDomain[]>();
+				.AsPaginatedResponseWithLinks<AuthenticatedDomain>();
 		}
 
 		/// <summary>