patch: add multiple conditionals to prevent Bean clashes #91

[rspiegl]

Fixes #91 by only creating default Beans if nothing is provided by the user.
session.store-type was removed in Spring Boot 3 (look at spring-projects/spring-boot#27756) and so I have removed it here as well.

Adds UnitTests to make sure that creation of default Beans is backed off in case there exists a Bean.

[ghost]

👇 Click on the image for a new way to code review

Legend

[rspiegl]

One inconvenience has popped up during further testing:

2024-01-15T16:20:54.414+01:00 WARN 10928 --- [ main] trationDelegate$BeanPostProcessorChecker : Bean 'io.cloudflight.platform.spring.caching.autoconfigure.CachingAutoConfiguration$RedisConfiguration' of type [io.cloudflight.platform.spring.caching.autoconfigure.CachingAutoConfiguration$RedisConfiguration$$SpringCGLIB$$0] is not eligible for getting processed by all BeanPostProcessors (for example: not eligible for auto-proxying). Is this bean getting eagerly injected into a currently created BeanPostProcessor [meterRegistryPostProcessor]? Check the corresponding BeanPostProcessor declaration and its dependencies.
2024-01-15T16:20:54.426+01:00 WARN 10928 --- [ main] trationDelegate$BeanPostProcessorChecker : Bean 'errorHandler' of type [io.cloudflight.platform.spring.caching.RedisCacheErrorHandler] is not eligible for getting processed by all BeanPostProcessors (for example: not eligible for auto-proxying). Is this bean getting eagerly injected into a currently created BeanPostProcessor [meterRegistryPostProcessor]? Check the corresponding BeanPostProcessor declaration and its dependencies.

need to look into how to fix it

[rspiegl]

I've fixed it by adding @ConditionalOnBean(RedisConnectionFactory::class) to the CachingConfigurer Configuration. Somehow the errorHandler needs this dependency. I could have split it up further but logically it makes sense to stay there.