darkhelm/notifications-api
1
0
Fork 0
mirror of https://github.com/GSA/notifications-api.git synced 2026-02-04 02:11:11 -05:00
Code Issues Packages Projects Releases Wiki Activity

Remove obsolete strip_leading_plus_one method

Browse Source
This commit is contained in:
Ryan Ahearn
2023-01-06 11:14:45 -05:00
parent 8a0535fa03
commit 828671ecfc
2 changed files with 0 additions and 22 deletions
Download Patch File Download Diff File Expand all files Collapse all files

8
app/notifications/receive_notifications.py
View File

@@ -134,11 +134,3 @@ def fetch_potential_service(inbound_number, provider_name):
def has_inbound_sms_permissions(permissions):
str_permissions = [p.permission for p in permissions]
return set([INBOUND_SMS_TYPE, SMS_TYPE]).issubset(set(str_permissions))
def strip_leading_plus_one(number):
if number.startswith('+'):
number = number.replace('+', '', 1)
if number.startswith('1'):
number = number.replace('1', '', 1)
return number

14
tests/app/notifications/test_receive_notification.py
View File

@@ -8,7 +8,6 @@ from app.models import EMAIL_TYPE, INBOUND_SMS_TYPE, SMS_TYPE, InboundSms
from app.notifications.receive_notifications import (
create_inbound_sms_object,
has_inbound_sms_permissions,
strip_leading_plus_one,
unescape_string,
)
from tests.app.db import (
@@ -238,19 +237,6 @@ def test_receive_notification_error_if_not_single_matching_service(client, notif
assert InboundSms.query.count() == 0
@pytest.mark.parametrize(
'number, expected',
[
('12028675309', '2028675309'),
('+12028675309', '2028675309'),
('2028675309', '2028675309'),
('15111111111', '5111111111')
]
)
def test_strip_leading_country_code(number, expected):
assert strip_leading_plus_one(number) == expected
@pytest.mark.skip(reason="Need to implement inbound SNS tests. Body here from MMG")
@pytest.mark.parametrize("auth, keys, status_code", [
["testkey", ["testkey"], 200],