added ScriptOptionsTests for C# lang version

dotnet · Mar 4, 2019 · bf068e1 · bf068e1
1 parent 4fb1288
commit bf068e1
Show file tree

Hide file tree

Showing 2 changed files with 29 additions and 0 deletions.
diff --git a/src/Scripting/CSharp/ScriptOptionsExtensions.cs b/src/Scripting/CSharp/ScriptOptionsExtensions.cs
@@ -8,6 +8,9 @@ public static class ScriptOptionsExtensions
     {
         public static ScriptOptions WithLanguageVersion(this ScriptOptions options, LanguageVersion languageVersion)
         {
+            if (options.ParseOptions is CSharpParseOptions csharpParseOptions && csharpParseOptions.LanguageVersion == languageVersion)
+                return options;
+
             return options.WithParseOptions(new CSharpParseOptions(kind: SourceCodeKind.Script, languageVersion: languageVersion));
         }
     }

diff --git a/src/Scripting/CSharpTest/ScriptOptionsTests.cs b/src/Scripting/CSharpTest/ScriptOptionsTests.cs
@@ -0,0 +1,26 @@
+// Copyright (c) Microsoft.  All Rights Reserved.  Licensed under the Apache License, Version 2.0.  See License.txt in the project root for license information.
+
+using Microsoft.CodeAnalysis.CSharp;
+using Microsoft.CodeAnalysis.CSharp.Scripting;
+using Roslyn.Test.Utilities;
+using Xunit;
+
+namespace Microsoft.CodeAnalysis.Scripting.Test
+{
+    public class ScriptOptionsTests : TestBase
+    {
+        [Fact]
+        public void WithLanguageVersion()
+        {
+            var options = ScriptOptions.Default.WithLanguageVersion(LanguageVersion.CSharp8);
+            Assert.Equal(LanguageVersion.CSharp8, ((CSharpParseOptions)options.ParseOptions).LanguageVersion);
+        }
+
+        [Fact]
+        public void WithEmitDebugInformation_SameValueTwice_DoesNotCreateNewInstance()
+        {
+            var options = ScriptOptions.Default.WithLanguageVersion(LanguageVersion.CSharp8);
+            Assert.Same(options, options.WithLanguageVersion(LanguageVersion.CSharp8));
+        }
+    }
+}