fix: check sqlalchemy_uri

[dpgaspar]

SUMMARY

Updates check_sqlalchemy_uri

BEFORE/AFTER SCREENSHOTS OR ANIMATED GIF

TESTING INSTRUCTIONS

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 #23901 (8a3e0a3) into master (31d3359) will increase coverage by 1.75%.
The diff coverage is 65.57%.

❗ Current head 8a3e0a3 differs from pull request most recent head 8815d4b. Consider uploading reports for the commit 8815d4b to get more accurate results

@@            Coverage Diff             @@
##           master   #23901      +/-   ##
==========================================
+ Coverage   66.34%   68.10%   +1.75%     
==========================================
  Files        1938     1939       +1     
  Lines       74972    75010      +38     
  Branches     8141     8154      +13     
==========================================
+ Hits        49739    51084    +1345     
+ Misses      23154    21843    -1311     
- Partials     2079     2083       +4     

Flag	Coverage Δ
hive	53.01% <33.33%> (?)
mysql	78.80% <73.33%> (+<0.01%)	⬆️
postgres	78.88% <73.33%> (+<0.01%)	⬆️
presto	52.92% <33.33%> (?)
python	82.67% <96.66%> (+3.66%)	⬆️
sqlite	77.39% <73.33%> (+<0.01%)	⬆️
unit	52.81% <73.33%> (?)

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

Impacted Files	Coverage Δ
...s/legacy-plugin-chart-calendar/src/controlPanel.ts	50.00% <ø> (ø)
...rts/src/BigNumber/BigNumberTotal/transformProps.ts	0.00% <0.00%> (ø)
...lugin-chart-echarts/src/BigNumber/BigNumberViz.tsx	0.00% <0.00%> (ø)
...tend/src/components/Chart/DrillBy/DrillByChart.tsx	100.00% <ø> (ø)
.../components/Chart/DrillBy/useDisplayModeToggle.tsx	100.00% <ø> (ø)
...c/components/Chart/DrillBy/useResultsTableView.tsx	100.00% <ø> (ø)
...onalFormattingControl/FormattingPopoverContent.tsx	51.35% <ø> (ø)
superset/examples/supported_charts_dashboard.py	0.00% <ø> (ø)
...harts/src/BigNumber/BigNumberTotal/controlPanel.ts	30.00% <22.22%> (-70.00%)	⬇️
...s/ExploreChartPanel/useResizeDetectorByObserver.ts	62.50% <62.50%> (ø)
... and 7 more

... and 88 files with indirect coverage changes

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

[villebro]

Nice, I find regex based allow/denylists are practically always better 👍