Skip to content
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.

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

Added some permission tests for accessory api endpoints #15473

Merged
merged 2 commits into from
Sep 10, 2024

Conversation

marcusmoore
Copy link
Collaborator

Description

On a testing kick...

This PR adds some simple permission tests for most of the accessory api routes. Definitely a starting point and will be built upon.

Type of change

  • Testing

Copy link

what-the-diff bot commented Sep 9, 2024

PR Summary

  • Testing Permission for Deleting Accessories
    A new test is added to ensure appropriate permissions are required for deleting an accessory in our system.
  • Testing Permission for Viewing Accessories List
    Another test is added to verify that the appropriate level of access is required to view the list of accessories.
  • Testing Permission for Displaying an Accessory
    We've implemented a test to confirm that the correct permissions are needed to display details of any accessory.
  • Testing Permission for Saving Accessories
    A new testing mechanism confirms users require the right permission level to save or upload an accessory.
  • Testing Permission for Updating Accessories
    Another test addition ensures that only users with suitable access rights can update accessory details.

@snipe snipe merged commit 7f3f77d into snipe:develop Sep 10, 2024
9 checks passed
@marcusmoore marcusmoore deleted the testing/accessory_api_tests branch September 10, 2024 16:35
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants