Jan

Results 60 comments of Jan

mh, so testing with Spring Boot 2.6 wil stay broken for now? @danielfernandez - maybe you can say something about that as it seems you are the relevant guy here...

I think this was fixed with #25564. You should try to reproduce with the latest version.

Probably introduced here https://github.com/fluent/helm-charts/commit/09607ddf83eec00645bcdcac2a9534c23d48ec7a#diff-06d5d393465a492f05f853f615cb0f9e1746a7981a378ff6930efd1ad94efcb1R37-R38 with #305

Anyone having any information here? Maybe @dioguerra as you seem to be the most active here?

The default JHipster Mapper Tests are also generated like this. https://github.com/jhipster/generator-jhipster/blob/e6d382426eaea8515301fb87b117210b8258c556/generators/spring-boot/templates/src/test/java/_package_/_entityPackage_/service/mapper/_entityClass_MapperTest.java.ejs#L27-L42 Since we also adjusted our mappers to make use of this as we need a bit more complex objects:...

Ok, well, for us it is more important to not duplicate mapping code and having a consistent mapping for sub-entities, so injection of an already well-defined mapper for another Entity...

Well, at least for avoiding code duplications for the MapStructExpression fields, this would be helpful. At the moment, that expression is duplicated into every mapper that accesses such a field,...

We do have the host name written into testcontainers.properties file in our classpath via `hub.image.name.prefix` property as explained in https://java.testcontainers.org/features/image_name_substitution/ ``` hub.image.name.prefix=myregistry.azurecr.io/docker.io/ ``` That's why we don't mention it explicitly...

> Hi, I think the code should be > > ```java > elasticsearchContainer = new ElasticsearchContainer( > DockerImageName.parse("myregistry.azurecr.io/docker.io/library/elasticsearch") > .asCompatibleSubstituteFor("docker.elastic.co/elasticsearch/elasticsearch") > .withTag("8.10.4") > ) > ``` Indeed, that did work...