diff --git a/app/aws/s3.py b/app/aws/s3.py index c56c4e3a0..47c307d44 100644 --- a/app/aws/s3.py +++ b/app/aws/s3.py @@ -66,9 +66,7 @@ def get_job_and_metadata_from_s3(service_id, job_id): def get_job_from_s3(service_id, job_id): - print(f"ENTERE get_job_from_s3 with {service_id} {job_id}") obj = get_s3_object(*get_job_location(service_id, job_id)) - print(f"obj is {obj}") return obj.get()["Body"].read().decode("utf-8") @@ -76,21 +74,11 @@ def get_phone_number_from_s3(service_id, job_id, job_row_number): # We don't want to constantly pull down a job from s3 every time we need a phone number. # At the same time we don't want to store it in redis or the db # So this is a little recycling mechanism to reduce the number of downloads. - print( - f"ENTER GET_PHONE_NUMBER_FROM_S3 WITH JOB_ID {job_id} SERVICE_ID {service_id} JOB_ROW_NUMBER {job_row_number}" - ) job = JOBS.get(job_id) - print(f"JOB FROM EXPIRINGDICT {job} JOB ID {job_id}") if job is None: - print("JOB IS NONE") job = get_job_from_s3(service_id, job_id) - print(f"FRESH JOB {job}") JOBS[job_id] = job - print("ADDED {job} to EXPRING DICT") - else: - print("JOB IS NOT NONE") job = job.split("\r\n") - print(f"JOB AFTER SPLIT {job}") first_row = job[0] job.pop(0) first_row = first_row.split(",") diff --git a/app/delivery/send_to_providers.py b/app/delivery/send_to_providers.py index 1a13608e8..68f151863 100644 --- a/app/delivery/send_to_providers.py +++ b/app/delivery/send_to_providers.py @@ -76,12 +76,10 @@ def send_sms_to_provider(notification): notification.job_id, notification.job_row_number, ) - print(f"MY PHONE FROM JOB {my_phone}") except BaseException: my_phone = redis_store.get(f"2facode_{notification.id}") if my_phone: my_phone = my_phone.decode("utf-8") - print(f"MY PHONE FROM VERIFY CODE {my_phone}") if my_phone is None: raise Exception("what happened to the phone number") send_sms_kwargs = { diff --git a/tests/app/aws/test_s3.py b/tests/app/aws/test_s3.py index f6f03f90a..6af212d87 100644 --- a/tests/app/aws/test_s3.py +++ b/tests/app/aws/test_s3.py @@ -62,7 +62,6 @@ def test_get_phone_number_from_s3( ): get_job_mock = mocker.patch("app.aws.s3.get_job_from_s3") get_job_mock.return_value = job - print(f"ABOUT TO CALL GET_PHONE_NUMBER_FROM_S3 WITH JOB_ID {job_id}") phone_number = get_phone_number_from_s3("service_id", job_id, job_row_number) assert phone_number == expected_phone_number