Skip to content

fix: incorrectly thrown exception in Directory.CreateSymbolicLink - #1026

Merged
mergify[bot] merged 6 commits into
TestableIO:mainfrom
vbreuss:topic/vb/fix-incorrectly-thrown-exception
Aug 25, 2023
Merged

fix: incorrectly thrown exception in Directory.CreateSymbolicLink#1026
mergify[bot] merged 6 commits into
TestableIO:mainfrom
vbreuss:topic/vb/fix-incorrectly-thrown-exception

Conversation

@vbreuss

Copy link
Copy Markdown
Member

Remove incorrectly thrown exception when the target does not exist in Directory.CreateSymbolicLink. The real file system does not throw an exception in this case.

See here for a succeeding test against the real file system.

@fgreinacherfgreinacher left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks! Should we indicate a breaking change?

@vbreuss

Copy link
Copy Markdown
MemberAuthor

@fgreinacher
I consider this a bug, as our goal is to mimic the behaviour of the real file system. As we did not change any interface, I would not consider this a breaking change.

@fgreinacher

Copy link
Copy Markdown
Contributor

Technically it's still a breaking change as folks might have relied on the exception. But I agree that in this case it's pretty unlikely.

@mergify
mergifyBot merged commit ebb5476 into TestableIO:mainAug 25, 2023
@vbreuss
vbreuss deleted the topic/vb/fix-incorrectly-thrown-exception branch August 25, 2023 12:25
mergifyBot referenced this pull request in Testably/Testably.Abstractions Aug 25, 2023
…19.2.67 (#390)
[![Mend
Renovate](https://app.renovatebot.com/images/banner.svg)](https://renovatebot.com)
This PR contains the following updates:
| Package | Type | Update | Change |
|---|---|---|---|
|
[TestableIO.System.IO.Abstractions](https://togithub.com/TestableIO/System.IO.Abstractions)
| nuget | patch | `19.2.64` -> `19.2.67` |
---
### Release Notes
<details>
<summary>TestableIO/System.IO.Abstractions
(TestableIO.System.IO.Abstractions)</summary>
###
[`v19.2.67`](https://togithub.com/TestableIO/System.IO.Abstractions/releases/tag/v19.2.67)
#### What's Changed
- fix: incorrectly thrown exception in `Directory.CreateSymbolicLink` by
[@&#8203;vbreuss](https://togithub.com/vbreuss) in
[https://github.com/TestableIO/System.IO.Abstractions/pull/1026](https://togithub.com/TestableIO/System.IO.Abstractions/pull/1026)
**Full Changelog**:
TestableIO/System.IO.Abstractions@v19.2.66...v19.2.67
###
[`v19.2.66`](https://togithub.com/TestableIO/System.IO.Abstractions/releases/tag/v19.2.66)
#### What's Changed
- chore(deps): update dependency githubactionstestlogger to v2.3.3 by
[@&#8203;renovate](https://togithub.com/renovate) in
[https://github.com/TestableIO/System.IO.Abstractions/pull/1031](https://togithub.com/TestableIO/System.IO.Abstractions/pull/1031)
- fix: incorrect thrown exception when calling `Path.GetFullPath` with
white-space by [@&#8203;vbreuss](https://togithub.com/vbreuss) in
[https://github.com/TestableIO/System.IO.Abstractions/pull/1030](https://togithub.com/TestableIO/System.IO.Abstractions/pull/1030)
**Full Changelog**:
TestableIO/System.IO.Abstractions@v19.2.64...v19.2.66
</details>
---
### Configuration
📅 **Schedule**: Branch creation - At any time (no schedule defined),
Automerge - At any time (no schedule defined).
🚦 **Automerge**: Disabled by config. Please merge this manually once you
are satisfied.
♻ **Rebasing**: Whenever PR becomes conflicted, or you tick the
rebase/retry checkbox.
🔕 **Ignore**: Close this PR and you won't be reminded about this update
again.
---
- [ ] <!-- rebase-check -->If you want to rebase/retry this PR, check
this box
---
This PR has been generated by [Mend
Renovate](https://www.mend.io/free-developer-tools/renovate/). View
repository job log
[here](https://developer.mend.io/github/Testably/Testably.Abstractions).
<!--renovate-debug:eyJjcmVhdGVkSW5WZXIiOiIzNi41Ni4wIiwidXBkYXRlZEluVmVyIjoiMzYuNTYuMCIsInRhcmdldEJyYW5jaCI6Im1haW4ifQ==-->
Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>
@github-actions

Copy link
Copy Markdown

This is addressed in release v19.2.67.

Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Labels

state: releasedIssues that are released

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants

@vbreuss@fgreinacher@vbtig