Merge pull request #154 from iwarapter/master

handle NPE when calling GitlabGroup.getLdapAccess()

src/main/java/org/gitlab/api/models/GitlabGroup.java

@@ -49,6 +49,9 @@ public class GitlabGroup {
     }
 
     public GitlabAccessLevel getLdapAccess() {
+        if (ldapAccess == null) {
+            return null;
+        }
         return GitlabAccessLevel.fromAccessValue(ldapAccess);
     }
 

src/test/java/org/gitlab/api/models/GitlabGroupTest.java

@@ -2,6 +2,8 @@ package org.gitlab.api.models;
 
 import org.junit.Test;
 
+import static org.junit.Assert.assertNull;
+
 /**
  * Tests for {@link GitlabGroup}
  */
@@ -13,4 +15,10 @@ public class GitlabGroupTest {
         group.setLdapAccess(null);
     }
 
+    @Test
+    public void getLdapAccessHandlesNull() {
+        GitlabGroup group = new GitlabGroup();
+        group.setLdapAccess(null);
+        assertNull( group.getLdapAccess() );
+    }
 }