fix: permission sqlalchemy events

[dpgaspar]

SUMMARY

There can be cases where a session will not be able to query records just created by a SQLA event using a connection,
on:

connection.execute(view_menu_table.insert().values(name=view_menu_name))
            view_menu = self.find_view_menu(view_menu_name)

This PR fixes this problem by querying just inserted PVM's using connections also.

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 #21454 (07264e3) into master (9c285da) will decrease coverage by 11.21%.
The diff coverage is 100.00%.

❗ Current head 07264e3 differs from pull request most recent head 7989e43. Consider uploading reports for the commit 7989e43 to get more accurate results

@@             Coverage Diff             @@
##           master   #21454       +/-   ##
===========================================
- Coverage   66.53%   55.31%   -11.22%     
===========================================
  Files        1791     1791               
  Lines       68599    68537       -62     
  Branches     7320     7320               
===========================================
- Hits        45645    37914     -7731     
- Misses      21064    28733     +7669     
  Partials     1890     1890               

Flag	Coverage Δ
hive	53.06% <100.00%> (+0.12%)	⬆️
mysql	?
postgres	?
presto	52.96% <100.00%> (+0.12%)	⬆️
python	58.06% <100.00%> (-23.19%)	⬇️
unit	51.07% <100.00%> (+0.11%)	⬆️

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

Impacted Files	Coverage Δ
superset/security/manager.py	64.54% <100.00%> (-31.17%)	⬇️
superset/utils/dashboard_import_export.py	0.00% <0.00%> (-100.00%)	⬇️
superset/key_value/commands/update.py	0.00% <0.00%> (-88.89%)	⬇️
superset/key_value/commands/delete.py	0.00% <0.00%> (-85.30%)	⬇️
superset/key_value/commands/delete_expired.py	0.00% <0.00%> (-80.77%)	⬇️
superset/dashboards/commands/importers/v0.py	15.62% <0.00%> (-76.25%)	⬇️
superset/datasets/commands/update.py	25.00% <0.00%> (-69.05%)	⬇️
superset/datasets/commands/create.py	31.25% <0.00%> (-68.75%)	⬇️
superset/datasets/commands/importers/v0.py	24.03% <0.00%> (-67.45%)	⬇️
superset/reports/commands/execute.py	24.64% <0.00%> (-66.91%)	⬇️
... and 283 more

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

[craig-rueda]

Argh - all these SQLA hooks :(

[sadpandajoe]

🏷️ preset:2022.35