JerseySseEventSource#register throws IllegalStateException (#3892)

Signed-off-by: Maxim Nesen <[email protected]>

Author: senivam

media/sse/src/main/java/org/glassfish/jersey/media/sse/internal/JerseySseEventSource.java

@@ -122,7 +122,7 @@ private void subscribe(final Consumer<Flow.Subscription> onSubscribe,
                            final Consumer<Throwable> onError,
                            final Runnable onComplete) {
         if (onSubscribe == null || onEvent == null || onError == null || onComplete == null) {
-            throw new IllegalStateException(LocalizationMessages.PARAMS_NULL());
+            throw new IllegalArgumentException(LocalizationMessages.PARAMS_NULL());
         }
 
         publisher.subscribe(new Flow.Subscriber<InboundSseEvent>() {

tests/e2e-entity/src/test/java/org/glassfish/jersey/tests/e2e/sse/GenericEntityTest.java

@@ -106,6 +106,13 @@ public void testGenericList() throws InterruptedException {
         }
     }
 
+
+    @Test(expected = IllegalArgumentException.class)
+    public void testRegisterNull() {
+        final SseEventSource source = SseEventSource.target(target("genericentityresource/string")).build();
+        source.register(null);
+    }
+
     @Singleton
     @Path("genericentityresource")
     public static class SSEGenericEntityResource {