Bed 4582

[stephanieslamb]

Description

Updated DeleteSavedQueries endpoint to check if query is public and if user has admin privileges.

Motivation and Context

This PR addresses: BED-4582

Why is this change required? What problem does it solve?
This change is part of the on-going initiative to saved cypher queries. This change will prevent users who are not admin from deleting public saved queries.

How Has This Been Tested?

Additional unit tests were added for additional functionality and all previous unit tests are passing.

Screenshots (optional):

Types of changes

• New feature (non-breaking change which adds functionality)

Checklist:

• I have met the contributing prerequisites
  • Assigned myself to this PR
  • Added the appropriate labels
  • Associated an issue: Issues and Pull Requests README #672
  • Read the Contributing guide: https://github.com/SpecterOps/BloodHound/wiki/Contributing
• I have ensured that related documentation is up-to-date
  • Open API docs
  • Code comments (GoDocs / JSDocs)
• I have followed proper test practices
  • Added/updated tests to cover my changes
  • All new and existing tests passed

[superlinkx]

A couple of things to hopefully reduce nested logic a bit and move some code back towards the database. Otherwise, the tests look pretty comprehensive and code looks as expected.