fix: import database engine validation

[dpgaspar]

SUMMARY

Fixes improves URI validation when importing databases, will work for importing datasets also.

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 #24697 (14c36c2) into master (65fb8e1) will increase coverage by 0.00%.
The diff coverage is 100.00%.

❗ Current head 14c36c2 differs from pull request most recent head a11867a. Consider uploading reports for the commit a11867a to get more accurate results

@@           Coverage Diff           @@
##           master   #24697   +/-   ##
=======================================
  Coverage   68.97%   68.98%           
=======================================
  Files        1901     1901           
  Lines       74006    74014    +8     
  Branches     8182     8182           
=======================================
+ Hits        51045    51056   +11     
+ Misses      20840    20837    -3     
  Partials     2121     2121           

Flag	Coverage Δ
hive	54.11% <44.44%> (-0.01%)	⬇️
mysql	79.36% <77.77%> (+<0.01%)	⬆️
postgres	79.45% <77.77%> (+<0.01%)	⬆️
presto	54.01% <44.44%> (-0.01%)	⬇️
python	83.46% <100.00%> (+0.01%)	⬆️
sqlite	78.04% <77.77%> (+<0.01%)	⬆️
unit	54.86% <100.00%> (+0.02%)	⬆️

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

Impacted Files	Coverage Δ
superset/databases/commands/importers/v1/utils.py	100.00% <100.00%> (ø)

... and 1 file with indirect coverage changes

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

[michael-s-molina]

LGTM