summaryrefslogtreecommitdiffstats
path: root/config-model/src/test/java/com/yahoo/searchdefinition/SearchDefinitionsParsingTestCase.java
diff options
context:
space:
mode:
Diffstat (limited to 'config-model/src/test/java/com/yahoo/searchdefinition/SearchDefinitionsParsingTestCase.java')
-rw-r--r--config-model/src/test/java/com/yahoo/searchdefinition/SearchDefinitionsParsingTestCase.java35
1 files changed, 8 insertions, 27 deletions
diff --git a/config-model/src/test/java/com/yahoo/searchdefinition/SearchDefinitionsParsingTestCase.java b/config-model/src/test/java/com/yahoo/searchdefinition/SearchDefinitionsParsingTestCase.java
index 278471cb37a..a26154fc8da 100644
--- a/config-model/src/test/java/com/yahoo/searchdefinition/SearchDefinitionsParsingTestCase.java
+++ b/config-model/src/test/java/com/yahoo/searchdefinition/SearchDefinitionsParsingTestCase.java
@@ -73,35 +73,16 @@ public class SearchDefinitionsParsingTestCase extends SearchDefinitionTestCase {
}
}
- private static class WarningCatcher extends Handler {
- volatile boolean gotYqlWarning = false;
-
- @Override
- public void publish(LogRecord record) {
- if (record.getLevel() == Level.WARNING && record.getMessage().indexOf("YQL") >= 0) {
- gotYqlWarning = true;
+ @Test
+ public void illegalSearchDefinitionName() throws IOException, ParseException {
+ try {
+ SearchBuilder.buildFromFile("src/test/examples/invalid-name.sd");
+ fail("Name with dash passed");
+ } catch (ParseException e) {
+ if ( ! e.getMessage().contains("invalid-name")) {
+ throw e;
}
}
-
- @Override
- public void flush() {
- // intentionally left blank
- }
-
- @Override
- public void close() throws SecurityException {
- // intentionally left blank
- }
}
-
- @Test
- public void requireYqlCompatibilityIsTested() throws Exception {
- Logger log = Logger.getLogger("DeployLogger");
- WarningCatcher w = new WarningCatcher();
- log.addHandler(w);
- assertNotNull(SearchBuilder.buildFromFile("src/test/examples/simple-with-weird-name.sd"));
- log.removeHandler(w);
- assertTrue(w.gotYqlWarning);
- }
}