-
-
Notifications
You must be signed in to change notification settings - Fork 4.2k
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
fix(dropSchema): missing mssql implementation #9081
Conversation
@jerfowler The test is failing |
Drops all tables and constraints associated with a MSSQL schema Closes sequelize#9080
Added top 1/order by to the guarantee constraints are dropped first. Also added more tests to add/drop schemas with constraints.
Fixing issue with prior commit that broke platforms that don't natively support schemas
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Test code use self = this
approach, with arrow function we don't need that anymore. Please fix those tests. Otherwise this is good to go
Sorry, @sushantdhiman, I'm not sure I follow what you're asking for. Am I not supposed to use arrow functions in test code? Can you give an example of the proper approach test code should follow? |
@jerfowler With arrow function we dont need to keep reference of parent scope So keep using arrow functions but get rid of |
…riable Using arrow functions we don't need to keep track of the parent `this` scope.
Not sure why that sqlite test failed... nothing I did. Can we force a retest and see if it clears? |
Closing the Pull request and Reopening worked, all tests pass! woohoo! |
Pull Request check-list
Please make sure to review and check all of these items:
npm run test
ornpm run test-DIALECT
pass with this change (including linting)?Description of change
This implements dropSchema for MSSQL.
Closes #9080