New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Adds auth coverage #9886

Merged

Rotorsoft merged 1 commit into master from rotorsoft/9885-add-auth-coverage

Nov 13, 2024

Contributor

Rotorsoft commented Nov 12, 2024

Link to Issue

Closes: #9885

Description of Changes

Adds more tests
Removes unnecessary defaults
Fixes group permissions query

"How We Fixed It"

Test Plan

Unit tested the FIXME() call.
CA (click around) tested on local and frack:
- TODO page

Deployment Plan

Other Considerations


          adds auth coverage

0d02765

Rotorsoft requested review from timolegros and rbennettcw

November 12, 2024 20:30

timolegros approved these changes

View reviewed changes

kurtisassad approved these changes

View reviewed changes

Rotorsoft merged commit 0bbe313 into master

11 checks passed

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet