8000 Merge pull request #269 from filipw/bugfix/268 · dotnet-script/dotnet-script@ef8b07c · GitHub
[go: up one dir, main page]

Skip to content

Commit ef8b07c

Browse files
authored
Merge pull request #269 from filipw/bugfix/268
issue 268 quick fix
2 parents a76a0f0 + 44bbbd1 commit ef8b07c

File tree

3 files changed

+14
-0
lines changed

3 files changed

+14
-0
lines changed

src/Dotnet.Script.Core/Dotnet.Script.Core.csproj

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -37,6 +37,7 @@
3737
<PackageReference Include="System.Collections.Immutable" Version="1.4.0" />
3838
<PackageReference Include="System.Reflection.Metadata" Version="1.5.0" />
3939
<PackageReference Include="System.ValueTuple" Version="4.4.0" />
40+
<PackageReference Include="System.Configuration.ConfigurationManager" Version="4.4.1" />
4041
</ItemGroup>
4142

4243
<ItemGroup>

src/Dotnet.Script.Tests/ScriptExecutionTests.cs

Lines changed: 7 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -224,6 +224,13 @@ public void ShouldExecuteRemoteScriptUsingTinyUrl()
224224
Assert.Contains("Hello World", result.output);
225225
}
226226

227+
[Fact]
228+
public void ShouldHandleIssue268()
229+
{
230+
var result = Execute($"{Path.Combine("Issue268", "Issue268.csx")}");
231+
Assert.Contains("value:", result.output);
232+
}
233+
227234
[Fact]
228235
public void ShouldThrowExceptionOnInvalidMediaType()
229236
{
Lines changed: 6 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,6 @@
1+
#r "nuget: System.Configuration.ConfigurationManager, 4.4.1"
2+
3+
using System.Configuration;
4+
5+
var value = ConfigurationManager.AppSettings.Get("SomeValue");
6+
Console.WriteLine("value: " + value);

0 commit comments

Comments
 (0)
0