Skip to content

Remove check on ddl triggers#31

Open
ADHUI wants to merge 1 commit intoGoEddie:masterfrom
ADHUI:Fix_Database_Triggers_Cause_Exception
Open

Remove check on ddl triggers#31
ADHUI wants to merge 1 commit intoGoEddie:masterfrom
ADHUI:Fix_Database_Triggers_Cause_Exception

Conversation

@ADHUI
Copy link

@ADHUI ADHUI commented Sep 6, 2018

Fixes #25 .

Changes proposed in this pull request:

  • Exclude objects where object name is not null. (DDL Triggers)

How to test this code:

  • Create a Database Trigger
  • Run tSQLt.RunAll when trigger is disabled.

Has been tested on (remove any that don't apply):

  • Case-sensitive SQL Server instance
  • SQL Server 2008
  • SQL Server 2008 R2
  • SQL Server 2012
  • SQL Server 2014
  • SQL Server 2016
  • Amazon RDS
  • Azure SQL DB

martinisaksen pushed a commit to martinisaksen/SQLCover that referenced this pull request Oct 11, 2019
…Hub. Once that PR is merged in this branch will use the new table to report on trigger covevrage.
@lastlink
Copy link
Contributor

I would add unit tests for this, github ations now works w/ integrated db tests, see .github/ .yml file

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.

[bug] Database Triggers Cause Exception

2 participants