deegree · copierrj · Nov 23, 2022 · Dec 14, 2012 · Dec 21, 2012 · Jan 8, 2013
diff --git a/deegree-core/deegree-core-featureinfo/pom.xml b/deegree-core/deegree-core-featureinfo/pom.xml
@@ -48,6 +48,16 @@
       <groupId>junit</groupId>
       <artifactId>junit</artifactId>
     </dependency>
+    <dependency>
+      <groupId>org.mockito</groupId>
+      <artifactId>mockito-core</artifactId>
+    </dependency>
+    <dependency>
+      <groupId>org.mockito</groupId>
+      <artifactId>mockito-all</artifactId>
+      <version>1.10.19</version>
+      <scope>test</scope>
+    </dependency>
   </dependencies>
 
 </project>

diff --git a/deegree-core/deegree-core-featureinfo/src/main/resources/org/deegree/featureinfo/html.gfi b/deegree-core/deegree-core-featureinfo/src/main/resources/org/deegree/featureinfo/html.gfi
@@ -77,19 +77,18 @@ if(! this.<?name>_switchfunction){
     th(row, "Field").style.cssText = headerStyle
     th(row, "Value").style.cssText = headerStyle
     for(var i in <?name>){
-      row = tr(table)
-      td(row, 'GML ID').style.cssText = headerStyle
-      td(row, '<?gmlid>').style.cssText = headerStyle
       <?name>[i](table)
     }
   }
-
   <?name> = []
 }
 
 <?name>[<?name>.length] = function (table) {
   var odd = true
   var row
+  row = tr(table)
+  td(row, 'GML ID').style.cssText = headerStyle
+  td(row, '<?gmlid>').style.cssText = headerStyle
   <?property *:properties>
 }
 

diff --git a/.../src/test/java/org/deegree/featureinfo/serializing/TemplateFeatureInfoSerializerTest.java b/.../src/test/java/org/deegree/featureinfo/serializing/TemplateFeatureInfoSerializerTest.java
@@ -0,0 +1,59 @@
+package org.deegree.featureinfo.serializing;
+
+import org.deegree.feature.FeatureCollection;
+import org.deegree.featureinfo.FeatureInfoContext;
+import org.deegree.featureinfo.FeatureInfoParams;
+import org.deegree.gml.GMLInputFactory;
+import org.deegree.gml.GMLStreamReader;
+import org.deegree.gml.GMLVersion;
+import org.junit.Test;
+
+import java.io.ByteArrayOutputStream;
+import java.io.IOException;
+import java.io.OutputStream;
+import java.net.URL;
+import java.util.HashMap;
+import java.util.Map;
+
+import static org.hamcrest.CoreMatchers.endsWith;
+import static org.hamcrest.CoreMatchers.startsWith;
+import static org.junit.Assert.assertThat;
+import static org.mockito.Mockito.mock;
+import static org.mockito.Mockito.when;
+
+/**
+ * @author <a href="mailto:goltz@lat-lon.de">Lyn Goltz </a>
+ */
+public class TemplateFeatureInfoSerializerTest {
+
+    @Test
+    public void testSerialize_DefautHtmlGfi_ShouldReturnHtml()
+                    throws Exception {
+        ByteArrayOutputStream bos = new ByteArrayOutputStream();
+        TemplateFeatureInfoSerializer serializer = new TemplateFeatureInfoSerializer();
+        FeatureInfoParams params = createParams();
+        FeatureInfoContext context = mockContext( bos );
+        serializer.serialize( params, context );
+
+        String html = bos.toString().trim();
+        assertThat( html, startsWith( "<html>" ) );
+        assertThat( html, endsWith( "</html>" ) );
+    }
+
+    private FeatureInfoParams createParams()
+                    throws Exception {
+        URL resource = TemplateFeatureInfoSerializer.class.getResource( "featurecollection.gml" );
+        GMLStreamReader gmlStreamReader = GMLInputFactory.createGMLStreamReader( GMLVersion.GML_32,
+                                                                                 resource );
+        Map<String, String> nsBindings = new HashMap<>();
+        FeatureCollection col = gmlStreamReader.readFeatureCollection();
+        return new FeatureInfoParams( nsBindings, col, "text/html", true, null, null, null );
+    }
+
+    private FeatureInfoContext mockContext( OutputStream os )
+                    throws IOException {
+        FeatureInfoContext mock = mock( FeatureInfoContext.class );
+        when( mock.getOutputStream() ).thenReturn( os );
+        return mock;
+    }
+}
diff --git a/...-featureinfo/src/test/resources/org/deegree/featureinfo/serializing/featurecollection.gml b/...-featureinfo/src/test/resources/org/deegree/featureinfo/serializing/featurecollection.gml
diff --git a/deegree-tests/deegree-wmts-tests/src/test/resources/getFeatureInfo/cached_gfi.html b/deegree-tests/deegree-wmts-tests/src/test/resources/getFeatureInfo/cached_gfi.html
@@ -64,19 +64,18 @@
     th(row, "Field").style.cssText = headerStyle
     th(row, "Value").style.cssText = headerStyle
     for(var i in DominantVegetation){
-      row = tr(table)
-      td(row, 'GML ID').style.cssText = headerStyle
-      td(row, 'DOMINANTVEGETATION_329').style.cssText = headerStyle
       DominantVegetation[i](table)
     }
   }
-
   DominantVegetation = []
 }
 
 DominantVegetation[DominantVegetation.length] = function (table) {
   var odd = true
   var row
+  row = tr(table)
+  td(row, 'GML ID').style.cssText = headerStyle
+  td(row, 'DOMINANTVEGETATION_329').style.cssText = headerStyle
 
 row = tr(table)
 td(row, 'objectid').style.cssText = odd ? oddStyle : evenStyle

diff --git a/deegree-tests/deegree-wmts-tests/src/test/resources/getFeatureInfo/remotewmsfi.html b/deegree-tests/deegree-wmts-tests/src/test/resources/getFeatureInfo/remotewmsfi.html
@@ -64,19 +64,18 @@
     th(row, "Field").style.cssText = headerStyle
     th(row, "Value").style.cssText = headerStyle
     for(var i in DominantVegetation){
-      row = tr(table)
-      td(row, 'GML ID').style.cssText = headerStyle
-      td(row, 'DOMINANTVEGETATION_329').style.cssText = headerStyle
       DominantVegetation[i](table)
     }
   }
-
   DominantVegetation = []
 }
 
 DominantVegetation[DominantVegetation.length] = function (table) {
   var odd = true
   var row
+  row = tr(table)
+  td(row, 'GML ID').style.cssText = headerStyle
+  td(row, 'DOMINANTVEGETATION_329').style.cssText = headerStyle
 
 row = tr(table)
 td(row, 'objectid').style.cssText = odd ? oddStyle : evenStyle