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

Have one code coverage report per database instance not per connection #27

Open
jgebal opened this issue Nov 19, 2015 · 2 comments
Open

Comments

@jgebal
Copy link
Contributor

jgebal commented Nov 19, 2015

This is a feature request.
It would be nice to have the coverage reports generated and saved differently then the current approach.
Currently one report is generated per one database connection.

When analysing code coverage we usually look at coverage per instance.
There are cases when for testing single instance multiple connections need to be used.
The code coverage in the current state does not give a true value in those cases.

The existing reports could still be generated and accessed with chooser on the index.thml, just similar to the way we currently have filter combo-box for schema names.

@javornikolov
Copy link
Collaborator

Actually it might be not even a single database. One app may be based on multiple databases (one or several schemas per app in a DB).

@jgebal
Copy link
Contributor Author

jgebal commented Nov 21, 2015

Yes. I totally agree - one app can use several databases.
From the coverage report, it would be good to have one report per database instance not per database connection.
The difference is, that if tests are using several connections to one database, the coverage report on a connection level will not give a true picture of code coverage.
The true code coverage will be a coverage as it is by all connections to a particular database.

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

2 participants