Merge pull request #1057 from cipous/1056_authenticationModes

Add authenticationModes to Account.java

Reviewed-by: Peter Plocháň
             https://github.com/peter-plochan

Author: gdgate

gooddata-java-model/src/main/java/com/gooddata/sdk/model/account/Account.java

@@ -47,6 +47,9 @@ public class Account {
     @JsonView(UpdateView.class)
     private List<String> ipWhitelist;
 
+    @JsonView(UpdateView.class)
+    private List<String> authenticationModes;
+
     @JsonIgnore
     private final Links links;
 
@@ -59,6 +62,7 @@ private Account(
             @JsonProperty("firstName") String firstName,
             @JsonProperty("lastName") String lastName,
             @JsonProperty("ipWhitelist") List<String> ipWhitelist,
+            @JsonProperty("authenticationModes") List<String> authenticationModes,
             @JsonProperty("links") Links links
     ) {
         this.login = login;
@@ -68,11 +72,12 @@ private Account(
         this.firstName = firstName;
         this.lastName = lastName;
         this.ipWhitelist = ipWhitelist;
+        this.authenticationModes = authenticationModes;
         this.links = links;
     }
 
     public Account(String firstName, String lastName, Links links) {
-        this(null, null, null, null, firstName, lastName, null, links);
+        this(null, null, null, null, firstName, lastName, null, null, links);
     }
 
     /**
@@ -83,7 +88,7 @@ public Account(String firstName, String lastName, Links links) {
      * @param password password
      */
     public Account(String email, String password, String firstName, String lastName) {
-        this(email, email, password, password, firstName, lastName, null, null);
+        this(email, email, password, password, firstName, lastName, null, null, null);
     }
 
     public String getLogin() {
@@ -129,6 +134,10 @@ public List<String> getIpWhitelist() {
         return ipWhitelist;
     }
 
+    public List<String> getAuthenticationModes() {
+        return authenticationModes;
+    }
+
     public void setEmail(final String email) {
         this.email = email;
     }
@@ -153,6 +162,10 @@ public void setIpWhitelist(final List<String> ipWhitelist) {
         this.ipWhitelist = ipWhitelist;
     }
 
+    public void setAuthenticationModes(final List<String> authenticationModes) {
+        this.authenticationModes = authenticationModes;
+    }
+
     @JsonIgnoreProperties(ignoreUnknown = true)
     private static class Links {
         private final String self;

gooddata-java-model/src/test/java/com/gooddata/sdk/model/account/AccountTest.java

@@ -16,7 +16,7 @@
 import static org.hamcrest.Matchers.contains;
 import static org.hamcrest.text.MatchesPattern.matchesPattern;
 
-import java.util.Arrays;
+import java.util.Collections;
 
 public class AccountTest {
 
@@ -37,12 +37,14 @@ public void testDeserialize() throws Exception {
         assertThat(account.getUri(), is("/gdc/account/profile/ID"));
         assertThat(account.getProjectsUri(), is("/gdc/account/profile/ID/projects"));
         assertThat(account.getIpWhitelist(), contains(IP));
+        assertThat(account.getAuthenticationModes(), contains("SSO"));
     }
 
     @Test
     public void testSerialization() {
         final Account account = new Account(FIRST_NAME, LAST_NAME, null);
-        account.setIpWhitelist(Arrays.asList("1.2.3.4/32"));
+        account.setIpWhitelist(Collections.singletonList("1.2.3.4/32"));
+        account.setAuthenticationModes(Collections.singletonList("SSO"));
         assertThat(account, jsonEquals(resource("account/account-input.json")));
     }
 

gooddata-java-model/src/test/resources/account/account-input.json

@@ -2,6 +2,7 @@
     "accountSetting" : {
         "firstName" : "Blah",
         "lastName" : "Muhehe",
-        "ipWhitelist" : ["1.2.3.4/32"]
+        "ipWhitelist" : ["1.2.3.4/32"],
+        "authenticationModes" : ["SSO"]
     }
 }

gooddata-java-model/src/test/resources/account/account.json

@@ -5,7 +5,7 @@
         "ssoProvider" : null,
         "timezone" : null,
         "position" : null,
-        "authenticationModes" : [],
+        "authenticationModes" : ["SSO"],
         "companyName" : "gdc",
         "login" : "fake@gooddata.com",
         "email" : "fake@gooddata.com",

gooddata-java/src/test/java/com/gooddata/sdk/service/account/AccountServiceAT.java

@@ -11,6 +11,7 @@
 import org.testng.annotations.AfterClass;
 import org.testng.annotations.Test;
 
+import java.util.Collections;
 import java.util.UUID;
 
 import static org.hamcrest.MatcherAssert.assertThat;
@@ -38,12 +39,13 @@ public void login() throws Exception {
     @Test(groups = "isolated_domain")
     public void createAccount() {
         final Account newAccount = new Account(LOGIN, "w4yYxSQpAbaODA64", "FistName", "LastName");
-
+        newAccount.setAuthenticationModes(Collections.singletonList("SSO"));
         account = accountService.createAccount(newAccount, getProperty("domain"));
 
         assertThat(account, is(notNullValue()));
         assertThat(account.getId(), is(notNullValue()));
         assertThat(account.getLogin(), is(LOGIN));
+        assertThat(account.getAuthenticationModes(), is(Collections.singletonList("SSO")));
     }
 
     @Test(groups = "isolated_domain", dependsOnMethods = "createAccount")

gooddata-java/src/test/java/com/gooddata/sdk/service/account/AccountServiceIT.java

@@ -13,6 +13,8 @@
 import org.testng.annotations.Test;
 
 import java.io.IOException;
+import java.util.Collections;
+import java.util.List;
 
 import static com.gooddata.sdk.common.util.ResourceUtils.*;
 import static net.jadler.Jadler.onRequest;
@@ -222,12 +224,14 @@ public void shouldUpdateAccount() throws Exception {
         final String newEmail = "fake2@gooddata.com";
         final String newPass = "password2";
         final String newLastName = "Muhehe2";
+        final List<String> authenticationModes = Collections.singletonList("SSO");
 
         toBeUpdated.setFirstName(newFirstName);
         toBeUpdated.setEmail(newEmail);
         toBeUpdated.setPassword(newPass);
         toBeUpdated.setVerifyPassword(newPass);
         toBeUpdated.setLastName(newLastName);
+        toBeUpdated.setAuthenticationModes(authenticationModes);
 
         gd.getAccountService().updateAccount(toBeUpdated);
     }

gooddata-java/src/test/resources/account/update-account.json

@@ -5,6 +5,7 @@
     "verifyPassword": "password2",
     "firstName" : "newFirstName2",
     "lastName" : "Muhehe2",
-    "ipWhitelist" : ["1.2.3.4/32"]
+    "ipWhitelist" : ["1.2.3.4/32"],
+    "authenticationModes" : ["SSO"]
   }
 }