Action policy/speedup

[deepsidhu85]

What does this PR do and why?

Describe in detail what your merge request does and why.

This PR refactors the project namespace, project, and group policies so that the access level is only retrieved once during the initialization which cuts down on the number of queries to the database.

Some benchmarks from accessing the samples page for streptococcus and descendant groups:

http://localhost:3000/-/groups/streptococcus/-/samples

main: Completed 200 OK in 502ms (Views: 370.9ms | ActiveRecord: 57.8ms (80 queries, 31 cached) | GC: 25.4ms)

actionpolicy speedup: Completed 200 OK in 396ms (Views: 267.2ms | ActiveRecord: 52.3ms (60 queries, 11 cached) | GC: 23.5ms)



http://localhost:3000/-/groups/streptococcus/streptococcus-pyogenes/-/samples

main: Completed 200 OK in 427ms (Views: 296.7ms | ActiveRecord: 49.8ms (76 queries, 32 cached) | GC: 20.8ms)

actionpolicy speedup: Completed 200 OK in 426ms (Views: 297.2ms | ActiveRecord: 55.6ms (56 queries, 12 cached) | GC: 8.0ms)



http://localhost:3000/streptococcus/streptococcus-pyogenes/streptococcus-pyogenes-m92/outbreak-2023/-/samples

main: Completed 200 OK in 1526ms (Views: 984.4ms | ActiveRecord: 66.6ms (97 queries, 84 cached) | GC: 332.5ms)

actionpolicy speedup: Completed 200 OK in 191ms (Views: 129.4ms | ActiveRecord: 19.8ms (22 queries, 9 cached) | GC: 2.5ms)

Screenshots or screen recordings

Screenshots are required for UI changes, and strongly recommended for all other pull requests.

How to set up and validate locally

Numbered steps to set up and validate the change are strongly suggested.

PR acceptance checklist

This checklist encourages us to confirm any changes have been analyzed to reduce risks in quality, performance, reliability, security, and maintainability.

• I have evaluated the PR acceptance checklist for this PR.

[ericenns]

This is looking good, I just found a couple issues and then a couple suggestions.

[github-actions]

Code Metrics Report

Coverage	Test Execution Time
92.9%	10m2s

Code coverage of files in pull request scope (96.7%)

Files	Coverage
app/components/samples/table_component.html.erb	98.5%
app/controllers/projects/application_controller.rb	93.7%
app/controllers/projects_controller.rb	91.9%
app/graphql/resolvers/samples_resolver.rb	100.0%
app/models/member.rb	95.0%
app/policies/group_policy.rb	94.4%
app/policies/namespaces/project_namespace_policy.rb	97.8%
app/policies/project_policy.rb	96.1%
app/policies/sample_policy.rb	100.0%
app/policies/workflow_execution_policy.rb	100.0%
app/services/members/create_service.rb	100.0%
app/services/members/destroy_service.rb	100.0%
app/services/members/update_service.rb	100.0%
app/services/projects/create_service.rb	100.0%
app/services/samples/transfer_service.rb	100.0%

---

Reported by octocov

[ericenns]

Looks good to me!