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

Test case fixes #28

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

Conversation

mkilstein
Copy link

No description provided.

Fixed transaction rollback handling. Modified the setup to allow per-test setup rather than the per-class (global) setup.
Added ability to override the schema creation due to how MSBuild in SSDT handles Create Schema for the test classes. Setting @IsMSBuild = 1 will override the dropping the schema
Incorporate all of the changes from TestCasesFixes  and ApplyUniqueIndexes into the single deployment script
Unique index (filtered/unfiltered)
Set transaction isolation level snapshot handling (add /***SNAPSHOT***/ to the test stored proc)
@mkilstein
Copy link
Author

Added missing functionality for unique indexes (filtered/unfiltered).
Added handling for transaction isolation level snapshot.
Added handling for /SNAPSHOT/ in test procedure to trigger snapshot isolation setting prior to 1st transaction
Added @IsMSBuild for test class creation to enable building tests in VS.net

Added @clone = 1 parameter to tSQLt.FakeTable, which allows creating exact copy of table+indexes+constraints during create table as a single statement. Useful in SNAPSHOT isolation where ALTER table is not allowed when transaction started
Modified handling to search for "test" and "setup" anywhere in the SP name.
Modified ExpectException and ExpectNoException handling to leverage SQL extended properties on test SP's if no "setup" procedure was executed to populate the #ExpectException
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.

1 participant