Skip to content

Commit

Permalink
Merge pull request #25 from Gaffney/fix-similar-email
Browse files Browse the repository at this point in the history
Fix similar email issue
  • Loading branch information
terite committed Apr 14, 2016
2 parents 4466562 + 5c96ed7 commit becca52
Show file tree
Hide file tree
Showing 3 changed files with 5 additions and 1 deletion.
2 changes: 1 addition & 1 deletion anonymizer/replacers.py
Original file line number Diff line number Diff line change
Expand Up @@ -124,7 +124,7 @@ def similar_email(anon, obj, field, val):
"""
Generate a random email address using the same domain.
"""
return val if 'betterworks.com' in val else '@'.join([anon.faker.username(field=field), val.split('@')[-1]])
return val if 'betterworks.com' in val else '@'.join([anon.faker.user_name(field=field), val.split('@')[-1]])


def full_address(anon, obj, field, val):
Expand Down
1 change: 1 addition & 0 deletions anonymizer/tests/models.py
Original file line number Diff line number Diff line change
Expand Up @@ -35,6 +35,7 @@ class EverythingModel(models.Model):
first_name = models.CharField(max_length=5)
last_name = models.CharField(max_length=5)
similar_email = models.EmailField()
similar_email_other = models.EmailField()
phonenumber = models.CharField(max_length=10)
last_name = models.CharField(max_length=11)
street_address = models.CharField(max_length=15)
Expand Down
3 changes: 3 additions & 0 deletions anonymizer/tests/tests.py
Original file line number Diff line number Diff line change
Expand Up @@ -69,6 +69,7 @@ class EverythingModelAnonymizer(Anonymizer):
('country', "varchar"),
('first_name', "first_name"),
('similar_email', "email"),
('similar_email_other', "email"),
('phonenumber', "phonenumber"),
('last_name', "last_name"),
('street_address', "full_address"),
Expand Down Expand Up @@ -116,6 +117,7 @@ def setUp(self):
first_name='Joe',
last_name='Schmoe',
similar_email='[email protected]',
similar_email_other='[email protected]',
phonenumber='(555) 555-5555',
street_address='123 Maple Street',
state='CA',
Expand Down Expand Up @@ -163,6 +165,7 @@ class EverythingAnonmyizer(Anonymizer):
('first_name', "first_name"),
('last_name', "last_name"),
('similar_email', "similar_email"),
('similar_email_other', "similar_email"),
('phonenumber', "phonenumber"),
('street_address', "street_address"),
('state', "state"),
Expand Down

0 comments on commit becca52

Please sign in to comment.