-
Notifications
You must be signed in to change notification settings - Fork 713
Open
Labels
backendsIssues related to all backendsIssues related to all backendsbugIncorrect behavior inside of ibisIncorrect behavior inside of ibistestsIssues or PRs related to testsIssues or PRs related to tests
Description
Is your feature request related to a problem?
It looks like we have a test for con.create_table(..., database=<str>):
ibis/ibis/backends/tests/test_client.py
Line 1606 in 1de88cb
| def test_insert_with_database_specified(con_create_database): |
but not for when database is a tuple.
Because we aren't testing it, there are bugs in implementations, such as this one in MySQL: #11976
We should add this test (marking any failing backends as xfail) and then the above PR is ready to be merged.
Describe the solution you'd like
Add another test that is very similar to the existing test_insert_with_database_specified test, except it uses a database tuple. It will have to be marked as not_yet for all the backends that don't support the two levels of heirarchy (catalog, database).
Reactions are currently unavailable
Metadata
Metadata
Assignees
Labels
backendsIssues related to all backendsIssues related to all backendsbugIncorrect behavior inside of ibisIncorrect behavior inside of ibistestsIssues or PRs related to testsIssues or PRs related to tests
Type
Projects
Status
backlog