BazelConfigurationTests.groovy: use dummy patterns
Use visibly fake target patterns in this file, to
make it obvious that the contents of the JSON data
don't have to be real.
The patterns looked real and I thought this was an
actual configuration file and it was broken...
Closes https://github.com/bazelbuild/continuous-integration/issues/159
Change-Id: If77fb86e3718d0129a7fa35e06eaa9d774913539
diff --git a/jenkins/lib/tests/build/bazel/ci/BazelConfigurationTests.groovy b/jenkins/lib/tests/build/bazel/ci/BazelConfigurationTests.groovy
index 8e153fb..f87a0d0 100644
--- a/jenkins/lib/tests/build/bazel/ci/BazelConfigurationTests.groovy
+++ b/jenkins/lib/tests/build/bazel/ci/BazelConfigurationTests.groovy
@@ -18,6 +18,10 @@
/** Tests for {@link BazelConguration} */
class BazelConfigurationTests {
+
+ // The contents of this JSON string don't matter, i.e. it doesn't have to
+ // refer to existing target patterns for example.
+ // The test only asserts the structure of the JSON.
static final String JSON_TEST = '''
// This is a test
@@ -68,9 +72,9 @@
],
"test_opts": ["-k", "--build_tests_only"],
"tests": [
- "//scripts/...",
- "//src/...",
- "//third_party/ijar/..."
+ "//dummy_path1/...",
+ "//dummy_path2/...",
+ "//dummy/path3/..."
],
"targets": []
}
@@ -90,9 +94,9 @@
"parameters": {
"test_opts": ["-k", "--build_tests_only"],
"tests": [
- "//src/test/java/...",
- "//src/test/cpp/...",
- "//src/test/naive:all_tests"
+ "//some/dummy/java_test/...",
+ "//some/dummy/cpp_test/...",
+ "//some/dummy/native_test:all_tests"
],
"targets": ["//src:bazel"]
}
@@ -107,7 +111,7 @@
}],
"parameters": {
"test_opts": ["-k", "--build_tests_only"],
- "tests": ["//src/tst/shell/bazel:bazel_windows_example_test"],
+ "tests": ["//dummy:test"],
"targets": []
}
}
