Changes to accept options as dictionary inplace of object

Author: shrinishLT

LambdaTest.Sdk.Utils/SmartUI.cs

@@ -1,3 +1,5 @@
+#nullable enable
+
 using System;
 using System.Net.Http;
 using System.Text.Json;
@@ -46,20 +48,36 @@ public static async Task<string> FetchDomSerializer()
             }
         }
 
-        public static async Task<string> PostSnapshot(DomObject snapshot, string pkg,  object options = null)
+        public static async Task<string> PostSnapshot(DomObject snapshot, string pkg,  Dictionary<string, object>? options =null)
         {
             try
             {     
-                object snapshotObject = new
+                // object snapshotObject = new
+                // {
+                //     dom = snapshot.Dom,
+                //     name = snapshot.Name,
+                //     url = snapshot.Url
+                // };
+
+                // var jsonObject = new
+                // {
+                //     snapshot = options != null ? new { dom = snapshot.Dom, name = snapshot.Name, url = snapshot.Url, options } : snapshotObject,
+                //     testType = pkg
+                // };
+
+                // var json = JsonSerializer.Serialize(jsonObject);
+
+                var snapshotData = new SnapshotData
                 {
                     dom = snapshot.Dom,
                     name = snapshot.Name,
-                    url = snapshot.Url
+                    url = snapshot.Url,
+                    options = options ?? new Dictionary<string, object>() 
                 };
 
                 var jsonObject = new
                 {
-                    snapshot = options != null ? new { dom = snapshot.Dom, name = snapshot.Name, url = snapshot.Url, options } : snapshotObject,
+                    snapshot = snapshotData,
                     testType = pkg
                 };
 
@@ -95,5 +113,13 @@ public class DomObject
             public string Url { get; set; } = string.Empty;
             public string Name { get; set; } = string.Empty;
         }
+
+        public class SnapshotData
+        {
+            public DomContent? dom { get; set; }
+            public string? name { get; set; }
+            public string? url { get; set; }
+            public Dictionary<string, object>? options { get; set; } // Nullable for handling cases with or without options
+        }
     }
 }

LambdaTest.Selenium.Driver/SmartUI.cs

@@ -12,7 +12,7 @@ public static class SmartUISnapshot
     {
          private static readonly ILogger SmartUILogger = Logger.CreateLogger("Lambdatest.Selenium.Driver");
 
-        public static async Task CaptureSnapshot(IWebDriver driver, string name, object? options = null)
+        public static async Task CaptureSnapshot(IWebDriver driver, string name, Dictionary<string, object>? options = null)
         {
             if (string.IsNullOrEmpty(name))
             {
@@ -80,7 +80,7 @@ public static async Task CaptureSnapshot(IWebDriver driver, string name, object?
                 };
 
                 var apiResponseJSON = await LambdaTest.Sdk.Utils.SmartUI.PostSnapshot(dom, "Lambdatest.Selenium.Driver", options);
-                var apiResponse = JsonSerializer.Deserialize<ApiResponse>(apiResponseJSON);
+                var apiResponse = JsonSerializer.Deserialize<ApiResponse>(apiResponseJSON, new JsonSerializerOptions { PropertyNameCaseInsensitive = true });
 
                 if (apiResponse?.Data?.Warnings != null && apiResponse.Data.Warnings.Count > 0)
                 {
@@ -106,6 +106,7 @@ private class ApiResponse
 
         private class ApiData
         {
+            public string Message { get; set; } = string.Empty;
             public List<string> Warnings { get; set; } = new List<string>();
         }