opendev/gerrit
Code Issues Proposed changes

Clean up Elasticsearch test build rules

Browse Source 
- Add "query" in the name of the query test rules.

- Remove unnecessary list comprehension around non-query test
  rule definition.

Change-Id: I91db0e6fda85b11484acfb8fe8c9c6e9840b22d4
This commit is contained in:
David Pursehouse
2018-07-11 19:58:59 +09:00
parent c1656bd8d2
commit eff2c8726d
1 changed files with 5 additions and 5 deletions
Download Patch File Download Diff File Expand all files Collapse all files

10
javatests/com/google/gerrit/elasticsearch/BUILD
View File

@@ -52,7 +52,7 @@ ELASTICSEARCH_TAGS = [
]
[junit_tests(
name = "elasticsearch_%ss_test" % name,
name = "elasticsearch_query_%ss_test" % name,
size = "large",
srcs = [src],
tags = ELASTICSEARCH_TAGS,
@@ -60,7 +60,7 @@ ELASTICSEARCH_TAGS = [
) for name, src in ELASTICSEARCH_TESTS.items()]
[junit_tests(
name = "elasticsearch_%ss_test_V5" % name,
name = "elasticsearch_query_%ss_test_V5" % name,
size = "large",
srcs = [src],
tags = ELASTICSEARCH_TAGS,
@@ -68,14 +68,14 @@ ELASTICSEARCH_TAGS = [
) for name, src in ELASTICSEARCH_TESTS_V5.items()]
[junit_tests(
name = "elasticsearch_%ss_test_V6" % name,
name = "elasticsearch_query_%ss_test_V6" % name,
size = "large",
srcs = [src],
tags = ELASTICSEARCH_TAGS + ["flaky"],
deps = ELASTICSEARCH_DEPS + [QUERY_TESTS_DEP % name],
) for name, src in ELASTICSEARCH_TESTS_V6.items()]
[junit_tests(
junit_tests(
name = "elasticsearch_tests",
size = "small",
srcs = glob(
@@ -90,4 +90,4 @@ ELASTICSEARCH_TAGS = [
"//lib/jgit/org.eclipse.jgit:jgit",
"//lib/truth",
],
)]
)