Skip to content

Commit

Permalink
feat: fix parameter converter
Browse files Browse the repository at this point in the history
Signed-off-by: Otavio Santana <otaviopolianasantana@gmail.com>
  • Loading branch information
otaviojava committed Mar 16, 2024
1 parent a83e758 commit 75ffa72
Show file tree
Hide file tree
Showing 2 changed files with 7 additions and 7 deletions.
Expand Up @@ -88,7 +88,7 @@ abstract void convert(EntityConverter converter,
static ParameterConverter of(ParameterMetaData parameter, EntitiesMetadata entities) {
return switch (parameter.mappingType()) {
case COLLECTION -> validateCollection(parameter, entities);
case ENTITY -> ENTITY;
case ENTITY, EMBEDDED_GROUP -> ENTITY;
default -> DEFAULT;
};
}
Expand Down
Expand Up @@ -240,13 +240,13 @@ void shouldConvertGroupEmbeddable(){
entity.add("factory", List.of(Element.of("name", "Napa Valley Factory"),
Element.of("location", "Napa Valley")));

Beer wine = converter.toEntity(entity);
Beer beer = converter.toEntity(entity);

SoftAssertions.assertSoftly(soft ->{
var factory = wine.factory();
soft.assertThat(wine).isNotNull();
soft.assertThat(wine.id()).isEqualTo("id");
soft.assertThat(wine.name()).isEqualTo("Vin Blanc");
var factory = beer.factory();
soft.assertThat(beer).isNotNull();
soft.assertThat(beer.id()).isEqualTo("id");
soft.assertThat(beer.name()).isEqualTo("Vin Blanc");
soft.assertThat(factory).isNotNull();
soft.assertThat(factory.name()).isEqualTo("Napa Valley Factory");
soft.assertThat(factory.location()).isEqualTo("Napa Valley");
Expand All @@ -263,7 +263,7 @@ void shouldConvertGroupEmbeddableToCommunication(){

SoftAssertions.assertSoftly(soft ->{
soft.assertThat(communication).isNotNull();
soft.assertThat(communication.name()).isEqualTo("Wine");
soft.assertThat(communication.name()).isEqualTo("Beer");
soft.assertThat(communication.find("_id").orElseThrow().get()).isEqualTo("id");
soft.assertThat(communication.find("name").orElseThrow().get()).isEqualTo("Vin Blanc");
communication.find("factory").ifPresent(e -> {
Expand Down

0 comments on commit 75ffa72

Please sign in to comment.