on-demand struct fields solves two problems seen here

vespa-engine · Mar 11, 2022 · 0f06668 · 0f06668
1 parent 5fe393b
commit 0f06668
Showing 1 changed file with 0 additions and 7 deletions.
diff --git a/...odel/src/test/java/com/yahoo/searchdefinition/derived/MatchSettingsResolvingTestCase.java b/...odel/src/test/java/com/yahoo/searchdefinition/derived/MatchSettingsResolvingTestCase.java
@@ -54,12 +54,8 @@ public void testMapDefaults() throws IOException, ParseException {
 
     @Test
     public void testMapWithStructSettings() throws IOException, ParseException {
-        /*
-         * does not work
-         * does not pick up settings from struct declaration
         assertCorrectDeriving("matchsettings_map_wss",
                               new TestProperties().setExperimentalSdParsing(false));
-        */
         assertCorrectDeriving("matchsettings_map_wss",
                               new TestProperties().setExperimentalSdParsing(true));
     }
@@ -74,11 +70,8 @@ public void testMapWithFieldSettings() throws IOException, ParseException {
 
     @Test
     public void testMapAfter() throws IOException, ParseException {
-        /* fails with:
-        java.lang.IllegalArgumentException: Could not find struct 'elem'.
         assertCorrectDeriving("matchsettings_map_after",
                               new TestProperties().setExperimentalSdParsing(false));
-        */
         assertCorrectDeriving("matchsettings_map_after",
                               new TestProperties().setExperimentalSdParsing(true));
     }