Make logging less verbose, explain the project

Author: mprins

.github/workflows/maven.yml

@@ -28,4 +28,8 @@ jobs:
           with:
             maven-version: ${{ env.MAVEN_VERSION }}
         - name: 'Build with Maven'
-          run: mvn clean install
+          run: mvn clean install -DskipTests
+        - name: 'Run WFSGetFeatureUtilTest with Maven'
+          run: mvn test -Dtest=WFSGetFeatureUtilTest
+        - name: 'Run WFSGetFeatureControllerTest (Spring Boot) with Maven'
+          run: mvn test -Dtest=WFSGetFeatureControllerTest

README.md

@@ -0,0 +1,39 @@
+# entityresolver-issues
+
+Small Spring Boot 4 + GeoTools test application for reproducing and investigating WFS entity resolver behavior and
+related integration issues.
+
+The app exposes simple REST endpoints under `/wfs` that fetch one feature from external WFS services, apply an optional
+CQL filter, and return feature attributes as JSON while omitting verbose geometry values.
+
+## Endpoints
+
+- `GET|POST /wfs/provincie` - fetches a single `bestuurlijkegebieden:Provinciegebied` feature from PDOK, filtered to
+  `naam='Gelderland'`.
+- `GET|POST /wfs/bak` - fetches a single `postgis:bak` feature from the Tailormap snapshot WFS.
+
+## Tests
+
+All tests should pass when running eg. `mvn test` or `mvn install`.
+
+### POJO (Unit) tests
+
+- `WFSGetFeatureUtilTest#testWFSGetFeatureProvinciegebied` - tests the `WFSGetFeatureUtil#getOneFeature` method with the
+  PDOK WFS service.
+- `WFSGetFeatureUtilTest#testWFSGetFeatureBak` - tests the `WFSGetFeatureUtil#getOneFeature` method with the Tailormap
+  snapshot WFS service.
+
+Run separately using `mvn test -Dtest=WFSGetFeatureUtilTest`.
+
+### REST (Spring Boot) tests
+
+- `GET /wfs/test/provincie` - is tested in `WFSGetFeatureControllerTest#getProvincieFeature`.
+- `GET /wfs/test/bak` - is tested in `WFSGetFeatureControllerTest#getBakFeature`.
+
+Run separately using `mvn test -Dtest=WFSGetFeatureControllerTest`.
+
+## Run
+
+```bash
+./mvnw spring-boot:run
+```

src/main/java/nl/b3partners/entityresolverissues/WFSGetFeatureController.java

@@ -16,6 +16,11 @@
 import org.springframework.web.bind.annotation.RequestMapping;
 import org.springframework.web.bind.annotation.RestController;
 
+/**
+ * A REST controller that gets a single feature from a WFS service using a CQL filter. The feature attributes are
+ * returned as a JSON object, with the geometry attribute value omitted. The parameters for the WFS requests are
+ * hardcoded.
+ */
 @RestController
 @RequestMapping(path = "/wfs", produces = MediaType.APPLICATION_JSON_VALUE)
 public class WFSGetFeatureController {
@@ -28,6 +33,13 @@ public class WFSGetFeatureController {
             "https://snapshot.tailormap.nl/geoserver/ows?service=WFS&acceptversions=2.0.0&request=GetCapabilities";
     private static final String TM_WFS_TYPE_NAME = "postgis:bak";
 
+    /**
+     * get a "provincie" feature.
+     *
+     * @return the feature attributes as a JSON encoded map, with the geometry attribute value omitted
+     * @throws IOException when WFS request fails
+     * @throws CQLException when filter parsing fails
+     */
     @RequestMapping(
             method = {GET, POST},
             path = "/provincie")
@@ -41,6 +53,13 @@ public ResponseEntity<?> getProvincieFeature() throws IOException, CQLException
         }
     }
 
+    /**
+     * Get a "bak" feature.
+     *
+     * @return the feature attributes as a JSON encoded map, with the geometry attribute value omitted
+     * @throws IOException when WFS request fails
+     * @throws CQLException when filter parsing fails
+     */
     @RequestMapping(
             method = {GET, POST},
             path = "/bak")

src/main/java/nl/b3partners/entityresolverissues/WFSGetFeatureUtil.java

@@ -36,7 +36,8 @@ public WFSGetFeatureUtil(String capabilitiesUrl) {
 
     /**
      * Get 1 feature from a WFS service using the provided filter.
-     * @param typeName the featture type name
+     *
+     * @param typeName the feature type name
      * @param cqlFilter an optional filter, may be @{code null}
      * @return a map of the attributes, keyed with attribute name, geometry value is omitted
      * @throws CQLException when filter parsing fails

src/main/resources/application.properties

@@ -1,12 +1,16 @@
 spring.application.name=@project.artifactId@
-debug=true
+#debug=true
 
 server.address=0.0.0.0
 server.http2.enabled=true
 
 logging.file.name=target/entityresolver-issues.log
 logging.include-application-name=false
 
-# log all GeoTools
 logging.level.org.geotools=DEBUG
+logging.level.org.geotools.data.util=INFO
+logging.level.org.geotools.referencing.factory=WARN
+logging.level.org.geotools.factory=WARN
+logging.level.org.geotools.util.factory=WARN
+logging.level.org.geotools.referencing=WARN
 logging.level.nl.b3partners.entityresolverissues=DEBUG

…es/WFSGetFeatureUtilIntegrationTest.java …esolverissues/WFSGetFeatureUtilTest.javasrc/test/java/nl/b3partners/entityresolverissues/WFSGetFeatureUtilIntegrationTest.java renamed to src/test/java/nl/b3partners/entityresolverissues/WFSGetFeatureUtilTest.java src/test/java/nl/b3partners/entityresolverissues/WFSGetFeatureUtilIntegrationTest.java renamed to src/test/java/nl/b3partners/entityresolverissues/WFSGetFeatureUtilTest.java

@@ -17,7 +17,7 @@
 import org.slf4j.Logger;
 import org.slf4j.LoggerFactory;
 
-public class WFSGetFeatureUtilIntegrationTest {
+public class WFSGetFeatureUtilTest {
     private static final Logger logger =
             LoggerFactory.getLogger(MethodHandles.lookup().lookupClass());