Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

feat: Support aggregations by conversing expressions to specifications #32

Conversation

mhewedy
Copy link
Owner

@mhewedy mhewedy commented May 15, 2024

No description provided.

@mhewedy mhewedy merged commit dfc0f79 into master May 15, 2024
2 checks passed
Copy link

what-the-diff bot commented May 15, 2024

PR Summary

  • Improvement to Expressions.java

    • This pull request enhances the Expressions.java file by incorporating a new tool - org.springframework.data.jpa.domain.Specification. This tool will bring better efficiency in how we handle and manage data.
    • It also introduces a new method named getSpecification(). This method provides additional flexibility and extensibility to the class.
  • Increased Accessibility in ExpressionsRepositoryImpl.java

    • The accessibility of the ExpressionsSpecification class has been modified from private to public. This change enhances the usability of the class and allows it to be used across different parts of the program, increasing its value.
  • Extension of ExpressionsRepositoryImplTest.java Test Cases

    • Several tools have been imported to assist with testing, specifically those related to structuring and querying data.
    • New field EntityManager entityManager; has been integrated, which helps in interacting with the database during tests.
    • Included a new test method testAggregations_by_convert_to_specifications_InJava(), enhancing the robustness of our testing, ensuring our application continues to deliver accurate and high-quality results.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant