Skip to content

Commit

Permalink
Update skip messages for unrunnable tests in ComponentReflectionTestE…
Browse files Browse the repository at this point in the history
…xtra

Signed-off-by: Jesse Whitehouse <[email protected]>
  • Loading branch information
Jesse Whitehouse committed Oct 18, 2023
1 parent 4dec379 commit 448334f
Showing 1 changed file with 12 additions and 0 deletions.
12 changes: 12 additions & 0 deletions src/databricks/sqlalchemy/test/test_suite.py
Original file line number Diff line number Diff line change
Expand Up @@ -344,6 +344,18 @@ def test_reflect_covering_index(self):
def test_reflect_expression_based_indexes(self):
pass

@pytest.mark.skip(reason="Databricks doesn't enforce String or VARCHAR length limitations.")
def test_varchar_reflection(self):
"""Even if a user specifies String(52), Databricks won't enforce that limit.
"""
pass

@pytest.mark.skip(reason="This dialect doesn't implement foreign key options checks.")
def test_get_foreign_key_options(self):
"""It's not clear from the test code what the expected output is here. Further research required.
"""
pass


class DifficultParametersTest(DifficultParametersTest):
@pytest.mark.skip(reason="Error during execution. Requires investigation.")
Expand Down

0 comments on commit 448334f

Please sign in to comment.