spring-projects / spring-aot-smoke-tests

Smoke tests for Spring's AOT and native support
https://spring-team-aot-smoke-tests-dashboard.azuremicroservices.io/
51 stars 35 forks source link

Add smoketests for relevant Neo4j features. #190

Closed michael-simons closed 1 year ago

michael-simons commented 1 year ago

This adds a bunch of imperative and reactive tests for Spring Data Neo4j. All of the tests require AoT hints.

I have included a class WorkaroundsForMissingTypeHintsAndSB37574Config which includes the workaround for https://github.com/spring-projects/spring-boot/pull/37574 as well as one more missing reflection hint on SDN site of things.

I'm opening this now with the workaround included to polish out everything that doesn't fit. The workarounds can be deleted next Boot and Data releases.

We are happy to maintain this module.

wilkinsona commented 1 year ago

Wow! Thanks very much for this, @michael-simons. Should we include these smoke tests for Spring Boot 3.0, 3.1, and 3.2?

michael-simons commented 1 year ago

While I did only test them with SDN 7.2 M3 plus the workaround here, we did backport all fixes to our AoT hints back to SDN 7.0. And with the fixes in Boot back to 3.0 I think they should all work. If not, we fix the causes.

mhalbritter commented 1 year ago

Thanks for the smoke tests! I've merged them for 3.0.x, 3.1.x and 3.2.x.

michael-simons commented 1 year ago

Excellent and please ping me if anything stops working. 🙇‍♂️