Rename

saucelabs · Mar 22, 2024 · 99f87b6 · 99f87b6
1 parent 61ae201
commit 99f87b6
Show file tree

Hide file tree

Showing 2 changed files with 9 additions and 7 deletions.
diff --git a/visual-dotnet/SauceLabs.Visual/VisualCheckOptions.cs b/visual-dotnet/SauceLabs.Visual/VisualCheckOptions.cs
@@ -28,15 +28,17 @@ public class VisualCheckOptions
 
         private bool HasIncompleteTestContext() => string.IsNullOrEmpty(SuiteName) || string.IsNullOrEmpty(TestName);
 
-        internal void PopulateTestContext(string callerMemberName, string? previousSuiteName)
+        internal void EnsureTestContextIsPopulated(string callerMemberName, string? previousSuiteName)
         {
-            if (!string.IsNullOrEmpty(callerMemberName) && HasIncompleteTestContext())
+            if (string.IsNullOrEmpty(callerMemberName) || HasIncompleteTestContext())
             {
-                var stack = new StackTrace();
-                var frame = stack.GetFrames()?.FirstOrDefault(f => f.GetMethod().Name == callerMemberName);
-                SuiteName ??= frame?.GetMethod().DeclaringType?.FullName ?? previousSuiteName;
-                TestName ??= callerMemberName;
+                return;
             }
+
+            var stack = new StackTrace();
+            var frame = stack.GetFrames()?.FirstOrDefault(f => f.GetMethod().Name == callerMemberName);
+            SuiteName ??= frame?.GetMethod().DeclaringType?.FullName ?? previousSuiteName;
+            TestName ??= callerMemberName;
         }
     }
 }
diff --git a/visual-dotnet/SauceLabs.Visual/VisualClient.cs b/visual-dotnet/SauceLabs.Visual/VisualClient.cs
@@ -229,7 +229,7 @@ public Task<string> VisualCheck(string name, VisualCheckOptions? options = null,
             [CallerMemberName] string callerMemberName = "")
         {
             options ??= new VisualCheckOptions();
-            options.PopulateTestContext(callerMemberName, _previousSuiteName);
+            options.EnsureTestContextIsPopulated(callerMemberName, _previousSuiteName);
             _previousSuiteName = options.SuiteName;
             return VisualCheckAsync(name, options);
         }