Split model classpath using comma delimiter for discover and variable injector

Author: rakillen

core/src/main/python/wlsdeploy/util/model_context.py

@@ -5,11 +5,13 @@
 
 import copy
 import os
+import re
 import tempfile
 
 import java.net.URI as URI
 
 from oracle.weblogic.deploy.util import XPathUtil
+from wlsdeploy.aliases.model_constants import MODEL_LIST_DELIMITER
 from wlsdeploy.aliases.wlst_modes import WlstModes
 from wlsdeploy.logging import platform_logger
 from wlsdeploy.util import validate_configuration
@@ -898,14 +900,17 @@ def tokenize_classpath(self, classpath):
         """
         Replace known types of directories with a tokens that represent the directory.
 
-        :param classpath: containing a string of directories separated by environment specific classpath separator
+        :param classpath: containing a string of directories separated by commas
         :return: tokenized classpath string
         """
-        cp_elements, separator = path_utils.split_classpath(classpath)
+        cp_elements = classpath.split(MODEL_LIST_DELIMITER)
         for index, value in enumerate(cp_elements):
+            path_is_windows = '\\' in value or re.match('^[a-zA-Z][:]', value)
+            if path_is_windows:
+                value = path_utils.fixup_path(value)
             cp_elements[index] = self.tokenize_path(value)
 
-        return separator.join(cp_elements)
+        return MODEL_LIST_DELIMITER.join(cp_elements)
 
     def copy(self, arg_map):
         model_context_copy = copy.copy(self)

core/src/main/python/wlsdeploy/util/path_utils.py

@@ -1,9 +1,8 @@
 """
-Copyright (c) 2017, 2020, Oracle Corporation and/or its affiliates.  All rights reserved.
+Copyright (c) 2017, 2022, Oracle Corporation and/or its affiliates.  All rights reserved.
 Licensed under the Universal Permissive License v 1.0 as shown at https://oss.oracle.com/licenses/upl.
 """
 import os
-import re
 
 import java.io.File as JFile
 
@@ -17,28 +16,6 @@
 _class_name = 'path_utils'
 
 
-def split_classpath(classpath):
-    """
-    Split the classpath string into a list of classpath directories. The classpath string will be split around
-        the environment specific file separator token.
-    :param classpath: string of token separated directories and files representing the classpath
-    :return: list of classpath nodes
-    """
-    classpath_is_windows = False
-    # This is not a definitive test but it tests the cases we care about...
-    if '\\' in classpath or ';' in classpath or re.match('^[a-zA-Z][:]', classpath):
-        classpath_is_windows = True
-
-    if classpath_is_windows:
-        my_classpath = fixup_path(classpath)
-        separator = ';'
-    else:
-        my_classpath = classpath
-        separator = ':'
-
-    return my_classpath.split(separator), separator
-
-
 def fixup_path(path):
     """
     Standardize the path, replacing the back slashes with forward slash. This is more suitable for wlst