Move business logic

saucelabs · Mar 22, 2024 · ea67975 · ea67975
1 parent c2417d9
commit ea67975
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 1 deletion.
diff --git a/visual-dotnet/SauceLabs.Visual/VisualCheckOptions.cs b/visual-dotnet/SauceLabs.Visual/VisualCheckOptions.cs
@@ -23,5 +23,7 @@ public class VisualCheckOptions
         /// <c>TestName</c> manually set the TestName of the Test.
         /// </summary>
         public string? TestName { get; set; }
+
+        internal bool HasIncompleteTestContext() => string.IsNullOrEmpty(SuiteName) || string.IsNullOrEmpty(TestName);
     }
 }
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();
-            if (!string.IsNullOrEmpty(callerMemberName) && (string.IsNullOrEmpty(options.SuiteName) || string.IsNullOrEmpty(options.TestName)))
+            if (!string.IsNullOrEmpty(callerMemberName) && options.HasIncompleteTestContext())
             {
                 var stack = new StackTrace();
                 var frame = stack.GetFrames()?.FirstOrDefault(f => f.GetMethod().Name == callerMemberName);