Add wrapper objects to builds and sessions

Add an internal wrapper object for builds and sessions to fix incorrect responses where all the properties had default values.

Author: martincostello

src/MartinCostello.BrowserStack.Automate/BrowserStackAutomateClient.cs

@@ -329,7 +329,11 @@ public virtual async Task<ICollection<Build>> GetBuildsAsync(
                 using (var response = await Client.SendAsync(request, cancellationToken).ConfigureAwait(false))
                 {
                     await EnsureSuccessAsync(response).ConfigureAwait(false);
-                    return await DeserializeAsync<List<Build>>(response).ConfigureAwait(false);
+                    var builds = await DeserializeAsync<List<AutomationBuild>>(response).ConfigureAwait(false);
+
+                    return builds
+                        .Select((p) => p.Build)
+                        .ToList();
                 }
             }
         }
@@ -403,7 +407,9 @@ public virtual async Task<SessionDetail> GetSessionAsync(string sessionId, Cance
                 using (var response = await Client.SendAsync(request, cancellationToken).ConfigureAwait(false))
                 {
                     await EnsureSuccessAsync(response).ConfigureAwait(false);
-                    return await DeserializeAsync<SessionDetail>(response).ConfigureAwait(false);
+                    var result = await DeserializeAsync<AutomationSessionDetail>(response).ConfigureAwait(false);
+
+                    return result?.SessionDetail;
                 }
             }
         }
@@ -523,7 +529,11 @@ public virtual async Task<ICollection<Session>> GetSessionsAsync(
                 using (var response = await Client.SendAsync(request, cancellationToken).ConfigureAwait(false))
                 {
                     await EnsureSuccessAsync(response).ConfigureAwait(false);
-                    return await DeserializeAsync<List<Session>>(response).ConfigureAwait(false);
+                    var sessions = await DeserializeAsync<List<AutomationSession>>(response).ConfigureAwait(false);
+
+                    return sessions
+                        .Select((p) => p.Session)
+                        .ToList();
                 }
             }
         }
@@ -708,7 +718,9 @@ public virtual async Task<Session> SetSessionStatusAsync(
                 using (var response = await Client.SendAsync(request, cancellationToken).ConfigureAwait(false))
                 {
                     await EnsureSuccessAsync(response).ConfigureAwait(false);
-                    return await DeserializeAsync<Session>(response).ConfigureAwait(false);
+                    var session = await DeserializeAsync<AutomationSession>(response).ConfigureAwait(false);
+
+                    return session?.Session;
                 }
             }
         }
@@ -942,5 +954,41 @@ private async Task<T> SetNameAsync<T>(
                 }
             }
         }
+
+        /// <summary>
+        /// A class representing a BrowserStack Automate build. This class cannot be inherited.
+        /// </summary>
+        private sealed class AutomationBuild
+        {
+            /// <summary>
+            /// Gets or sets the build.
+            /// </summary>
+            [JsonProperty("automation_build")]
+            public Build Build { get; set; }
+        }
+
+        /// <summary>
+        /// A class representing a BrowserStack Automate session. This class cannot be inherited.
+        /// </summary>
+        private sealed class AutomationSession
+        {
+            /// <summary>
+            /// Gets or sets the session.
+            /// </summary>
+            [JsonProperty("automation_session")]
+            public Session Session { get; set; }
+        }
+
+        /// <summary>
+        /// A class representing a BrowserStack Automate session's details. This class cannot be inherited.
+        /// </summary>
+        private sealed class AutomationSessionDetail
+        {
+            /// <summary>
+            /// Gets or sets the session.
+            /// </summary>
+            [JsonProperty("automation_session")]
+            public SessionDetail SessionDetail { get; set; }
+        }
     }
 }