#154 Formatted code and added an explicit assertion (instead of failing by NPE).

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

@@ -49,7 +49,7 @@ public class GitlabGroup {
     }
 
     public GitlabAccessLevel getLdapAccess() {
-        if(ldapAccess == null) {
+        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}
  */
@@ -17,6 +19,6 @@ public class GitlabGroupTest {
     public void getLdapAccessHandlesNull() {
         GitlabGroup group = new GitlabGroup();
         group.setLdapAccess(null);
-        group.getLdapAccess();
+        assertNull( group.getLdapAccess() );
     }
 }