fix: Adding trailing / to URLs set (#2364)

* [fix] Adding trailing / to URLs set

* Update tests

---------

Co-authored-by: Mark Hamilton <mhamilton723@gmail.com>

Author: sss04

cognitive/src/main/scala/com/microsoft/azure/synapse/ml/services/openai/OpenAIDefaults.scala

@@ -45,7 +45,8 @@ object OpenAIDefaults {
   }
 
   def setURL(v: String): Unit = {
-    GlobalParams.setGlobalParam(URLKey, v)
+    val url = if (v.endsWith("/")) v else v + "/"
+    GlobalParams.setGlobalParam(URLKey, url)
   }
 
   def getURL: Option[String] = {

cognitive/src/test/python/synapsemltest/services/openai/test_OpenAIDefaults.py

@@ -21,12 +21,12 @@ def test_setters_and_getters(self):
         defaults.set_deployment_name("Bing Bong")
         defaults.set_subscription_key("SubKey")
         defaults.set_temperature(0.05)
-        defaults.set_URL("Test URL")
+        defaults.set_URL("Test URL/")
 
         self.assertEqual(defaults.get_deployment_name(), "Bing Bong")
         self.assertEqual(defaults.get_subscription_key(), "SubKey")
         self.assertEqual(defaults.get_temperature(), 0.05)
-        self.assertEqual(defaults.get_URL(), "Test URL")
+        self.assertEqual(defaults.get_URL(), "Test URL/")
 
     def test_resetters(self):
         defaults = OpenAIDefaults()
@@ -39,7 +39,7 @@ def test_resetters(self):
         self.assertEqual(defaults.get_deployment_name(), "Bing Bong")
         self.assertEqual(defaults.get_subscription_key(), "SubKey")
         self.assertEqual(defaults.get_temperature(), 0.05)
-        self.assertEqual(defaults.get_URL(), "Test URL")
+        self.assertEqual(defaults.get_URL(), "Test URL/")
 
         defaults.reset_deployment_name()
         defaults.reset_subscription_key()