Remove unnecessary mockdata parameters

[michplunkett]

Fixes issue

#1125

Description of Changes

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

Tests and Linting

• This branch is up-to-date with the develop branch.
• pytest passes on my local development environment.
• pre-commit passes on my local development environment.

[abandoned-prototype]

Might also want to remove these lines:

OpenOversight/OpenOversight/tests/conftest.py

Lines 751 to 753 in 8b12b4b

	@pytest.fixture
	def mockdata(session):
	pass

[michplunkett]

@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.