fix: force_screenshot migration

[betodealmeida]

SUMMARY

The migration was failing because chart_id was not present in the ReportSchedule model defined in the migration.

BEFORE/AFTER SCREENSHOTS OR ANIMATED GIF

N/A

TESTING INSTRUCTIONS

1. Create an alert.
2. Downgrade to the version before the broken migration: superset db downgrade 31bb738bd1d2
3. Upgrade the DB: superset db upgrade
4. Verify that the column force_screenshot was added, and has the value true for the created alert:

SELECT force_screenshot FROM report_schedule;
 force_screenshot
------------------
 t
(1 row)

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 #17928 (3c27a8a) into master (de3d397) will decrease coverage by 0.00%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master   #17928      +/-   ##
==========================================
- Coverage   67.07%   67.07%   -0.01%     
==========================================
  Files        1609     1609              
  Lines       64903    64903              
  Branches     6866     6866              
==========================================
- Hits        43535    43531       -4     
- Misses      19502    19506       +4     
  Partials     1866     1866              

Flag	Coverage Δ
hive	53.29% <ø> (ø)
mysql	82.18% <ø> (-0.01%)	⬇️
postgres	82.23% <ø> (-0.02%)	⬇️
presto	53.13% <ø> (ø)
python	82.67% <ø> (-0.02%)	⬇️
sqlite	81.92% <ø> (ø)

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

Impacted Files	Coverage Δ
superset/reports/commands/log_prune.py	85.71% <0.00%> (-3.58%)	⬇️
superset/commands/importers/v1/utils.py	89.13% <0.00%> (-2.18%)	⬇️
superset/common/query_object.py	95.08% <0.00%> (-0.55%)	⬇️
superset/reports/commands/execute.py	90.90% <0.00%> (-0.38%)	⬇️

---

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 de3d397...3c27a8a. Read the comment docs.

[rosemarie-chiu]

🏷 2022.1