Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

FIX - Jaxb configuration changed to serialize EntityUniquenessExceptionInfo #3943

Conversation

Agnul97
Copy link
Contributor

@Agnul97 Agnul97 commented Jan 19, 2024

Brief description of the PR.
Before this PR Jaxb was not correctly serializing the EntityUniquenessExceptionInfo that we insert in the response body if needed. It was, instead, serializing to a basic ExceptionInfo without the additional fields in EntityUniquenessExceptionInfo class.

Description of the solution adopted
The problem was that the EntityUniquenessExceptionInfo class was not inserted inside the configuration of jaxb so the framework was not able to perform the operation

Copy link

codecov bot commented Jan 19, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Comparison is base (1c6be69) 20.73% compared to head (40b073c) 20.73%.
Report is 14 commits behind head on develop.

Additional details and impacted files

Impacted file tree graph

@@            Coverage Diff             @@
##             develop    #3943   +/-   ##
==========================================
  Coverage      20.73%   20.73%           
  Complexity         6        6           
==========================================
  Files           1951     1951           
  Lines          41767    41767           
  Branches        3959     3959           
==========================================
  Hits            8661     8661           
  Misses         32703    32703           
  Partials         403      403           
Files Coverage Δ
...eclipse/kapua/app/api/web/JaxbContextResolver.java 77.77% <ø> (ø)

@Coduz Coduz added the Bug This is a bug or an unexpected behaviour. Fix it! label Jan 27, 2024
@Coduz Coduz merged commit 474caaf into eclipse-kapua:develop Jan 27, 2024
33 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Bug This is a bug or an unexpected behaviour. Fix it!
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants