Add testing steps for running with identity enabled and security plug…

…in installed

Signed-off-by: Craig Perkins <[email protected]>

sandbox/libs/authn/docs/IDENTITY_TESTING.md

@@ -0,0 +1,64 @@
+## Testing with Security Plugin installed
+
+Assemble a local distribution of core:
+
+```
+./gradlew localDistro
+```
+
+Navigate to the platform specific distribution:
+
+```
+cd distribution/archives/<platform>/build/install/opensearch-<version>-SNAPSHOT
+```
+
+### Assemble the Security Plugin
+
+Checkout the security repo: https://github.com/opensearch-project/security
+
+Run gradlew assemble:
+
+```
+./gradlew assemble
+```
+
+### Install the Security Plugin
+
+In the root directory of the local distribution built in core, run:
+
+```
+./bin/opensearch-plugin install --verbose file:<path-to-opensearch-security>/build/distributions/opensearch-security-<version>-SNAPSHOT.zip
+```
+
+to install the security plugin. Answer yes to the prompt. The step above may require the use of `sudo`
+
+Navigate to the security plugin tools directory:
+
+```
+cd plugins/opensearch-security/tools
+```
+
+and run the install_demo_configuration script:
+
+```
+./install_demo_configuration.sh
+```
+
+and answer yes to all prompts.
+
+Go back up to the root directory by running: `cd ../../..`
+
+and open the `opensearch.yml` file:
+
+```
+vim config/opensearch.yml
+```
+
+and enter the following lines:
+
+```
+identity.enabled: true
+identity.auth_manager_class: org.opensearch.identity.authmanager.internal.InternalAuthenticationManager
+```
+
+Note that `identity.auth_manager_class` is optional and will default to the InternalAuthenticationManager. Use this setting if you would like to use a non-default AuthenticationManager.

sandbox/libs/authn/src/main/java/org/opensearch/authn/Identity.java

@@ -8,6 +8,8 @@
 
 package org.opensearch.authn;
 
+import java.util.Objects;
+
 /**
  * Application wide access for identity systems
  *
@@ -25,6 +27,7 @@ private Identity() {}
      * Gets the Authentication Manager for this application
      */
     public static AuthenticationManager getAuthManager() {
+        Objects.requireNonNull(AUTH_MANAGER, "Auth Manager has not been set");
         return AUTH_MANAGER;
     }
 

sandbox/modules/identity/src/main/java/org/opensearch/identity/authmanager/noop/NoopAuthenticationManager.java

@@ -23,7 +23,6 @@ public class NoopAuthenticationManager implements AuthenticationManager {
 
     @Override
     public Subject getSubject() {
-        System.out.println("NoopAuthenticationManager.getSubject");
         return new NoopSubject();
     }
 

sandbox/modules/identity/src/test/java/org/opensearch/identity/noop/NoopAuthenticationManagerTests.java → sandbox/modules/identity/src/test/java/org/opensearch/identity/authmanager/noop/NoopAuthenticationManagerTests.java

@@ -6,7 +6,7 @@
  * compatible open source license.
  */
 
-package org.opensearch.identity.noop;
+package org.opensearch.identity.authmanager.noop;
 
 import org.opensearch.test.OpenSearchTestCase;
 

sandbox/modules/identity/src/test/java/org/opensearch/identity/noop/NoopSubjectTests.java → sandbox/modules/identity/src/test/java/org/opensearch/identity/authmanager/noop/NoopSubjectTests.java

@@ -6,12 +6,11 @@
  * compatible open source license.
  */
 
-package org.opensearch.identity.noop;
+package org.opensearch.identity.authmanager.noop;
 
 import org.opensearch.test.OpenSearchTestCase;
 import org.opensearch.authn.Subject;
 
-import static org.hamcrest.MatcherAssert.assertThat;
 import static org.hamcrest.Matchers.equalTo;
 
 public class NoopSubjectTests extends OpenSearchTestCase {