refactor(ImportModelsCommand): clean code, extract logic into methods

[michael-hoffman-26]

SUMMARY

refactor the validate function of ImportModelsCommand.
extracting logic into functions with meaningful names, thats make the code cleaner.
no logic change.

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 #18866 (d2cd27f) into master (0edb979) will increase coverage by 0.00%.
The diff coverage is 100.00%.

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

@@           Coverage Diff           @@
##           master   #18866   +/-   ##
=======================================
  Coverage   66.36%   66.37%           
=======================================
  Files        1621     1620    -1     
  Lines       63058    63047   -11     
  Branches     6382     6375    -7     
=======================================
- Hits        41851    41846    -5     
+ Misses      19547    19542    -5     
+ Partials     1660     1659    -1     

Flag	Coverage Δ
hive	52.28% <27.27%> (-0.01%)	⬇️
mysql	81.55% <100.00%> (+<0.01%)	⬆️
postgres	81.60% <100.00%> (+<0.01%)	⬆️
presto	52.12% <27.27%> (-0.01%)	⬇️
python	82.04% <100.00%> (+<0.01%)	⬆️
sqlite	81.29% <100.00%> (+<0.01%)	⬆️

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

Impacted Files	Coverage Δ
superset/commands/importers/v1/__init__.py	95.12% <100.00%> (+0.38%)	⬆️
...omponents/nativeFilters/FilterBar/Header/index.tsx	95.23% <0.00%> (-4.77%)	⬇️
...nents/nativeFilters/FilterBar/FilterSets/index.tsx	36.84% <0.00%> (-0.83%)	⬇️
...ilters/FilterBar/FilterControls/FilterControls.tsx	77.14% <0.00%> (-0.64%)	⬇️
superset-frontend/src/utils/common.js	89.28% <0.00%> (-0.19%)	⬇️
superset-frontend/src/dashboard/constants.ts	100.00% <0.00%> (ø)
superset-frontend/src/components/Button/index.tsx	100.00% <0.00%> (ø)
...frontend/plugins/plugin-chart-table/src/Styles.tsx	100.00% <0.00%> (ø)
...tend/plugins/plugin-chart-table/src/TableChart.tsx	38.57% <0.00%> (ø)
...mponents/controls/AnnotationLayerControl/index.jsx	10.16% <0.00%> (ø)
... and 4 more

---

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 0edb979...a135b59. Read the comment docs.

[amitmiran137]

LGTM,
need to fix lint