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 unit tests for rdbms-crud module #45

Open
derek63 opened this issue Sep 15, 2014 · 3 comments
Open

Add unit tests for rdbms-crud module #45

derek63 opened this issue Sep 15, 2014 · 3 comments

Comments

@derek63
Copy link
Member

derek63 commented Sep 15, 2014

There is a stub for tests of this module, however no actual tests currently exist. Add as necessary to validate functionality and increase code coverage

@luan-cestari
Copy link
Collaborator

I will create some test (I dont think the goal is 100% test coverage by now, but if it is, please let me know). I will make unit test cases using mock as the integration tests in the rest-module is responsible to test using the real external components

@jewzaam
Copy link
Member

jewzaam commented Sep 22, 2014

100% coverage shouldn't be a goal for anything! We are comfortable with 65% on lightblue-core and it's actually gotten above 70% now. So, given on specific policy I'd say shooting for 60%+ is reasonable in the short/mid term.

@luan-cestari
Copy link
Collaborator

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

No branches or pull requests

3 participants