Skip to content

Commit 2456c51

Browse files
authored
Fix Integration test failure of migration_tables (#3108)
<!-- REMOVE IRRELEVANT COMMENTS BEFORE CREATING A PULL REQUEST --> ## Changes <!-- Summary of your changes that are easy to understand. Add screenshots when necessary --> This change unskips the two integration test which are failing. The underlying issue is fixed as part of #3020 Resolves #3054 Resolves #3055 ### Functionality ### Tests <!-- How is this tested? Please see the checklist below and also describe any other relevant tests --> - [ ] added integration tests - [ ] verified on staging environment (screenshot attached)
1 parent c4df7b1 commit 2456c51

File tree

2 files changed

+0
-3
lines changed

2 files changed

+0
-3
lines changed

tests/integration/hive_metastore/test_migrate.py

Lines changed: 0 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -175,7 +175,6 @@ def test_migrate_external_table(
175175
assert migration_status[0].dst_table == src_external_table.name
176176

177177

178-
@pytest.mark.skip("https://github.com/databrickslabs/ucx/issues/3054")
179178
def test_migrate_managed_table_to_external_table_without_conversion(
180179
ws,
181180
sql_backend,
@@ -219,7 +218,6 @@ def test_migrate_managed_table_to_external_table_without_conversion(
219218
assert migration_status[0].dst_table == src_external_table.name
220219

221220

222-
@pytest.mark.skip("https://github.com/databrickslabs/ucx/issues/3055")
223221
def test_migrate_managed_table_to_external_table_with_clone(
224222
ws,
225223
sql_backend,

tests/unit/source_code/test_directfs_access.py

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -11,7 +11,6 @@
1111
DirectFsAccess,
1212
DirectFsAccessOwnership,
1313
)
14-
from databricks.labs.ucx.source_code.python.python_ast import DfsaPyCollector
1514

1615

1716
def test_crawler_appends_dfsas() -> None:

0 commit comments

Comments
 (0)