Merge pull request #85 from dex3r/stack_overflow_fix

Fix stack overflow bug in ChatMessageRole

Author: OkGoDoIt

OpenAI_API/Chat/ChatMessageRole.cs

@@ -94,7 +94,7 @@ public bool Equals(ChatMessageRole other)
 		/// Gets the string value for this role to pass to the API
 		/// </summary>
 		/// <param name="value">The ChatMessageRole to convert</param>
-		public static implicit operator String(ChatMessageRole value) { return value; }
+		public static implicit operator String(ChatMessageRole value) { return value.Value; }
 
 		///// <summary>
 		///// Used during the Json serialization process

OpenAI_Tests/ChatMessageRoleTests.cs

@@ -0,0 +1,22 @@
+using NUnit.Framework;
+using OpenAI_API.Chat;
+
+namespace OpenAI_Tests
+{
+    public class ChatMessageRoleTests
+    {
+        [Test]
+        public void TestImplicitConversionNotThrowing()
+        {
+            // ReSharper disable once UnusedVariable
+            string result = ChatMessageRole.System;
+        }
+        
+        [Test]
+        public void TestImplicitConversionValue()
+        {
+            string result = ChatMessageRole.System;
+            Assert.AreEqual("system", result);
+        }
+    }
+}