Skip to content

Commit 43284ca

Browse files
authored
Fix failed integration test test_migrate_external_tables_with_spn_azure (#1496)
## Changes <!-- Summary of your changes that are easy to understand. Add screenshots when necessary --> Not sure why didn't the integration test run in PR #1413 catch the failure of this test `test_migrate_external_tables_with_spn_azure`, but we need this fix to avoid the `ValueError: too many values to unpack (expected 2)` error. ### Linked issues <!-- DOC: Link issue with a keyword: close, closes, closed, fix, fixes, fixed, resolve, resolves, resolved. See https://docs.github.com/en/issues/tracking-your-work-with-issues/linking-a-pull-request-to-an-issue#linking-a-pull-request-to-an-issue-using-a-keyword --> Resolves #1497
1 parent d2d0179 commit 43284ca

File tree

1 file changed

+1
-1
lines changed

1 file changed

+1
-1
lines changed

tests/integration/hive_metastore/test_migrate.py

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -493,7 +493,7 @@ def test_migrate_external_tables_with_spn_azure(
493493
):
494494
if not ws.config.is_azure:
495495
pytest.skip("temporary: only works in azure test env")
496-
ctx, table_full_name = prepared_principal_acl
496+
ctx, table_full_name, _, _ = prepared_principal_acl
497497
cluster = make_cluster(single_node=True, spark_conf=_SPARK_CONF, data_security_mode=DataSecurityMode.NONE)
498498
ctx.with_dummy_resource_permission()
499499

0 commit comments

Comments
 (0)