don't overload the setter

Author: Mudaafi

constructorio-client/src/main/java/io/constructor/client/CatalogRequest.java

@@ -86,7 +86,9 @@ public String getSection() {
 
     /**
      * @param email the email address to receive a notification in case the task fails
+     * @deprecated Use {@link #setNotificationEmails(List)} instead to support multiple emails.
      */
+    @Deprecated
     public void setNotificationEmail(String email) {
         this.notificationEmails.clear();
         if (email != null) {
@@ -98,7 +100,7 @@ public void setNotificationEmail(String email) {
      * @param emails list of email addresses where you'd like to receive notifications in case the
      *     task fails
      */
-    public void setNotificationEmail(List<String> emails) {
+    public void setNotificationEmails(List<String> emails) {
         this.notificationEmails.clear();
         if (emails != null) {
             this.notificationEmails.addAll(emails);

constructorio-client/src/test/java/io/constructor/client/CatalogRequestTest.java

@@ -86,7 +86,7 @@ public void setNotificationEmailShouldSet() throws Exception {
         CatalogRequest request = new CatalogRequest(files, "Products");
 
         List<String> emails = Arrays.asList("a@constructor.io", "b@constructor.io");
-        request.setNotificationEmail(emails);
+        request.setNotificationEmails(emails);
 
         assertEquals(request.getNotificationEmails(), emails);
         assertEquals(request.getNotificationEmail(), "a@constructor.io");

constructorio-client/src/test/java/io/constructor/client/ConstructorIONotificationEmailTest.java

@@ -296,7 +296,7 @@ public void replaceCatalogWithMultipleEmailsShouldIncludeAllEmailsInUrl() throws
         Map<String, File> files = new HashMap<String, File>();
         files.put("items", new File("src/test/resources/items.csv"));
         CatalogRequest req = new CatalogRequest(files, "Products");
-        req.setNotificationEmail(Arrays.asList("a@constructor.io", "b@constructor.io"));
+        req.setNotificationEmails(Arrays.asList("a@constructor.io", "b@constructor.io"));
 
         constructor.replaceCatalog(req);
 
@@ -342,7 +342,7 @@ public void updateCatalogWithMultipleEmailsShouldIncludeAllEmailsInUrl() throws
         Map<String, File> files = new HashMap<String, File>();
         files.put("items", new File("src/test/resources/items.csv"));
         CatalogRequest req = new CatalogRequest(files, "Products");
-        req.setNotificationEmail(Arrays.asList("a@constructor.io", "b@constructor.io"));
+        req.setNotificationEmails(Arrays.asList("a@constructor.io", "b@constructor.io"));
 
         constructor.updateCatalog(req);
 
@@ -369,7 +369,7 @@ public void patchCatalogWithMultipleEmailsShouldIncludeAllEmailsInUrl() throws E
         Map<String, File> files = new HashMap<String, File>();
         files.put("items", new File("src/test/resources/items.csv"));
         CatalogRequest req = new CatalogRequest(files, "Products");
-        req.setNotificationEmail(Arrays.asList("a@constructor.io", "b@constructor.io"));
+        req.setNotificationEmails(Arrays.asList("a@constructor.io", "b@constructor.io"));
 
         constructor.patchCatalog(req);