Merge pull request #91 from cuenca-mx/clean-phone

fix just clean correct way
cuenca-mx · May 26, 2020 · c5c14ac · c5c14ac
2 parents fbd2892 + d913fd2
commit c5c14ac
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 1 deletion.
diff --git a/arcusd/arcusactions.py b/arcusd/arcusactions.py
@@ -29,7 +29,7 @@ def unit_to_cents(unit: float) -> int:
 
 
 def clean(value: str) -> str:
-    return re.sub(r'\D', '', value)
+    return re.sub(r'[\W_]+', '', value)
 
 
 def amount_to_unit(cents: int) -> float:

diff --git a/tests/test_arcusd.py b/tests/test_arcusd.py
@@ -13,3 +13,4 @@ def test_clean_phone_number():
     assert clean('(555) 555 5555') == '5555555555'
     assert clean('555-555') == '555555'
     assert clean('555?444') == '555444'
+    assert clean('-.-5544MMCCXX**''') == '5544MMCCXX'