[Java] Allow test/suitename to be optional when not using TestMetaExtension utility (#237)

Co-authored-by: Logan Graham <[email protected]>

Author: omacranger

visual-java/src/main/java/com/saucelabs/visual/graphql/CreateSnapshotFromWebDriverMutation.java

@@ -100,11 +100,11 @@ public CreateSnapshotFromWebDriverIn(
     }
 
     public void setTestName(String testName) {
-      this.testName = Optional.of(testName);
+      this.testName = Optional.ofNullable(testName);
     }
 
     public void setSuiteName(String suiteName) {
-      this.suiteName = Optional.of(suiteName);
+      this.suiteName = Optional.ofNullable(suiteName);
     }
 
     public void setCaptureDom(Boolean captureDom) {

visual-java/src/test/java/com/saucelabs/visual/integration/IntegrationBase.java

@@ -74,8 +74,8 @@ public static void tearDown() {
     driver.quit();
   }
 
-  protected void sauceVisualCheck(String name) {
-    sauceVisualCheck(name, new CheckOptions());
+  protected String sauceVisualCheck(String name) {
+    return sauceVisualCheck(name, new CheckOptions());
   }
 
   protected String sauceVisualCheck(String name, CheckOptions checkOptions) {

visual-java/src/test/java/com/saucelabs/visual/integration/WithoutTestMetaIT.java

@@ -0,0 +1,42 @@
+package com.saucelabs.visual.integration;
+
+import au.com.origin.snapshots.Expect;
+import au.com.origin.snapshots.junit5.SnapshotExtension;
+import com.saucelabs.visual.CheckOptions;
+import org.junit.jupiter.api.BeforeAll;
+import org.junit.jupiter.api.Test;
+import org.junit.jupiter.api.extension.ExtendWith;
+
+@ExtendWith({SnapshotExtension.class})
+public class WithoutTestMetaIT extends IntegrationBase {
+  Expect expect;
+
+  @BeforeAll
+  public static void login() {
+    LoginPage loginPage = new LoginPage();
+    loginPage.open();
+    loginPage.login();
+    InventoryLongPage inventoryPage = new InventoryLongPage();
+    inventoryPage.open();
+  }
+
+  @Test
+  public void checkWithoutTestMeta() {
+    String id = sauceVisualCheck("No Test Meta");
+    String result = getSnapshotResult(id);
+    expect.toMatchSnapshot(result);
+  }
+
+  @Test
+  public void checkSupplyingValuesManually() {
+    String id =
+        sauceVisualCheck(
+            "Manual Test Meta",
+            new CheckOptions.Builder()
+                .withSuiteName("Custom Suite Name")
+                .withTestName("Custom Test Name")
+                .build());
+    String result = getSnapshotResult(id);
+    expect.toMatchSnapshot(result);
+  }
+}

visual-java/src/test/java/com/saucelabs/visual/integration/__snapshots__/WithoutTestMetaIT.snap

@@ -0,0 +1,44 @@
+com.saucelabs.visual.integration.WithoutTestMetaIT.checkSupplyingValuesManually=[
+{
+  "snapshot" : {
+    "name" : "Manual Test Meta (Sauce)",
+    "suiteName" : "Custom Suite Name",
+    "testName" : "Custom Test Name",
+    "operatingSystem" : "WINDOWS",
+    "operatingSystemVersion" : "10",
+    "device" : "Desktop (1384x1007)",
+    "devicePixelRatio" : 1,
+    "hasDom" : true,
+    "diffs" : {
+      "nodes" : [ {
+        "status" : "EQUAL",
+        "diffClusters" : [ ]
+      } ]
+    },
+    "ignoreRegions" : [ ]
+  }
+}
+]
+
+
+com.saucelabs.visual.integration.WithoutTestMetaIT.checkWithoutTestMeta=[
+{
+  "snapshot" : {
+    "name" : "No Test Meta (Sauce)",
+    "suiteName" : null,
+    "testName" : null,
+    "operatingSystem" : "WINDOWS",
+    "operatingSystemVersion" : "10",
+    "device" : "Desktop (1384x1007)",
+    "devicePixelRatio" : 1,
+    "hasDom" : true,
+    "diffs" : {
+      "nodes" : [ {
+        "status" : "EQUAL",
+        "diffClusters" : [ ]
+      } ]
+    },
+    "ignoreRegions" : [ ]
+  }
+}
+]