feat(ingest/snowflake): improvements for view ingestion

[hsheth2]

Improves upon #13369

[codecov]

Codecov Report

❌ Patch coverage is 58.13953% with 18 lines in your changes missing coverage. Please review.
✅ All tests successful. No failed tests found.

Files with missing lines	Patch %	Lines
...hub/ingestion/source/snowflake/snowflake_schema.py	53.33%	14 Missing ⚠️
...ahub/ingestion/source/snowflake/snowflake_query.py	66.66%	2 Missing ⚠️
...ingestion/source/snowflake/snowflake_schema_gen.py	60.00%	2 Missing ⚠️

❌ Your patch status has failed because the patch coverage (58.13%) is below the target coverage (75.00%). You can increase the patch coverage or adjust the target coverage.

📢 Thoughts on this report? Let us know!

[sgomezvillamor]

when calling the method, db name goes first

    @staticmethod
    def get_views_for_schema(db_name: str, schema_name: str) -> str:

Suggested change

	SnowflakeQuery.get_views_for_schema(schema_name, db_name),
	SnowflakeQuery.get_views_for_schema(db_name=db_name, schema_name=schema_name),

[sgomezvillamor]

LGTM

[gabe-lyons]

Note to self: this needs to be populated via show views in cases where the role is not the owner of the view

[sgomezvillamor]

Superseded by PR #14444