fix: Assign an owner when creating a dataset from a csv, excel or tabular

[cccs-joel]

SUMMARY

The existing code has a bug and no owner gets assigned when a dataset was created from a data upload. The fix assigns the current user as the owner when creating a dataset from a csv, excel or tabular.

BEFORE/AFTER SCREENSHOTS OR ANIMATED GIF

TESTING INSTRUCTIONS

1. Ensure you have a defined database that allows data upload
2. From the Data drop-down menu in the top menu bar, select Upload a CSV
3. Enter appropriate values and hit SAVE.
4. Find the dataset created in /tablemodelview/list/ and click the Edit icon.
5. In the Settings tab of the dialog, under Owners, the user who uploaded the data is listed.

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 #17986 (3941192) into master (05d9cde) will decrease coverage by 0.26%.
The diff coverage is 100.00%.

@@            Coverage Diff             @@
##           master   #17986      +/-   ##
==========================================
- Coverage   67.10%   66.83%   -0.27%     
==========================================
  Files        1612     1631      +19     
  Lines       64999    65739     +740     
  Branches     6872     6872              
==========================================
+ Hits        43616    43937     +321     
- Misses      19513    19932     +419     
  Partials     1870     1870              

Flag	Coverage Δ
hive	52.17% <0.00%> (-1.15%)	⬇️
mysql	81.28% <100.00%> (-0.95%)	⬇️
postgres	81.32% <100.00%> (-0.95%)	⬇️
presto	52.01% <0.00%> (-1.15%)	⬇️
python	81.76% <100.00%> (-0.95%)	⬇️
sqlite	81.02% <100.00%> (-0.94%)	⬇️

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

Impacted Files	Coverage Δ
superset/views/database/views.py	88.46% <100.00%> (ø)
superset/examples/multi_line.py	0.00% <0.00%> (-53.85%)	⬇️
superset/utils/encrypt.py	41.46% <0.00%> (-46.54%)	⬇️
superset/commands/importers/v1/examples.py	0.00% <0.00%> (-38.64%)	⬇️
superset/examples/big_data.py	0.00% <0.00%> (-35.00%)	⬇️
superset/examples/misc_dashboard.py	0.00% <0.00%> (-33.34%)	⬇️
superset/examples/utils.py	0.00% <0.00%> (-28.58%)	⬇️
superset/examples/tabbed_dashboard.py	0.00% <0.00%> (-27.59%)	⬇️
superset/examples/bart_lines.py	0.00% <0.00%> (-25.81%)	⬇️
superset/utils/mock_data.py	0.00% <0.00%> (-25.19%)	⬇️
... and 85 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 05d9cde...3941192. Read the comment docs.

[cccs-joel]

@villebro I thiiiik you offered some help to write integration tests for that, I would take it!

[villebro]

villebro I thiiiik you offered some help to write integration tests for that, I would take it!

@cccs-joel yes, I faintly recall doing that 😄 Let me take a stab at adding a test tomorrow 👍

[villebro]

LGTM, thanks for the great integration tests!