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

Add integration tests #516

Open
michael-simons opened this issue Jan 17, 2024 · 2 comments
Open

Add integration tests #516

michael-simons opened this issue Jan 17, 2024 · 2 comments

Comments

@michael-simons
Copy link
Collaborator

From sql2cypher created by lukaseder: neo4j-contrib/sql2cypher#23

It's probably not the top priority, but eventually, might be reasonable to think about integration tests, e.g. using testcontainers:
https://www.testcontainers.org/modules/databases/neo4j/

For this, it might be reasonable to compare the neo4j translated behaviour e.g. with H2?

The problems that the current test suite won't cover:

  • Correct query semantics (e.g. when nesting / correlating stuff, there are always hairy edge cases)
  • Data type conversion, etc.
  • Other things?
@michael-simons
Copy link
Collaborator Author

Great idea.

Which free relational DB is best for a cleanest SQL standard? H2 / MySQL / Postgres?

@michael-simons
Copy link
Collaborator Author

I'd use H2 for this, it's become very standards compliant.

@michael-simons michael-simons transferred this issue from another repository Feb 27, 2024
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

No branches or pull requests

1 participant