Add null check to workaround NPE in unit tests with Mockito/PowerMock (…

…apache#14006)

- Fixes apache#13620

pulsar-broker/src/main/java/org/apache/pulsar/broker/service/ServerCnx.java

@@ -231,7 +231,10 @@ public ServerCnx(PulsarService pulsar) {
     }
 
     public ServerCnx(PulsarService pulsar, String listenerName) {
-        super(pulsar.getBrokerService().getKeepAliveIntervalSeconds(), TimeUnit.SECONDS);
+        // pulsar.getBrokerService() can sometimes be null in unit tests when using mocks
+        // the null check is a workaround for #13620
+        super(pulsar.getBrokerService() != null ? pulsar.getBrokerService().getKeepAliveIntervalSeconds() : 0,
+                TimeUnit.SECONDS);
         this.service = pulsar.getBrokerService();
         this.schemaService = pulsar.getSchemaRegistryService();
         this.listenerName = listenerName;