Fixes #27842: Fix unitycataog error when httpPath is missing

[mohittilala]

Describe your changes:

Fixes #27842

UnityCatalog connections results into AttributeError: 'NoneType' object has no attribute 'root' when httpPath parameter is not provided even though it being optional parameter. This PR aims to resolve the bug in connection.connectionArguments resolution when httpPath and connectionArguments are not provided in the connection.

Type of change:

• Bug fix
• Improvement
• New feature
• Breaking change (fix or feature that would cause existing functionality to not work as expected)
• Documentation

Checklist:

• I have read the CONTRIBUTING document.
• My PR title is Fixes <issue-number>: <short explanation>
• I have commented on my code, particularly in hard-to-understand areas.
• For JSON Schema changes: I updated the migration scripts or explained why it is not needed.

Bug fix

• I have added a test that covers the exact scenario we are fixing. For complex issues, comment the issue number in the test for future reference.

[Copilot]

Pull request overview

Fixes a UnityCatalog ingestion crash when httpPath and connectionArguments are omitted by ensuring connection.connectionArguments is initialized before injecting auth args.

Changes:

• Initialize connection.connectionArguments earlier in get_sqlalchemy_connection to avoid AttributeError when it is missing.
• Add unit tests covering engine creation when httpPath/connectionArguments are unset and when httpPath is provided.

Reviewed changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated 3 comments.

File	Description
ingestion/src/metadata/ingestion/source/database/unitycatalog/connection.py	Initializes connectionArguments before adding http_path and auth args when building the SQLAlchemy engine.
ingestion/tests/unit/topology/database/test_unity_catalog_connection.py	Adds regression/unit coverage for missing httpPath/connectionArguments and basic engine creation scenarios.

[github-actions]

The Python checkstyle failed.

Please run make py_format and py_format_check in the root of your repository and commit the changes to this PR.
You can also use pre-commit to automate the Python code formatting.

You can install the pre-commit hooks with make install_test precommit_install.

[gitar-bot]

Code Review ✅ Approved

Handles missing httpPath by adding a null check in the UnityCatalog client, preventing potential runtime errors. No issues found.

Options

Display: compact → Showing less information.

Comment with these commands to change:

Compact

gitar display:verbose

_{Was this helpful? React with 👍 / 👎 | Gitar}

[Copilot]

Pull request overview

Copilot reviewed 2 out of 2 changed files in this pull request and generated 1 comment.

[github-actions]

🟡 Playwright Results — all passed (10 flaky)

✅ 3988 passed · ❌ 0 failed · 🟡 10 flaky · ⏭️ 86 skipped

Shard	Passed	Failed	Flaky	Skipped
🟡 Shard 1	298	0	1	4
🟡 Shard 2	749	0	5	8
🟡 Shard 3	744	0	2	7
✅ Shard 4	775	0	0	18
✅ Shard 5	687	0	0	41
🟡 Shard 6	735	0	2	8

🟡 10 flaky test(s) (passed on retry)

• Features/MetricCustomUnitFlow.spec.ts › Should create metric and test unit of measurement updates (shard 1, 1 retry)
• Features/ActivityAPI.spec.ts › Activity event is created when description is updated (shard 2, 1 retry)
• Features/ActivityAPI.spec.ts › Activity event shows the actor who made the change (shard 2, 1 retry)
• Features/DataQuality/ColumnLevelTests.spec.ts › Column Value Mean To Be Between (shard 2, 1 retry)
• Features/Glossary/GlossaryWorkflow.spec.ts › should display correct status badge color and icon (shard 2, 1 retry)
• Features/Glossary/GlossaryWorkflow.spec.ts › should start term as Draft when glossary has reviewers (shard 2, 1 retry)
• Features/Permissions/DataProductPermissions.spec.ts › Data Product deny operations (shard 3, 1 retry)
• Features/RTL.spec.ts › Verify Following widget functionality (shard 3, 1 retry)
• Pages/Lineage/LineageFilters.spec.ts › Verify lineage schema filter selection (shard 6, 1 retry)
• Pages/Lineage/LineageRightPanel.spec.ts › Verify custom properties tab IS visible for supported type: searchIndex (shard 6, 1 retry)

📦 Download artifacts

How to debug locally

# Download playwright-test-results-<shard> artifact and unzip
npx playwright show-trace path/to/trace.zip    # view trace

[sonarqubecloud]

Quality Gate passed for 'open-metadata-ingestion'

Issues
1 New issue
0 Accepted issues

Measures
0 Security Hotspots
100.0% Coverage on New Code
0.0% Duplication on New Code

See analysis details on SonarQube Cloud

[github-actions]

Changes have been cherry-picked to the 1.13 branch.

[github-actions]

Changes have been cherry-picked to the 1.12.7 branch.

[mohittilala]

Manually cherry-picked to 1.12.6 for 1.12.6.4 patch- b9d8846