Merge pull request #3071 from DevChu/refine

refine test case for F96_ZK_4330_2Test
zkoss · Oct 5, 2023 · 44dac5b · 44dac5b
2 parents 011040e + 3c00910
commit 44dac5b
Show file tree

Hide file tree

Showing 4 changed files with 8 additions and 45 deletions.
diff --git a/zktest/src/main/webapp/test2/config.properties b/zktest/src/main/webapp/test2/config.properties
@@ -2012,7 +2012,7 @@ B70-ZK-2992.zul=A,E,Grid,Scrollbar,PE,iPad
 ##zats##B80-ZK-1696-tree-2.zul=A,H,Paging,Paginal,Pageable,rendering
 ##zats##B80-ZK-2760.zul=B,M,ZHTML,XHTML,input,Parser
 ##zats##B80-ZK-2912.zul=B,E,unless,EL,EL3.0,Label
-##zats##B80-ZK-2888.zul=B,E,Javascript,UserAgent,iPhone,Chrome
+##ztl##B80-ZK-2888.zul=B,E,Javascript,UserAgent,iPhone,Chrome
 ##Manually##B80-ZK-2872.zul=A,E,Dropupload,IE11
 ##zats##B80-ZK-2874.zul=B,E,Bookmark,Redirect
 ##zats##B80-ZK-2911.zul=A,E,Custom-attributes,Page,EL

diff --git a/zktest/src/test/java/org/zkoss/zktest/zats/test2/B80_ZK_2888Test.java b/zktest/src/test/java/org/zkoss/zktest/zats/test2/B80_ZK_2888Test.java
diff --git a/zktest/src/test/java/org/zkoss/zktest/zats/test2/F86_ZK_4184Test.java b/zktest/src/test/java/org/zkoss/zktest/zats/test2/F86_ZK_4184Test.java
@@ -32,6 +32,7 @@ public void test() {
 		testUpdate();
 		// Test 4 more times
 		for (int i = 0; i < 4; i++) {
+			System.out.println(">>>> test -> " + (i + 1));
 			closeAuAlert();
 			testUpdate();
 		}
@@ -45,7 +46,9 @@ private void testUpdate() {
 
 	private void closeAuAlert() {
 		JQuery $aualert = jq("$aualert");
+		System.out.println(">>>> closeAuAlert is called!");
 		if ($aualert.isVisible()) {
+			System.out.println(">>>> closeAuAlert: try to close");
 			click($aualert.find("@button"));
 			waitResponse();
 		}

diff --git a/zktest/src/test/java/org/zkoss/zktest/zats/test2/F96_ZK_4330_2Test.java b/zktest/src/test/java/org/zkoss/zktest/zats/test2/F96_ZK_4330_2Test.java
@@ -36,9 +36,10 @@ public void test() {
 		waitResponse();
 		click(selAllBtn);
 		click(selCountBtn);
-		Assertions.assertFalse(tree1.find(".z-treecol-checkable").hasClass("z-treecol-checked"));
-		Assertions.assertFalse(tree2.find(".z-treecol-checkable").hasClass("z-treecol-checked"));
-		Assertions.assertFalse(tree3.find(".z-treecol-checkable").hasClass("z-treecol-checked"));
+		waitResponse();
+		Assertions.assertTrue(tree1.find(".z-treecol-checkable").hasClass("z-treecol-checked"));
+		Assertions.assertTrue(tree2.find(".z-treecol-checkable").hasClass("z-treecol-checked"));
+		Assertions.assertTrue(tree3.find(".z-treecol-checkable").hasClass("z-treecol-checked"));
 		click(jq("@button:eq(1)"));
 		click(jq("@button:eq(5)"));
 		click(selAllBtn);