Merge 0dda8c4 into 2806583

inspire_schemas/builders/authors.py

@@ -28,6 +28,7 @@
 from inspire_utils.date import normalize_date, PartialDate
 from inspire_utils.helpers import force_list
 from inspire_utils.name import normalize_name
+from inspire_utils.record import get_value
 
 from ..utils import EMPTIES, filter_empty_parameters, load_schema
 
@@ -114,9 +115,11 @@ def add_email_address(self, email):
             :param email: public email of the author.
             :type email: string
         """
-        self._append_to('email_addresses', {
-            "value": email
-        })
+        emails = get_value(self.obj, 'email_addresses.value')
+        if emails is None or email not in emails:
+            self._append_to('email_addresses', {
+                "value": email
+            })
 
     @filter_empty_parameters
     def set_status(self, status):

tests/unit/test_author_builder.py

@@ -202,6 +202,23 @@ def test_add_multiple_email_addresses():
     assert expected == result
 
 
+def test_add_email_addresses_skips_duplicate_ones():
+    schema = load_schema('authors')
+    subschema = schema['properties']['email_addresses']
+
+    author = AuthorBuilder()
+    author.add_email_address('example@test.com')
+    author.add_email_address('example@test.com')
+
+    expected = [{
+        "value": 'example@test.com'
+    }]
+    result = author.obj['email_addresses']
+
+    assert validate(result, subschema) is None
+    assert expected == result
+
+
 def test_set_status():
     schema = load_schema('authors')
     subschema = schema['properties']['status']