lucyparsons / OpenOversight

Police oversight and accountability through public data 👮
https://openoversight.com
GNU General Public License v3.0
240 stars 79 forks source link

Remove unnecessary `mockdata` parameters #1126

Closed michplunkett closed 1 month ago

michplunkett commented 1 month ago

Fixes issue

https://github.com/lucyparsons/OpenOversight/issues/1125

Description of Changes

Removed the mockdata parameter from test functions that don't require it.

Tests and Linting

michplunkett commented 1 month ago

@abandoned-prototype, we still use mockdata in other tests throughout various files (test_descriptions.py, test_image_tagging.py, test_functional, test_models, etc.) and , so we can't get rid of it quite yet. To remove those lines, we'd need to do some general test restructuring that would go beyond this simple pruning.