Added tests to check users and admin were receiving different data about volunteers.
Deleted duplicate tests, and tests that were no longer relevant.
Refactored mock data so that each test received a fresh copy of the data.
In the route itself, we removed the logic for making a deep clone of the data that came back from the database because no other functions access this data so it's unnecessary to clone it.
Added tests to check users and admin were receiving different data about volunteers. Deleted duplicate tests, and tests that were no longer relevant. Refactored mock data so that each test received a fresh copy of the data. In the route itself, we removed the logic for making a deep clone of the data that came back from the database because no other functions access this data so it's unnecessary to clone it.