#951 Replace AuthMode.OFF by AuthMode.NONE

src/main/java/org/zalando/nakadi/config/SecurityConfiguration.java

@@ -210,7 +210,7 @@ private static Status fromStatusCode(final int code) throws UnknownStatusCodeExc
     public class WebSecurityConfiguration extends WebSecurityConfigurerAdapter {
         @Override
         public void configure(final WebSecurity web) throws Exception {
-            if (settings.getAuthMode() == SecuritySettings.AuthMode.OFF) {
+            if (settings.getAuthMode().isNoAuthentication()) {
                 web.ignoring().anyRequest();
             }
         }

src/main/java/org/zalando/nakadi/config/SecuritySettings.java

@@ -11,9 +11,15 @@ public class SecuritySettings {
 
     public enum AuthMode {
         OFF,   // no authentication at all
+        NONE,  // same as OFF
         BASIC, // only checks that the token is valid (has "uid" scope)
         REALM, // checks that the token is valid and contains at least one required realm
         FULL   // full authentication and authorization using oauth2 scopes
+        ;
+
+        public boolean isNoAuthentication() {
+            return AuthMode.OFF == this || AuthMode.NONE == this;
+        }
     }
 
     private final String tokenInfoUrl;

src/main/java/org/zalando/nakadi/security/ClientResolver.java

@@ -19,8 +19,6 @@
 import java.util.Map;
 import java.util.Optional;
 
-import static org.zalando.nakadi.config.SecuritySettings.AuthMode.OFF;
-
 @Component
 public class ClientResolver implements HandlerMethodArgumentResolver {
 
@@ -49,7 +47,7 @@ public Client resolveArgument(final MethodParameter parameter,
         final String clientId = authorizationService.getSubject().map(Subject::getName)
                 .orElse(SecuritySettings.UNAUTHENTICATED_CLIENT_ID);
 
-        if(settings.getAuthMode() == OFF) {
+        if(settings.getAuthMode().isNoAuthentication()) {
             return new FullAccessClient(clientId);
         } else {
             if (clientId.equals(settings.getAdminClientId())) {

src/test/java/org/zalando/nakadi/config/SecuritySettingsTest.java

@@ -0,0 +1,28 @@
+package org.zalando.nakadi.config;
+
+import org.junit.Assert;
+import org.junit.Test;
+
+import java.util.Arrays;
+import java.util.LinkedList;
+import java.util.List;
+
+public class SecuritySettingsTest {
+
+    @Test
+    public void doesNotRequireAuthentication() {
+        Assert.assertTrue(SecuritySettings.AuthMode.NONE.isNoAuthentication());
+        Assert.assertTrue(SecuritySettings.AuthMode.OFF.isNoAuthentication());
+    }
+
+    @Test
+    public void mustRequireAuthentication() {
+        final List<SecuritySettings.AuthMode> authModesWithoutAuthentication =
+                new LinkedList<>(Arrays.asList(SecuritySettings.AuthMode.values()));
+        authModesWithoutAuthentication.remove(SecuritySettings.AuthMode.NONE);
+        authModesWithoutAuthentication.remove(SecuritySettings.AuthMode.OFF);
+        authModesWithoutAuthentication.forEach(authMode ->
+                Assert.assertFalse(authMode.isNoAuthentication()));
+    }
+
+}