Skip to content

SQL Cover help #31

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

Open
wants to merge 6 commits into
base: main
Choose a base branch
from

Conversation

martinisaksen
Copy link

Added new table and procs to keep track of trigger object ids created during testing. This is to help SQL Cover report on trigger coverage.

@lizbaron
Copy link
Contributor

@martinisaksen, do you still need this? Please let us know!

@martinisaksen
Copy link
Author

Hi @lizbaron. I've been using my own version so I'm not sure if the current version of tSQLt supports code coverage for triggers. The original issue was that SQL Cover had no way of tying temp triggers created during tested back to the original trigger. This would leave triggers with 0% code coverage all the time.

So I believe this is still needed.

@sgtwilko
Copy link

Hiya @lizbaron
We're using Redgate's SQL Test and we'd love to be able to have code coverage for triggers.

Thanks!

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.

4 participants