Skip to content
This repository was archived by the owner on Jan 23, 2023. It is now read-only.

Commit 4d45934

Browse files
committed
Merge pull request #2829 from stephentoub/fix_pipe_impersonation_test
Fix System.IO.Pipes test on 64-bit
2 parents 54f98dd + c2b2639 commit 4d45934

File tree

2 files changed

+2
-3
lines changed

2 files changed

+2
-3
lines changed

src/System.IO.Pipes/tests/NamedPipeTests/NamedPipeTest.Specific.cs

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -2,6 +2,7 @@
22
// Licensed under the MIT license. See LICENSE file in the project root for full license information.
33

44
using System.Runtime.InteropServices;
5+
using System.Security.Principal;
56
using System.Threading;
67
using System.Threading.Tasks;
78
using Xunit;
@@ -89,7 +90,7 @@ public void Windows_MessagePipeTransissionMode()
8990

9091
using (NamedPipeServerStream server = new NamedPipeServerStream(pipeName, PipeDirection.InOut, 1, PipeTransmissionMode.Message))
9192
{
92-
using (NamedPipeClientStream client = new NamedPipeClientStream(".", pipeName, PipeDirection.InOut, PipeOptions.None, Security.Principal.TokenImpersonationLevel.Identification))
93+
using (NamedPipeClientStream client = new NamedPipeClientStream(".", pipeName, PipeDirection.InOut, PipeOptions.None, TokenImpersonationLevel.Impersonation))
9394
{
9495
server.ReadMode = PipeTransmissionMode.Message;
9596
Assert.Equal(PipeTransmissionMode.Message, server.ReadMode);

src/System.IO.Pipes/tests/System.IO.Pipes.Tests.csproj

Lines changed: 0 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -11,8 +11,6 @@
1111
<ProjectGuid>{142469EC-D665-4FE2-845A-FDA69F9CC557}</ProjectGuid>
1212
<NuGetPackageImportStamp>d2615b94</NuGetPackageImportStamp>
1313
<AllowUnsafeBlocks>true</AllowUnsafeBlocks>
14-
<!-- Temporary workaround until tests are fixed on x64 -->
15-
<TestArchitecture>x86</TestArchitecture>
1614
</PropertyGroup>
1715
<!-- Default configurations to help VS understand the configurations -->
1816
<PropertyGroup Condition=" '$(Configuration)|$(Platform)' == 'Debug|AnyCPU' ">

0 commit comments

Comments
 (0)