fix: Ensure verbose mapping exists for SQL Lab Query model

[john-bodley]

SUMMARY

This PR fixes an error when applying post processing for a pivot chart et al. when the underlying datasource is non-materialized, i.e., a superset.models.sql_lab.Query object as opposed to a materialized datasource, i.e., a superset.connectors.base.models.BaseDatasource object.

The underlying issue is that the superset.models.sql_lab.Query.data property did not include the verbose_map field which is present in the superset.connectors.base.models.BaseDatasource.dataproperty, i.e.,

...
  File "<prefix>/superset/charts/post_processing.py", line 352, in apply_post_process
    df.rename(columns=datasource.data["verbose_map"], inplace=True)
KeyError: 'verbose_map'

BEFORE/AFTER SCREENSHOTS OR ANIMATED GIF

TESTING INSTRUCTIONS

CI and tested locally.

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

[john-bodley]

Present here for the superset.connectors.base.models.BaseDatasource model.

[codecov]

Codecov Report

Merging #23597 (c53f60c) into master (56dcf25) will decrease coverage by 0.01%.
The diff coverage is 57.14%.

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

@@            Coverage Diff             @@
##           master   #23597      +/-   ##
==========================================
- Coverage   67.72%   67.72%   -0.01%     
==========================================
  Files        1916     1916              
  Lines       74051    74054       +3     
  Branches     8040     8040              
==========================================
+ Hits        50153    50154       +1     
- Misses      21850    21852       +2     
  Partials     2048     2048              

Flag	Coverage Δ
hive	52.75% <57.14%> (-0.01%)	⬇️
mysql	78.48% <57.14%> (-0.01%)	⬇️
postgres	78.56% <57.14%> (-0.01%)	⬇️
presto	52.67% <57.14%> (-0.01%)	⬇️
python	82.41% <57.14%> (-0.01%)	⬇️
sqlite	77.07% <57.14%> (-0.01%)	⬇️
unit	52.64% <57.14%> (-0.01%)	⬇️

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

Impacted Files	Coverage Δ
superset/models/sql_lab.py	75.00% <ø> (ø)
superset/utils/core.py	90.90% <33.33%> (-0.10%)	⬇️
superset/charts/post_processing.py	89.36% <50.00%> (-0.64%)	⬇️
superset/charts/data/api.py	88.88% <100.00%> (+0.06%)	⬆️

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

[michael-s-molina]

LGTM. Thank you for improving type safety.