feat: add prop to setDBEngine in DatabaseModal

[hughhhh]

SUMMARY

Add prop setDBEngine to programmatically skip the step 1 and push users in to step2 for the DatabaseConnectionForm.

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 #18653 (65a975d) into master (00eb6b1) will decrease coverage by 0.00%.
The diff coverage is 92.85%.

❗ Current head 65a975d differs from pull request most recent head 1cbdb13. Consider uploading reports for the commit 1cbdb13 to get more accurate results

@@            Coverage Diff             @@
##           master   #18653      +/-   ##
==========================================
- Coverage   66.29%   66.28%   -0.01%     
==========================================
  Files        1603     1603              
  Lines       62744    62745       +1     
  Branches     6320     6323       +3     
==========================================
- Hits        41593    41592       -1     
- Misses      19499    19501       +2     
  Partials     1652     1652              

Flag	Coverage Δ
javascript	51.28% <60.00%> (-0.01%)	⬇️

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

Impacted Files	Coverage Δ
...hart-echarts/src/MixedTimeseries/transformProps.ts	0.00% <0.00%> (ø)
.../src/explore/components/controls/SliderControl.tsx	0.00% <0.00%> (ø)
superset/db_engine_specs/mssql.py	95.83% <ø> (ø)
superset/connectors/sqla/models.py	88.74% <95.83%> (ø)
...lugin-chart-echarts/src/Timeseries/transformers.ts	51.23% <100.00%> (+0.40%)	⬆️
...c/views/CRUD/data/database/DatabaseModal/index.tsx	32.90% <100.00%> (+0.43%)	⬆️
superset/db_engine_specs/base.py	88.43% <100.00%> (ø)

---

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 00eb6b1...1cbdb13. Read the comment docs.

[ad-m]

I have small suggestions to consider.

[ad-m]

I wonder if this property name is optimal because it sounds a bit like an operation instead of a React-style declarative declaration. Can we delete set part of that name?

[eschutho]

agree with this 👍

[betodealmeida]

Good point, this sounds mode like a hook for setting state than a value.

[pkdotson]

lgtm!

[eschutho]

still a console log

[eschutho]

Looks good!