fix(ssh_tunnel): Address Base PR comments from peer review

[Antonio-RiveroMartnez]

SUMMARY

Address code reviews comments:

• feat(ssh-tunnelling): Setup SSH Tunneling Commands for Database Connections  #21912 (comment)
• feat(ssh-tunnelling): Setup SSH Tunneling Commands for Database Connections  #21912 (comment)
• feat(ssh-tunnelling): Setup SSH Tunneling Commands for Database Connections  #21912 (comment)
• https://github.com/apache/superset/pull/21912/files#r1028278374
• https://github.com/apache/superset/pull/21912/files#r1028286296
• https://github.com/apache/superset/pull/21912/files#r1026937885
• https://github.com/apache/superset/pull/21912/files#r1026956904
• https://github.com/apache/superset/pull/21912/files#r1035561067
• https://github.com/apache/superset/pull/21912/files#r1035558379
• https://github.com/apache/superset/pull/21912/files#r1035558237

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 #22306 (d24786f) into create-sshtunnelconfig-tbl (3d3b71b) will decrease coverage by 0.17%.
The diff coverage is 50.00%.

❗ Current head d24786f differs from pull request most recent head 882477e. Consider uploading reports for the commit 882477e to get more accurate results

@@                      Coverage Diff                       @@
##           create-sshtunnelconfig-tbl   #22306      +/-   ##
==============================================================
- Coverage                       55.77%   55.59%   -0.18%     
==============================================================
  Files                            1852     1852              
  Lines                           70473    70461      -12     
  Branches                         7689     7689              
==============================================================
- Hits                            39303    39174     -129     
- Misses                          29176    29293     +117     
  Partials                         1994     1994              

Flag	Coverage Δ
hive	?
presto	52.34% <40.47%> (-0.14%)	⬇️
python	57.56% <50.00%> (-0.37%)	⬇️
unit	51.04% <47.61%> (+<0.01%)	⬆️

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

Impacted Files	Coverage Δ
superset/models/core.py	76.32% <26.92%> (-0.15%)	⬇️
superset/databases/ssh_tunnel/models.py	77.41% <80.00%> (+0.75%)	⬆️
superset/constants.py	100.00% <100.00%> (ø)
superset/databases/commands/test_connection.py	64.47% <100.00%> (+2.27%)	⬆️
superset/databases/schemas.py	85.47% <100.00%> (+0.09%)	⬆️
superset/databases/ssh_tunnel/commands/create.py	90.47% <100.00%> (-0.83%)	⬇️
superset/databases/ssh_tunnel/commands/update.py	88.00% <100.00%> (-0.47%)	⬇️
superset/db_engines/hive.py	0.00% <0.00%> (-85.19%)	⬇️
superset/db_engine_specs/hive.py	34.38% <0.00%> (-30.44%)	⬇️
superset/connectors/sqla/utils.py	77.66% <0.00%> (-2.92%)	⬇️
... and 5 more

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

[eschutho]

thanks @Antonio-RiveroMartnez. Looks great!

[rusackas]

Re-running CI... will merge if it passes 🤞