Remove IE from default list of browsers used for java tests (#12152)

shs96c · diemol · web-flow · commit 370428cb9ec9 · 2023-06-06T05:35:03.000+02:00
Co-authored-by: Diego Molina &lt;diemol@users.noreply.github.com&gt;
diff --git a/java/private/selenium_test.bzl b/java/private/selenium_test.bzl
@@ -56,7 +56,9 @@ BROWSERS = {
     },
 }
 
-def selenium_test(name, test_class, size = "medium", browsers = BROWSERS.keys(), **kwargs):
+DEFAULT_BROWSERS = [b for b in BROWSERS.keys() if b != "ie"]
+
+def selenium_test(name, test_class, size = "medium", browsers = DEFAULT_BROWSERS, **kwargs):
     if len(browsers) == 0:
         fail("At least one browser must be specified.")
 

Original file line number	Diff line number	Diff line change
`@@ -56,7 +56,9 @@ BROWSERS = {`
`56`	`56`	`},`
`57`	`57`	`}`
`58`	`58`
`59`		`-def selenium_test(name, test_class, size = "medium", browsers = BROWSERS.keys(), **kwargs):`
	`59`	`+DEFAULT_BROWSERS = [b for b in BROWSERS.keys() if b != "ie"]`
	`60`	`+`
	`61`	`+def selenium_test(name, test_class, size = "medium", browsers = DEFAULT_BROWSERS, **kwargs):`
`60`	`62`	`if len(browsers) == 0:`
`61`	`63`	`fail("At least one browser must be specified.")`
`62`	`64`