Skip to content

Conversation

psteinroe
Copy link
Collaborator

turns out I was so focused on the rewrite of that layer that I forgot to add even a single test for the sql fn body extraction 😢

fixes #342


let stmts = d.iter(DefaultMapper).collect::<Vec<_>>();

assert_eq!(stmts.len(), 2);
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Love it!

should we add a test asserting the content as well?
and maybe a test using the $$ notation?

Copy link
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

not really necessary imo - we can trust pg_query here

Copy link
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

will do it in a follow up

@psteinroe psteinroe merged commit d33bc4f into main Apr 19, 2025
7 checks passed
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.

Intelligence for sql function bodies
2 participants