[11.x] Non-default schema names

[hafezdivandari]

This PR adds tests for all schema operations with prefixed table and with custom schema name. Some schema methods were not working properly in these cases that are fixed on this PR:

Prefixed table

• Renaming a column on a prefixed table is fixed on SQL Server
• Renaming an index on a prefixed table is fixed on SQL Server
• Specifying a starting value for auto-incrementing column on prefixed table is fixed on PostgreSQL

Non-default Schema name

• Dropping a primary key on non-default schema is fixed on PostgreSQL
• Renaming a table on non-default schema is fixed on SQL Server
• Renaming a column of a table on non-default schema is fixed on SQL Server
• Renaming an index of a table on non-default schema is fixed on SQL Server

Prefixed table on non-default schema

Most of schema operations wasn't working on this case, that is fixed now. Grammar was always prefixing the first segment of passed values, that causes a problem when passing a table as schema.table and being wrapped as prefix_schema.table, this conditions is fixed by separating the logic for wrapping columns/values from wrapping tables after this PR:

• table.column being wrapped to prefix_table.column using Grammar::wrap() as before.
• schema.table being wrapped as schema.prefix_table using Grammar::wrapTable() as expected.

P.S: This is just a fix and adds support for prefixed table and using non-default schema on all cases, it's not changing behavior so we don't need to add anything to upgrade guide.

Summary of tested and fixed cases

DB	PostgreSQL	SQL Server	PostgreSQL	SQL Server	PostgreSQL	SQL Server
Feature	Schema Name		Prefix		Schema Name + Prefix
Create Table	✓	✓	✓	✓	✓ (fixed)	✓ (fixed)
Rename Table	✓	✓ (fixed)	✓	✓	✓ (fixed)	✓ (fixed)
Drop Table	✓	✓	✓	✓	✓ (fixed)	✓ (fixed)
Has Table	✓	✓ (#49965)	✓	✓	✓	✓ (#49965)
Table w/ Comment	✓	n/a	✓	n/a	✓ (fixed)	n/a
Add Column	✓	✓	✓	✓	✓ (fixed)	✓ (fixed)
Rename Column	✓ (#45258)	✓ (fixed)	✓	✓ (fixed)	✓ (fixed)	✓ (fixed)
Modify Column	✓ (#45487)	✓ (#45487)	✓	✓	✓ (fixed)	✓ (fixed)
Drop Column	✓	✓	✓	✓	✓ (fixed)	✓ (fixed)
Column w/ Default	✓	✓ (#49965)	✓	✓	✓ (fixed)	✓ (fixed)
Column w/ Comment	✓	n/a	✓	n/a	✓ (fixed)	n/a
Column w/ Starting Value	✓	n/a	✓ (fixed)	n/a	✓ (fixed)	n/a
Get Columns	✓	✓ (#49965)	✓	✓	✓	✓ (#49965)
Has Column(s)	✓	✓ (#49965)	✓	✓	✓	✓ (#49965)
Get Column Type	✓	✓ (#49965)	✓	✓	✓	✓ (#49965)
Add Index	✓	✓	✓	✓	✓ (fixed)	✓ (fixed)
Rename Index	✓	✓ (fixed)	✓	✓ (fixed)	✓ (fixed)	✓ (fixed)
Drop Primary	✓ (fixed)	✓	✓	✓	✓ (fixed)	✓ (fixed)
Drop Unique	✓	✓	✓	✓	✓ (fixed)	✓ (fixed)
Drop Index	✓	✓	✓	✓	✓ (fixed)	✓ (fixed)
Get Indexes	✓	✓ (#49965)	✓	✓	✓	✓ (#49965)
Has Index	✓	✓ (#49965)	✓	✓	✓	✓ (#49965)
Add Foreign Key	✓	✓	✓	✓	✓ (fixed)	✓ (fixed)
Drop Foreign Key	✓	✓	✓	✓	✓ (fixed)	✓ (fixed)
Get Foreign Keys	✓	✓ (#49965)	✓	✓	✓	✓ (#49965)
Has View	✓ (#49965)	✓ (#49965)	✓	✓	✓ (#49965)	✓ (#49965)

[github-actions]

Thanks for submitting a PR!

Note that draft PR's are not reviewed. If you would like a review, please mark your pull request as ready for review in the GitHub user interface.

Pull requests that are abandoned in draft may be closed due to inactivity.

[taylorotwell]

Any breaking changes that will need to be documented here?

[hafezdivandari]

@taylorotwell no, it's just a fix.

[taylorotwell]

Thanks!

[arjenvanoostrum]

Hi @hafezdivandari Thanks for your effort to now fully support schema names in SQLserver! Brings the database component to an even higher level.