Never return null from Strings.tokenizeToStringArray (#28224)

This method has a different contract than all the other methods in this class, returning null instead of an empty array
when receiving a null input. While switching over some methods from delimitedListToStringArray to this method
tokenizeToStringArray, this resulted in unexpected nulls in some places of our code.

Author: ywelsch

core/src/main/java/org/elasticsearch/common/Strings.java

@@ -508,6 +508,9 @@ public static String[] split(String toSplit, String delimiter) {
      * @see #delimitedListToStringArray
      */
     public static String[] tokenizeToStringArray(String str, String delimiters) {
+        if (str == null) {
+            return EMPTY_ARRAY;
+        }
         return tokenizeToStringArray(str, delimiters, true, true);
     }
 

core/src/test/java/org/elasticsearch/cluster/routing/allocation/decider/FilterAllocationDeciderTests.java

@@ -194,6 +194,14 @@ public void testInvalidIPFilter() {
         assertEquals("invalid IP address [" + invalidIP + "] for [" + filterSetting.getKey() + ipKey + "]", e.getMessage());
     }
 
+    public void testNull() {
+        Setting<String> filterSetting = randomFrom(IndexMetaData.INDEX_ROUTING_REQUIRE_GROUP_SETTING,
+            IndexMetaData.INDEX_ROUTING_INCLUDE_GROUP_SETTING, IndexMetaData.INDEX_ROUTING_EXCLUDE_GROUP_SETTING);
+
+        IndexMetaData.builder("test")
+            .settings(settings(Version.CURRENT).putNull(filterSetting.getKey() + "name")).numberOfShards(2).numberOfReplicas(0).build();
+    }
+
     public void testWildcardIPFilter() {
         String ipKey = randomFrom("_ip", "_host_ip", "_publish_ip");
         Setting<String> filterSetting = randomFrom(IndexMetaData.INDEX_ROUTING_REQUIRE_GROUP_SETTING,