Incorrect documented IOException in ZipFile.ExtractToDirectory

[pkanavos]

In ZipFile.ExtractToDirectory and ZipFileExtensions.ExtractToDirectory, the remarks and documentation say that IOException is thrown if the destination directory already exists. This is incorrect, as the method that's eventually called, the internal ExtractRelativeToDirectory method works fine with existing destination folders.

This was first asked in this SO question. I also opened an issue in the dotnett-api-docs repo

runtime/src/libraries/System.IO.Compression.ZipFile/src/System/IO/Compression/ZipFile.Extract.cs

Line 64 in 0e557e8

/// <exception cref="IOException">The directory specified by destinationDirectoryName already exists.

[ghost]

Tagging subscribers to this area: @carlossanlop
See info in area-owners.md if you want to be subscribed.

Issue Details

---

In ExtractToDirectory, the remarks and documentation say that IOException is thrown if the destination directory already exists. This is incorrect, as the method that's eventually called, the internal ExtractRelativeToDirectory method works fine with existing destination folders.

This was first asked in this SO question

runtime/src/libraries/System.IO.Compression.ZipFile/src/System/IO/Compression/ZipFile.Extract.cs

Line 64 in 0e557e8

/// <exception cref="IOException">The directory specified by destinationDirectoryName already exists.

Author:	pkanavos
Assignees:	-
Labels:	area-System.IO.Compression, untriaged
Milestone:	-

[adamsitnik]

Hi @pkanavos

Would you like to contribute a PR where you would remove the remark and add a test that ensures that it's true?

Thanks!

[Jozkee]

@pkanavos for System.IO.Compression namespace, the source of truth of documentation is the triple-slash comments in our source, can you please send the PR to this repo (dotnet/runtime) instead? We will port it to dotnet-api-docs ourselves, it's going to be automated soon.

cc @carlossanlop