feat: issue 23, issue #58, issue #57 (#59)

* Fixing incorrect JPA Mapping which was failing the Postgres IT test. Closes Issue #53.

* Updating postress driver version to latest.

* Refactoring to use Jackson 3, and Jackson for XML Serialization. Closes #23

* Adding Jackson 3 deps for Jakarta XML Binding w/JAXB, Closes #23, #58

* Updating test configuration for test containers, also changed H2 to use flyway for migrations

* Removing unnecessary services and Spring profiles. Issue #57

* Removing unused profiles, update flyway config to enable migrations for tests. closes #57

* whitespace change to trigger build

* feat - fixing failing tests from adding flyway migration for H2

Author: springframeworkguru

.github/workflows/ci.yml

@@ -14,34 +14,6 @@ jobs:
   build-and-test:
     name: Build and Test All Modules
     runs-on: ubuntu-latest
-
-    services:
-      mysql:
-        image: mysql:8.0
-        env:
-          MYSQL_ROOT_PASSWORD: root
-          MYSQL_DATABASE: test_db
-        ports:
-          - 3306:3306
-        options: >-
-          --health-cmd="mysqladmin ping"
-          --health-interval=10s
-          --health-timeout=5s
-          --health-retries=3
-
-      postgres:
-        image: postgres:18
-        env:
-          POSTGRES_PASSWORD: postgres
-          POSTGRES_DB: test_db
-        ports:
-          - 5432:5432
-        options: >-
-          --health-cmd=pg_isready
-          --health-interval=10s
-          --health-timeout=5s
-          --health-retries=3
-
     steps:
       - name: Checkout code
         uses: actions/checkout@v4
@@ -75,13 +47,9 @@ jobs:
 
       - name: Run tests - openespi-common
         run: mvn test -pl openespi-common -am
-        env:
-          SPRING_PROFILES_ACTIVE: test
 
       - name: Run tests - openespi-datacustodian
         run: mvn test -pl openespi-datacustodian -am
-        env:
-          SPRING_PROFILES_ACTIVE: test
 
       # TODO: Enable when authserver implementation is complete
       # - name: Run tests - openespi-authserver
@@ -91,13 +59,9 @@ jobs:
 
       - name: Run tests - openespi-thirdparty
         run: mvn test -pl openespi-thirdparty -am
-        env:
-          SPRING_PROFILES_ACTIVE: test
 
       - name: Run integration tests with TestContainers
         run: mvn verify -Pintegration-tests -pl openespi-common,openespi-datacustodian,openespi-thirdparty
-        env:
-          SPRING_PROFILES_ACTIVE: testcontainers
 
       # JaCoCo reports are automatically generated during test phase (configured in root pom.xml)
 

openespi-common/pom.xml

@@ -300,7 +300,17 @@
             <groupId>commons-codec</groupId>
             <artifactId>commons-codec</artifactId>
         </dependency>
-        <!-- Jakarta XML Binding Dependencies -->
+        <!-- Jakarta / Jackson XML Binding Dependencies -->
+        <dependency>
+            <groupId>tools.jackson.dataformat</groupId>
+            <artifactId>jackson-dataformat-xml</artifactId>
+            <version>${jackson-bom.version}</version>
+        </dependency>
+        <dependency>
+            <groupId>tools.jackson.module</groupId>
+            <artifactId>jackson-module-jakarta-xmlbind-annotations</artifactId>
+            <version>${jackson-bom.version}</version>
+        </dependency>
         <dependency>
             <groupId>jakarta.xml.bind</groupId>
             <artifactId>jakarta.xml.bind-api</artifactId>

openespi-common/src/main/java/org/greenbuttonalliance/espi/common/domain/customer/entity/PhoneNumberEntity.java

@@ -23,7 +23,9 @@
 import lombok.Getter;
 import lombok.NoArgsConstructor;
 import lombok.Setter;
+import org.hibernate.annotations.JdbcTypeCode;
 import org.hibernate.proxy.HibernateProxy;
+import org.hibernate.type.SqlTypes;
 
 import java.util.Objects;
 import java.util.UUID;
@@ -49,7 +51,8 @@ public class PhoneNumberEntity {
      */
     @Id
     @GeneratedValue(strategy = GenerationType.UUID)
-    @Column(name = "id", nullable = false)
+    @JdbcTypeCode(SqlTypes.CHAR)
+    @Column(length = 36, columnDefinition = "char(36)", updatable = false, nullable = false)
     private UUID id;
 
     /**

openespi-common/src/main/java/org/greenbuttonalliance/espi/common/domain/customer/entity/StatementRefEntity.java

@@ -23,7 +23,9 @@
 import lombok.Getter;
 import lombok.NoArgsConstructor;
 import lombok.Setter;
+import org.hibernate.annotations.JdbcTypeCode;
 import org.hibernate.proxy.HibernateProxy;
+import org.hibernate.type.SqlTypes;
 
 import java.util.Objects;
 import java.util.UUID;
@@ -48,7 +50,8 @@ public class StatementRefEntity {
      */
     @Id
     @GeneratedValue(strategy = GenerationType.UUID)
-    @Column(name = "id", nullable = false)
+    @JdbcTypeCode(SqlTypes.CHAR)
+    @Column(length = 36, columnDefinition = "char(36)", updatable = false, nullable = false)
     private UUID id;
 
     /**

openespi-common/src/main/java/org/greenbuttonalliance/espi/common/domain/usage/AggregatedNodeRefEntity.java

@@ -23,7 +23,9 @@
 import lombok.Getter;
 import lombok.NoArgsConstructor;
 import lombok.Setter;
+import org.hibernate.annotations.JdbcTypeCode;
 import org.hibernate.proxy.HibernateProxy;
+import org.hibernate.type.SqlTypes;
 
 import java.util.Objects;
 import java.util.UUID;
@@ -49,7 +51,8 @@ public class AggregatedNodeRefEntity {
      */
     @Id
     @GeneratedValue(strategy = GenerationType.UUID)
-    @Column(name = "id", nullable = false)
+    @JdbcTypeCode(SqlTypes.CHAR)
+    @Column(length = 36, columnDefinition = "char(36)", updatable = false, nullable = false)
     private UUID id;
 
     /**

openespi-common/src/main/java/org/greenbuttonalliance/espi/common/domain/usage/BatchListEntity.java

@@ -25,7 +25,9 @@
 import lombok.Getter;
 import lombok.NoArgsConstructor;
 import lombok.Setter;
+import org.hibernate.annotations.JdbcTypeCode;
 import org.hibernate.proxy.HibernateProxy;
+import org.hibernate.type.SqlTypes;
 
 import java.net.URI;
 import java.net.URISyntaxException;
@@ -59,7 +61,8 @@ public class BatchListEntity {
      */
     @Id
     @GeneratedValue(strategy = GenerationType.UUID)
-    @Column(name = "id", nullable = false)
+    @JdbcTypeCode(SqlTypes.CHAR)
+    @Column(length = 36, columnDefinition = "char(36)", updatable = false, nullable = false)
     private UUID id;
 
     /**

openespi-common/src/main/java/org/greenbuttonalliance/espi/common/domain/usage/IntervalReadingEntity.java

@@ -25,7 +25,9 @@
 import lombok.Setter;
 import org.greenbuttonalliance.espi.common.domain.common.DateTimeInterval;
 import org.hibernate.annotations.BatchSize;
+import org.hibernate.annotations.JdbcTypeCode;
 import org.hibernate.proxy.HibernateProxy;
+import org.hibernate.type.SqlTypes;
 
 import java.util.ArrayList;
 import java.util.List;
@@ -54,7 +56,8 @@ public class IntervalReadingEntity {
      */
     @Id
     @GeneratedValue(strategy = GenerationType.UUID)
-    @Column(name = "id", nullable = false)
+    @JdbcTypeCode(SqlTypes.CHAR)
+    @Column(length = 36, columnDefinition = "char(36)", updatable = false, nullable = false)
     private UUID id;
 
     /**

openespi-common/src/main/java/org/greenbuttonalliance/espi/common/domain/usage/LineItemEntity.java

@@ -25,7 +25,9 @@
 import lombok.Getter;
 import lombok.NoArgsConstructor;
 import lombok.Setter;
+import org.hibernate.annotations.JdbcTypeCode;
 import org.hibernate.proxy.HibernateProxy;
+import org.hibernate.type.SqlTypes;
 
 import java.math.BigDecimal;
 import java.math.RoundingMode;
@@ -62,7 +64,8 @@ public class LineItemEntity {
      */
     @Id
     @GeneratedValue(strategy = GenerationType.UUID)
-    @Column(name = "id", nullable = false)
+    @JdbcTypeCode(SqlTypes.CHAR)
+    @Column(length = 36, columnDefinition = "char(36)", updatable = false, nullable = false)
     private UUID id;
 
     /**

openespi-common/src/main/java/org/greenbuttonalliance/espi/common/domain/usage/ReadingQualityEntity.java

@@ -25,7 +25,9 @@
 import lombok.Getter;
 import lombok.NoArgsConstructor;
 import lombok.Setter;
+import org.hibernate.annotations.JdbcTypeCode;
 import org.hibernate.proxy.HibernateProxy;
+import org.hibernate.type.SqlTypes;
 
 import java.util.Objects;
 import java.util.UUID;
@@ -57,7 +59,8 @@ public class ReadingQualityEntity {
      */
     @Id
     @GeneratedValue(strategy = GenerationType.UUID)
-    @Column(name = "id", nullable = false)
+    @JdbcTypeCode(SqlTypes.CHAR)
+    @Column(length = 36, columnDefinition = "char(36)", updatable = false, nullable = false)
     private UUID id;
 
     // Quality constants based on common industry standards

openespi-common/src/main/java/org/greenbuttonalliance/espi/common/dto/atom/AtomContentDto.java

@@ -19,21 +19,41 @@
 
 package org.greenbuttonalliance.espi.common.dto.atom;
 
+import com.fasterxml.jackson.annotation.*;
 import jakarta.xml.bind.annotation.*;
+import org.greenbuttonalliance.espi.common.dto.usage.MeterReadingDto;
+import org.greenbuttonalliance.espi.common.dto.usage.ReadingTypeDto;
+import org.greenbuttonalliance.espi.common.dto.usage.UsagePointDto;
+import tools.jackson.dataformat.xml.annotation.JacksonXmlProperty;
 
 /**
  * Atom Content DTO record for content within Atom entries.
  * 
  * Represents the content section of an Atom entry containing the actual
  * Green Button resource data (Customer, UsagePoint, MeterReading, etc.).
+ *
+ * @deprecated AtomContentDto no longer used. Payload moved to AtomEntryDto
  */
+@Deprecated
 @XmlAccessorType(XmlAccessType.FIELD)
 @XmlType(name = "AtomContent", namespace = "http://www.w3.org/2005/Atom")
 public record AtomContentDto(
 
     @XmlAttribute(name = "type")
     String type,
-    
+
+    @JsonProperty("resource")
+    @JacksonXmlProperty(namespace = "http://naesb.org/espi")
+
+    @XmlElementWrapper(name = "resource", namespace = "http://naesb.org/espi")
+    @JsonTypeInfo(use = JsonTypeInfo.Id.NAME,
+            include = JsonTypeInfo.As.WRAPPER_OBJECT,
+            property = "type")
+    @JsonSubTypes({
+            @JsonSubTypes.Type(value = UsagePointDto.class, name = "UsagePoint"),
+            @JsonSubTypes.Type(value = MeterReadingDto.class, name = "MeterReading"),
+            @JsonSubTypes.Type(value = ReadingTypeDto.class, name = "ReadingType")
+    })
     @XmlAnyElement(lax = true)
     Object resource
 ) {