Skip to content

Commit 0a73b64

Browse files
Added extra comments about "Eyes open"
1 parent c79ee0c commit 0a73b64

File tree

1 file changed

+19
-5
lines changed

1 file changed

+19
-5
lines changed

src/test/java/com/applitools/example/AcmeBankTests.java

Lines changed: 19 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -110,10 +110,24 @@ public void openBrowserAndEyes(TestInfo testInfo) {
110110
// Open Eyes to start visual testing.
111111
// It is a recommended practice to set all four inputs:
112112
eyes.open(
113-
page, // Page object to "watch"
114-
"ACME Bank Web App", // The name of the app under test
115-
testInfo.getDisplayName(), // The name of the test case
116-
new RectangleSize(1024, 768)); // The viewport size for the local browser
113+
114+
// The page to "watch".
115+
page,
116+
117+
// The name of the application under test.
118+
// All tests for the same app should share the same app name.
119+
// Set this name wisely: Applitools features rely on a shared app name across tests.
120+
"ACME Bank Web App",
121+
122+
// The name of the test case for the given application.
123+
// Additional unique characteristics of the test may also be specified as part of the test name,
124+
// such as localization information ("Home Page - EN") or different user permissions ("Login by admin").
125+
testInfo.getDisplayName(),
126+
127+
// The viewport size for the local browser.
128+
// Eyes will resize the web browser to match the requested viewport size.
129+
// This parameter is optional but encouraged in order to produce consistent results.
130+
new RectangleSize(1024, 768));
117131
}
118132

119133
@Test
@@ -168,4 +182,4 @@ public static void printResults() {
168182
TestResultsSummary allTestResults = runner.getAllTestResults();
169183
System.out.println(allTestResults);
170184
}
171-
}
185+
}

0 commit comments

Comments
 (0)