Don't hardcode /tmp in named pipe tests, it's not for us on Android

[directhex]

Closes: #49873

[ghost]

Tagging subscribers to this area: @dotnet/area-system-io
See info in area-owners.md if you want to be subscribed.

Issue Details

---

Closes: #49873

Author:	directhex
Assignees:	-
Labels:	area-System.IO
Milestone:	-

[akoeplinger]

LGTM. It seems we hit cases years ago where TMPDIR on macOS was very long so the named pipe failed, if that's the case then I'd just special case this for Android.

[directhex]

The OSX tests have passed. Is there some test beyond what's in the libs tests for this TMPDIR issue?

[jozkee]

Is this other one not an issue for Android?

runtime/src/libraries/System.IO.Pipes/tests/NamedPipeTests/NamedPipeTest.CurrentUserOnly.cs

Line 46 in 222da17

string name = Path.Combine("/tmp", PipeStreamConformanceTests.GetUniquePipeName());

[directhex]

Is this other one not an issue for Android?

runtime/src/libraries/System.IO.Pipes/tests/NamedPipeTests/NamedPipeTest.CurrentUserOnly.cs

Line 46 in 222da17

string name = Path.Combine("/tmp", PipeStreamConformanceTests.GetUniquePipeName());

Nope.

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

Line 26 in 222da17

<ItemGroup Condition="'$(TargetPlatformIdentifier)' == 'windows'">