fix(security): can_access with None crashes on builtin roles

[dpgaspar]

SUMMARY

Calling can_access with None values causes an exception when using builtin roles. Will make this more resilient on FAB, but this should be fixed here also.

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

[codecov-commenter]

Codecov Report

Merging #10039 into master will increase coverage by 3.29%.
The diff coverage is 100.00%.

@@            Coverage Diff             @@
##           master   #10039      +/-   ##
==========================================
+ Coverage   65.59%   68.89%   +3.29%     
==========================================
  Files         585      584       -1     
  Lines       31046    31058      +12     
  Branches     3277     3180      -97     
==========================================
+ Hits        20364    21396    +1032     
+ Misses      10501     9553     -948     
+ Partials      181      109      -72     

Flag	Coverage Δ
#cypress	53.85% <ø> (?)
#javascript	59.48% <ø> (+0.11%)	⬆️
#python	67.34% <100.00%> (-2.64%)	⬇️

Impacted Files	Coverage Δ
superset/security/manager.py	89.11% <100.00%> (+0.07%)	⬆️
superset/connectors/druid/models.py	28.16% <0.00%> (-54.36%)	⬇️
superset/connectors/base/models.py	86.42% <0.00%> (-2.51%)	⬇️
superset/views/datasource.py	91.80% <0.00%> (-1.64%)	⬇️
superset/views/core.py	75.39% <0.00%> (-0.89%)	⬇️
superset/db_engine_specs/hive.py	48.27% <0.00%> (-0.16%)	⬇️
superset/viz.py	57.06% <0.00%> (-0.09%)	⬇️
superset/sql_parse.py	99.29% <0.00%> (-0.01%)	⬇️
superset/viz_sip38.py	0.00% <0.00%> (ø)
superset/datasets/api.py	92.56% <0.00%> (ø)
... and 157 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 91517a5...e81167e. Read the comment docs.

[villebro]

LGTM. It would be great to have type checking here to do this work for us.

[dpgaspar]

True, I'm incrementally adding them on FAB also