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 better tests for new prepared_statements #60

Open
jwilberding opened this issue Jun 5, 2014 · 0 comments
Open

Add better tests for new prepared_statements #60

jwilberding opened this issue Jun 5, 2014 · 0 comments

Comments

@jwilberding
Copy link
Contributor

Right now it is hard to test the new method of specifying prepared_statements, as it requires we bootstrap a separate app that has the sqerl_rec behavior. Right now the current test basically tests that including {app, sqerl} doesn't break anything, but it isn't really testing to see if the module introspection is working, since the test modules with the sqerl_rec behavior do not have an associated app we test with.

Relevant code:
https://github.com/opscode/sqerl/blob/master/test/sqerl_rec_tests.erl#L143
https://github.com/opscode/sqerl/blob/master/test/sqerl_test_helper.erl#L19-L20

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