Skip to content

Commit e701dc9

Browse files
committed
Disable tests taged as 'ChromeUITest' when tests are run through run-tests.py
1 parent c2f6a6d commit e701dc9

File tree

2 files changed

+7
-0
lines changed

2 files changed

+7
-0
lines changed

dev/run-tests.py

Lines changed: 5 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -33,6 +33,9 @@
3333
from sparktestsupport.toposort import toposort_flatten
3434
import sparktestsupport.modules as modules
3535

36+
always_excluded_tags = [
37+
"org.apache.spark.tags.ChromeUITest"
38+
]
3639

3740
# -------------------------------------------------------------------------------------------------
3841
# Functions for traversing module dependency graph
@@ -606,6 +609,8 @@ def main():
606609
print("[info] Found the following changed modules:",
607610
", ".join(x.name for x in changed_modules))
608611

612+
excluded_tags.extend(always_excluded_tags)
613+
609614
# setup environment variables
610615
# note - the 'root' module doesn't collect environment variables for all modules. Because the
611616
# environment variables should not be set if a module is not changed, even if running the 'root'

pom.xml

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -243,6 +243,7 @@
243243
things breaking.
244244
-->
245245
<spark.test.home>${session.executionRootDirectory}</spark.test.home>
246+
<spark.test.webdriver.chrome.driver></spark.test.webdriver.chrome.driver>
246247

247248
<CodeCacheSize>1g</CodeCacheSize>
248249
</properties>
@@ -2512,6 +2513,7 @@
25122513
<spark.ui.enabled>false</spark.ui.enabled>
25132514
<spark.ui.showConsoleProgress>false</spark.ui.showConsoleProgress>
25142515
<spark.unsafe.exceptionOnMemoryLeak>true</spark.unsafe.exceptionOnMemoryLeak>
2516+
<spark.test.webdriver.chrome.driver>${spark.test.webdriver.chrome.driver}</spark.test.webdriver.chrome.driver>
25152517
<!-- Needed by sql/hive tests. -->
25162518
<test.src.tables>__not_used__</test.src.tables>
25172519
</systemProperties>

0 commit comments

Comments
 (0)