factor out DatasourceAccessRequest model

[suddjian]

CATEGORY

• Bug Fix
• Enhancement (new features, refinement)
• Refactor
• Add tests
• Build / Development Environment
• Documentation

SUMMARY

Part of a multi-part refactoring effort, more PRs incoming

TEST PLAN

ADDITIONAL INFORMATION

• Has associated issue:
• Changes UI
• Requires DB Migration.
• Confirm DB Migration upgrade and downgrade tested.
• Introduces new feature or API
• Removes existing feature or API

REVIEWERS

[craig-rueda]

👍

[willbarrett]

Woot. Thanks for breaking this up.

[codecov-io]

Codecov Report

Merging #8809 into master will increase coverage by 0.32%.
The diff coverage is 57.62%.

@@            Coverage Diff             @@
##           master    #8809      +/-   ##
==========================================
+ Coverage    65.9%   66.22%   +0.32%     
==========================================
  Files         483      484       +1     
  Lines       24165    24507     +342     
  Branches     2770     2770              
==========================================
+ Hits        15926    16230     +304     
- Misses       8061     8099      +38     
  Partials      178      178

Impacted Files	Coverage Δ
superset/models/core.py	85.58% <ø> (+2.48%)	⬆️
superset/models/__init__.py	100% <100%> (ø)	⬆️
superset/views/core.py	72.15% <100%> (+0.01%)	⬆️
superset/models/datasource_access_request.py	55.35% <55.35%> (ø)
superset/views/log/views.py	100% <0%> (ø)	⬆️
superset/exceptions.py	100% <0%> (ø)	⬆️
superset/sql_parse.py	97.74% <0%> (+1.61%)	⬆️
superset/views/base.py	77.35% <0%> (+2.61%)	⬆️
superset/translations/utils.py	90.9% <0%> (+3.4%)	⬆️
... 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 60914fa...c0ef8d5. Read the comment docs.

[mistercrunch]