Update to test as per review comments.

However, I am curious if there is evidence that using fake_uuid over creating one in the test is actually faster.

app/celery/tasks.py

@@ -163,7 +163,7 @@ def process_job(job_id):
         if template.template_type == 'sms':
             send_sms.apply_async((
                 str(job.service_id),
-                str(create_uuid()),
+                create_uuid(),
                 encrypted,
                 datetime.utcnow().strftime(DATETIME_FORMAT)),
                 queue='bulk-sms'
@@ -172,7 +172,7 @@ def process_job(job_id):
         if template.template_type == 'email':
             send_email.apply_async((
                 str(job.service_id),
-                str(create_uuid()),
+                create_uuid(),
                 '"{}" <{}@{}>'.format(
                     service.name,
                     service.email_from,

tests/app/authentication/test_authentication.py

@@ -276,14 +276,14 @@ def test_authentication_returns_error_when_api_client_has_no_secrets(notify_api,
 def test_authentication_returns_error_when_service_has_no_secrets(notify_api,
                                                                   notify_db,
                                                                   notify_db_session,
-                                                                  sample_service):
+                                                                  sample_service,
+                                                                  fake_uuid):
     with notify_api.test_request_context():
         with notify_api.test_client() as client:
-            import uuid
             token = create_jwt_token(
                 request_method="GET",
                 request_path='/service',
-                secret=str(uuid.uuid4()),
+                secret=fake_uuid,
                 client_id=str(sample_service.id))
 
             response = client.get(