Fix usPhone with empty phone number
This commit is contained in:
parent
a8e9430196
commit
3e50ce3d48
@ -30,6 +30,8 @@ def justCents(value):
|
|||||||
|
|
||||||
|
|
||||||
def usPhone(number):
|
def usPhone(number):
|
||||||
|
if not number:
|
||||||
|
return ""
|
||||||
phone = re.sub(r"\D", "", number)
|
phone = re.sub(r"\D", "", number)
|
||||||
return "+1 ({}) {} - {}".format(phone[0:3], phone[3:6], phone[6:])
|
return "+1 ({}) {} - {}".format(phone[0:3], phone[3:6], phone[6:])
|
||||||
|
|
||||||
|
@ -1,6 +1,6 @@
|
|||||||
import pytest
|
import pytest
|
||||||
|
|
||||||
from atst.filters import dollars, renderAuditEvent
|
from atst.filters import dollars, renderAuditEvent, usPhone
|
||||||
from atst.models import AuditEvent
|
from atst.models import AuditEvent
|
||||||
|
|
||||||
|
|
||||||
@ -28,3 +28,9 @@ def test_render_audit_event_with_unknown_resource_type():
|
|||||||
event = AuditEvent(resource_type="boat")
|
event = AuditEvent(resource_type="boat")
|
||||||
result = renderAuditEvent(event)
|
result = renderAuditEvent(event)
|
||||||
assert "<article" in result
|
assert "<article" in result
|
||||||
|
|
||||||
|
|
||||||
|
def test_usPhone():
|
||||||
|
assert usPhone("1234567890") == "+1 (123) 456 - 7890"
|
||||||
|
assert usPhone(number=None) == ""
|
||||||
|
assert usPhone(number="") == ""
|
||||||
|
Loading…
x
Reference in New Issue
Block a user