feat(bigquery): get_catalog_names

[betodealmeida]

SUMMARY

Implement get_catalog_names for BigQuery. This is needed to support catalog-level permissions, and also for #22862.

BEFORE/AFTER SCREENSHOTS OR ANIMATED GIF

N/A

TESTING INSTRUCTIONS

I configured a BigQuery database and ran this script:

from superset.app import create_app

app = create_app()
with app.app_context():
    from superset import db
    from superset.db_engine_specs.bigquery import BigQueryEngineSpec
    from superset.models.core import Database

    database = db.session.query(Database).first()
    with database.get_inspector_with_context() as inspector:
        print(BigQueryEngineSpec.get_catalog_names(database, inspector))

When ran, it produced the expected output:

['covid19-293320']

ADDITIONAL INFORMATION

• Has associated issue:
• Required feature flags:
• Changes UI
• Includes DB Migration (follow approval process in SIP-59)
  • Migration is atomic, supports rollback & is backwards-compatible
  • Confirm DB migration upgrade and downgrade tested
  • Runtime estimates and downtime expectations provided
• Introduces new feature or API
• Removes existing feature or API

[codecov]

Codecov Report

Merging #23461 (dee6b5d) into master (95db6c0) will increase coverage by 0.08%.
The diff coverage is 45.45%.

❗ Current head dee6b5d differs from pull request most recent head 4d09c1f. Consider uploading reports for the commit 4d09c1f to get more accurate results

@@            Coverage Diff             @@
##           master   #23461      +/-   ##
==========================================
+ Coverage   67.62%   67.70%   +0.08%     
==========================================
  Files        1914     1914              
  Lines       73987    74002      +15     
  Branches     8030     8030              
==========================================
+ Hits        50031    50106      +75     
+ Misses      21906    21846      -60     
  Partials     2050     2050              

Flag	Coverage Δ
hive	52.75% <42.42%> (?)
mysql	78.48% <45.45%> (-0.01%)	⬇️
postgres	78.56% <45.45%> (-0.01%)	⬇️
presto	52.68% <42.42%> (+0.01%)	⬆️
python	82.40% <45.45%> (+0.17%)	⬆️
sqlite	77.06% <45.45%> (-0.01%)	⬇️
unit	52.64% <42.42%> (+0.01%)	⬆️

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files	Coverage Δ
superset/db_engine_specs/bigquery.py	69.54% <45.45%> (-0.18%)	⬇️

... and 3 files with indirect coverage changes

📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more

[betodealmeida]

This test made no sense, so I removed it.

[eschutho]

It looks like this is just catching a formerly uncaught exception, is that right?

[eschutho]

nm, I see where it was caught before with the ImportError