fix: load examples as anon user

[betodealmeida]

SUMMARY

In #23200 I added permission checks when importing related assets. Before, users importing dashboards could create databases and datasets as a side effect of the import, even if they didn't have the appropriate permissions. The PR added some extra checks to prevent this from happening:

superset/superset/databases/commands/importers/v1/utils.py

Line 34 in 56dcf25

can_write = security_manager.can_access("can_write", "Database")

The change above required modifying load-examples to run as a user with username "admin":

superset/superset/commands/importers/v1/examples.py

Line 72 in 56dcf25

with override_user(security_manager.find_user(username="admin")):

But the user "admin" might not exist, or might be called differently. So this PR changes load-examples so that the imports happen without a current_user, as before, while keeping the permission checks for other imports initiated via the UI.

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 #23600 (daabdbe) into master (56dcf25) will increase coverage by 0.00%.
The diff coverage is 40.00%.

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

@@           Coverage Diff           @@
##           master   #23600   +/-   ##
=======================================
  Coverage   67.72%   67.72%           
=======================================
  Files        1916     1916           
  Lines       74051    74050    -1     
  Branches     8040     8040           
=======================================
  Hits        50153    50153           
+ Misses      21850    21849    -1     
  Partials     2048     2048           

Flag	Coverage Δ
hive	52.75% <0.00%> (+<0.01%)	⬆️
mysql	78.49% <40.00%> (+<0.01%)	⬆️
postgres	78.57% <40.00%> (+<0.01%)	⬆️
presto	52.68% <0.00%> (+<0.01%)	⬆️
python	82.42% <40.00%> (+<0.01%)	⬆️
sqlite	77.07% <40.00%> (+<0.01%)	⬆️
unit	52.65% <40.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/examples.py	0.00% <0.00%> (ø)
superset/charts/commands/importers/v1/utils.py	100.00% <100.00%> (ø)
superset/dashboards/commands/importers/v1/utils.py	85.00% <100.00%> (ø)
superset/databases/commands/importers/v1/utils.py	100.00% <100.00%> (ø)
superset/datasets/commands/importers/v1/utils.py	78.64% <100.00%> (ø)

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

[Antonio-RiveroMartnez]

LGTM!