Revert "Temporaly disabling ksp testing"

[velo]

This reverts commit 128a207.

[velo]

@IceBlizz6 made the issue a separated PR, any assistance you can give, would be appreciated

[IceBlizz6]

I'm looking into it.
Not currently able to recreate the issue, the project built as expected and all the tests ran successfully.

@velo
Are you able to recreate this issue on your machine?

[IceBlizz6]

Nevermind, was able to recreate it now i think.
Forgot to recreate the properties file with the new versions.

[IceBlizz6]

Seems to be solved by updating hibernate.version
7.0.0.Beta1 -> 7.0.0.Beta3

[velo]

Seems to be solved by updating hibernate.version 7.0.0.Beta1 -> 7.0.0.Beta3

Ow, that is a problem
#738

[IceBlizz6]

I could try to take a closer look if there are other solutions, but i have some doubts of whether that is even an option.
There will always be some risks when trying to use beta versions.

may i ask why we are trying to update to the hibernate beta version?

[velo]

Jpa 3.2 support... And that is forcing java 11 out, that is why I decided to start a 7 series branch

[IceBlizz6]

It looks like the error came from just trying to add a Kotlin entity class.
So this issue may not be related to KSP at all.

Configuration()
            .setProperty(AvailableSettings.JAKARTA_JDBC_DRIVER, org.h2.Driver::class.qualifiedName!!)
            .setProperty(AvailableSettings.JAKARTA_JDBC_URL, "jdbc:h2:mem:my-database;")
            .setProperty(AvailableSettings.HBM2DDL_AUTO, "create-drop")
            .setProperty(AvailableSettings.SHOW_SQL, "true")
            .addAnnotatedClass(Person::class.java) // Fails only if this line is included
            .buildSessionFactory()
            .unwrap(EntityManagerFactory::class.java)

If this is correct then anyone with a kotlin project on hibernate would be unable to use beta1 anyway.

[github-actions]

This pull request has been marked as stale due to inactivity.

[github-actions]

This pull request has been closed due to inactivity.

[github-actions]

This pull request has been marked as stale due to inactivity.

[codecov]

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 70.83%. Comparing base (8bca6ae) to head (25db085).
Report is 52 commits behind head on querydsl-7.0.

✅ All tests successful. No failed tests found.

Additional details and impacted files

@@                Coverage Diff                 @@
##             querydsl-7.0     #735      +/-   ##
==================================================
+ Coverage           61.06%   70.83%   +9.76%     
  Complexity             84       84              
==================================================
  Files                 828      854      +26     
  Lines               31905    32239     +334     
  Branches             3582     3592      +10     
==================================================
+ Hits                19482    22835    +3353     
+ Misses              11171     8176    -2995     
+ Partials             1252     1228      -24     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.