Conversation
|
Tagging subscribers to this area: @dotnet/area-system-io Issue DetailsFix #30108. remove the test: I cannot see how this test could pass. when you create a directory/file, it does not have default values for its various time attributes.
|
|
I was looking through the ActiveIssues on System.IO.FileSystem to get an idea of how much cruft is there. Of 59 ActiveIssue attributes, there are only 12 distinct issues, of which 6 are device/blazor specific and thus relatively new, and of the remaining 6 I looked at 4 and clearly at least these 3 should not exist. They have all been present for years -- one of these issues is over 4 years old. This suggests there's a lot of cruft. cc @stephentoub @eerhardt as both have pointed out we are over due for cleanup. |
|
Here's a snippet of one of the issues -- we should Be Bold in such cases: |
src/libraries/System.IO.FileSystem/tests/Directory/EnumerableTests.cs
Outdated
Show resolved
Hide resolved
Co-authored-by: Stephen Toub <[email protected]>
ghost
locked as resolved and limited conversation to collaborators
3 participants
Fix #30108. remove the test: I cannot see how this test could pass. when you create a directory/file, it does not have default values for its various time attributes.
Fix #24295. We do not plan to make the enumerator thread safe. It happens to be thread safe on Windows, so leaving the test for Windows so that if we do change this for some reason, we can document it in case anyone depends on it.
Fix #43166. Per discussion, this cannot be made to work on modern Linux distros.