Fail rest tests on yaml files (#24740)

We've switched to supporting only `yml` files but anyone who didn't
notice will commit a `yaml` file which won't be executed
which is bad because it is easy not to notice. The test to catch this is
simple enough that I think it is worth adding just to warn folks about
their mistake.
This commit is contained in:
Nik Everett 2017-05-17 10:24:57 -04:00 committed by GitHub
parent f56d8676cf
commit 0189a65e6b
2 changed files with 2 additions and 0 deletions

View File

@ -200,6 +200,8 @@ public abstract class ESClientYamlSuiteTestCase extends ESRestTestCase {
Files.walk(path).forEach(file -> { Files.walk(path).forEach(file -> {
if (file.toString().endsWith(".yml")) { if (file.toString().endsWith(".yml")) {
addSuite(root, file, files); addSuite(root, file, files);
} else if (file.toString().endsWith(".yaml")) {
throw new IllegalArgumentException("yaml files are no longer supported: " + file);
} }
}); });
} else { } else {