diff options
author | Harald Musum <musum@yahooinc.com> | 2022-04-28 15:01:42 +0200 |
---|---|---|
committer | Harald Musum <musum@yahooinc.com> | 2022-04-28 15:01:42 +0200 |
commit | d5a2caa23b848a18352127b502f4817aa9bfc45d (patch) | |
tree | 1cd48b848b802bc3590d681a83c9b77cf33b7c86 /config-application-package/src/test | |
parent | 04add1191a91616173e3a84a9e69a9215a108d6d (diff) |
Validate file extension for app files, part 4
* Allow .gitignore files everywhere and .profile files in schemas/ directory
* Move exception handling and logging to config server code
Diffstat (limited to 'config-application-package/src/test')
2 files changed, 8 insertions, 8 deletions
diff --git a/config-application-package/src/test/java/com/yahoo/config/model/application/provider/FilesApplicationPackageTest.java b/config-application-package/src/test/java/com/yahoo/config/model/application/provider/FilesApplicationPackageTest.java index e3a27ecddf0..29bc2b6fbd6 100644 --- a/config-application-package/src/test/java/com/yahoo/config/model/application/provider/FilesApplicationPackageTest.java +++ b/config-application-package/src/test/java/com/yahoo/config/model/application/provider/FilesApplicationPackageTest.java @@ -144,20 +144,20 @@ public class FilesApplicationPackageTest { @Test public void testValidFileExtensions() { - File appDir = new File("src/test/resources/app-with-deployment");; + File appDir = new File("src/test/resources/app-with-deployment"); FilesApplicationPackage app = FilesApplicationPackage.fromFile(appDir); - app.validateFileExtensions(true); + app.validateFileExtensions(); } @Test public void testInvalidFileExtensions() { - File appDir = new File("src/test/resources/app-with-invalid-files-in-subdir");; + File appDir = new File("src/test/resources/app-with-invalid-files-in-subdir"); FilesApplicationPackage app = FilesApplicationPackage.fromFile(appDir); try { - app.validateFileExtensions(true); + app.validateFileExtensions(); fail("expected an exception"); } catch (IllegalArgumentException e) { - assertEquals("File in application package with unknown suffix: search/query-profiles/file-with-invalid.extension, " + + assertEquals("File in application package with unknown extension: search/query-profiles/file-with-invalid.extension, " + "please delete or move file to another directory.", e.getMessage()); } @@ -165,13 +165,13 @@ public class FilesApplicationPackageTest { @Test public void testInvalidFileExtensionInSubDirOfSubDir() { - File appDir = new File("src/test/resources/app-with-files-with-invalid-extension-in-subdir-of-subdir/");; + File appDir = new File("src/test/resources/app-with-files-with-invalid-extension-in-subdir-of-subdir/"); FilesApplicationPackage app = FilesApplicationPackage.fromFile(appDir); try { - app.validateFileExtensions(true); + app.validateFileExtensions(); fail("expected an exception"); } catch (IllegalArgumentException e) { - assertEquals("File in application package with unknown suffix: schemas/foo/bar.junk, " + + assertEquals("File in application package with unknown extension: schemas/foo/bar.junk, " + "please delete or move file to another directory.", e.getMessage()); } diff --git a/config-application-package/src/test/resources/app-with-deployment/schemas/.gitignore b/config-application-package/src/test/resources/app-with-deployment/schemas/.gitignore new file mode 100644 index 00000000000..e69de29bb2d --- /dev/null +++ b/config-application-package/src/test/resources/app-with-deployment/schemas/.gitignore |