`MixService.save(MixSaveDto)` utilise maintenant le méthode MixTypeService.createForNameAndMaterialType(String, MaterialType)

La méthode utilisait directement le constructeur de MixType, qui ne vérifie pas les conflits de nom avec les Materials existants, d'où l'erreur de contrainte unique.

Le test de la méthode est ajuster pour vérifier l'utilisation de la méthode adéquate.

src/main/kotlin/dev/fyloz/trial/colorrecipesexplorer/model/Mix.kt

@@ -26,7 +26,7 @@ data class Mix(
     @ManyToOne
     val recipe: Recipe,
 
-    @ManyToOne(cascade = [CascadeType.PERSIST])
+    @ManyToOne
     var mixType: MixType,
 
     @OneToMany(cascade = [CascadeType.ALL])

src/main/kotlin/dev/fyloz/trial/colorrecipesexplorer/service/MixService.kt

@@ -33,7 +33,7 @@ class MixServiceImpl(
     override fun save(entity: MixSaveDto): Mix {
         val recipe = recipeService.getById(entity.recipeId)
         val materialType = materialTypeService.getById(entity.materialTypeId)
-        val mixType = MixType(entity.name, materialType)
+        val mixType = mixTypeService.createForNameAndMaterialType(entity.name, materialType)
 
         var mix = save(mix(recipe = recipe, mixType = mixType))
         val mixMaterials =

src/main/resources/application-h2.properties

@@ -1,5 +1,5 @@
-#spring.datasource.url=jdbc:h2:mem:cre
-spring.datasource.url=jdbc:h2:file:./workdir/recipes
+spring.datasource.url=jdbc:h2:mem:cre
+#spring.datasource.url=jdbc:h2:file:./workdir/recipes
 spring.datasource.username=sa
 spring.datasource.password=LWK4Y7TvEbNyhu1yCoG3
 spring.h2.console.path=/dbconsole

src/test/kotlin/dev/fyloz/trial/colorrecipesexplorer/service/MixServiceTest.kt

@@ -56,6 +56,7 @@ class MixServiceTest : AbstractExternalModelServiceTest<Mix, MixSaveDto, MixUpda
             whenever(recipeService.getById(recipe.id!!)).doReturn(recipe)
             whenever(materialTypeService.getById(materialType.id!!)).doReturn(materialType)
             whenever(mixMaterialService.createFromMap(mixWithId, entitySaveDto.mixMaterials!!)).doReturn(mixMaterials)
+            whenever(mixTypeService.createForNameAndMaterialType(mixType.name, mixType.material.materialType!!)).doReturn(mixType)
             doReturn(true).whenever(service).existsById(mixWithId.id!!)
             doReturn(mixWithId).whenever(service).save(mix)
             doReturn(mixWithMaterials).whenever(service).update(mixWithMaterials)
@@ -66,6 +67,9 @@ class MixServiceTest : AbstractExternalModelServiceTest<Mix, MixSaveDto, MixUpda
             verify(service).update(mixWithMaterials)
             verify(recipeService).addMix(recipe, mix)
 
+            // Verify if this method is called instead of the MixType's constructor, which does not check if the name is already taken by a material.
+            verify(mixTypeService.createForNameAndMaterialType(mixType.name, mixType.material.materialType!!))
+
             assertEquals(mixWithMaterials, found)
         }
     }