Skip to content

fix: stop FileStream.New(handle) from re-applying the handle's FileMode - #1098

Merged
vbreuss merged 2 commits into
mainfrom
fix/filestream-from-handle-keeps-content
Sep 27, 2026
Merged

vbreuss merged 2 commits into
mainfrom
fix/filestream-from-handle-keeps-content

Conversation

@vbreuss

@vbreuss vbreuss commented Sep 25, 2026

Copy link
Copy Markdown
Member

A stream created from a handle wraps the already open file, so replaying CreateNew, Create/Truncate or Append threw or wiped content written through the handle.

…eMode`

A stream created from a handle wraps the already open file, so replaying `CreateNew`, `Create`/`Truncate` or `Append` threw or wiped content written through the handle.
@vbreuss vbreuss self-assigned this Sep 25, 2026
Copilot AI lite review requested due to automatic review settings September 25, 2026 21:04

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Copilot was unable to review this pull request because the user who requested the review has reached their quota limit.

@vbreuss
vbreuss marked this pull request as draft September 25, 2026 21:11
@github-actions

Copy link
Copy Markdown

Test Results

     62 files  ±  0       62 suites  ±0   1h 20m 8s ⏱️ + 4m 8s
 63 338 tests +174   56 537 ✅ +174   6 801 💤 ±0  0 ❌ ±0 
147 426 runs  +180  128 437 ✅ +180  18 989 💤 ±0  0 ❌ ±0 

Results for commit 07cdd8a. ± Comparison against base commit 015a573.

This pull request removes 60105 and adds 60278 tests. Note that renamed tests count towards both.
  )
)
Testably.Abstractions.AccessControl.Tests.DirectoryInfoAclExtensionsTests ‑ SetAccessControl_ShouldNotUpdateTimes(07a46bab-dc7a-4c80-8200-460a3681588c)
Testably.Abstractions.AccessControl.Tests.DirectoryInfoAclExtensionsTests ‑ SetAccessControl_ShouldNotUpdateTimes(1c5f782d-9ffe-4807-9400-c2bd3a10bbe9)
Testably.Abstractions.AccessControl.Tests.DirectoryInfoAclExtensionsTests ‑ SetAccessControl_ShouldNotUpdateTimes(28a7f2f8-4327-4e6a-bc3e-0f1e2a24828f)
Testably.Abstractions.AccessControl.Tests.DirectoryInfoAclExtensionsTests ‑ SetAccessControl_ShouldNotUpdateTimes(332884ea-4147-463a-bfe4-8b03073fe23f)
Testably.Abstractions.AccessControl.Tests.DirectoryInfoAclExtensionsTests ‑ SetAccessControl_ShouldNotUpdateTimes(7b467c46-d364-4d22-92f0-38812ea655f6)
Testably.Abstractions.AccessControl.Tests.DirectoryInfoAclExtensionsTests ‑ SetAccessControl_ShouldNotUpdateTimes(861085aa-8637-45df-86e6-6af028d3e60e)
Testably.Abstractions.AccessControl.Tests.DirectoryInfoAclExtensionsTests ‑ SetAccessControl_ShouldNotUpdateTimes(932614a8-ee31-406b-b356-e6b49da4f34d)
Testably.Abstractions.AccessControl.Tests.DirectoryInfoAclExtensionsTests ‑ SetAccessControl_ShouldNotUpdateTimes(a55169b7-077d-4a8c-82c0-8c8272eb22a1)
…
Testably.Abstractions.AccessControl.Tests.DirectoryInfoAclExtensionsTests ‑ SetAccessControl_ShouldNotUpdateTimes(1b456256-f3d9-400d-87d9-800b34c2e48e)
Testably.Abstractions.AccessControl.Tests.DirectoryInfoAclExtensionsTests ‑ SetAccessControl_ShouldNotUpdateTimes(1c2e83a9-d455-4b23-b4e4-36cf09576b0e)
Testably.Abstractions.AccessControl.Tests.DirectoryInfoAclExtensionsTests ‑ SetAccessControl_ShouldNotUpdateTimes(451dcd94-2952-4a22-a614-e296fe0d9550)
Testably.Abstractions.AccessControl.Tests.DirectoryInfoAclExtensionsTests ‑ SetAccessControl_ShouldNotUpdateTimes(4d9ed0ce-77b2-4348-b481-91b3c0114e4e)
Testably.Abstractions.AccessControl.Tests.DirectoryInfoAclExtensionsTests ‑ SetAccessControl_ShouldNotUpdateTimes(4e17dfa2-0760-4306-af6f-1af50a896bb5)
Testably.Abstractions.AccessControl.Tests.DirectoryInfoAclExtensionsTests ‑ SetAccessControl_ShouldNotUpdateTimes(637fd240-f6b0-4fc9-9205-63811a0f7673)
Testably.Abstractions.AccessControl.Tests.DirectoryInfoAclExtensionsTests ‑ SetAccessControl_ShouldNotUpdateTimes(6455a536-a282-4464-8cad-1d195938e052)
Testably.Abstractions.AccessControl.Tests.DirectoryInfoAclExtensionsTests ‑ SetAccessControl_ShouldNotUpdateTimes(7969e7b1-5b49-492b-8b52-0350cbee6c50)
Testably.Abstractions.AccessControl.Tests.DirectoryInfoAclExtensionsTests ‑ SetAccessControl_ShouldNotUpdateTimes(892ca55f-0eec-470f-b6b1-599e53460b7e)
Testably.Abstractions.AccessControl.Tests.DirectoryInfoAclExtensionsTests ‑ SetAccessControl_ShouldNotUpdateTimes(90e8ff36-1ed7-4ef6-8d12-3070c6d6fbf4)
…
This pull request removes 6743 skipped tests and adds 6743 skipped tests. Note that renamed tests count towards both.
Testably.Abstractions.AccessControl.Tests.DirectoryInfoAclExtensionsTests ‑ SetAccessControl_ShouldNotUpdateTimes(07a46bab-dc7a-4c80-8200-460a3681588c)
Testably.Abstractions.AccessControl.Tests.DirectoryInfoAclExtensionsTests ‑ SetAccessControl_ShouldNotUpdateTimes(1c5f782d-9ffe-4807-9400-c2bd3a10bbe9)
Testably.Abstractions.AccessControl.Tests.DirectoryInfoAclExtensionsTests ‑ SetAccessControl_ShouldNotUpdateTimes(28a7f2f8-4327-4e6a-bc3e-0f1e2a24828f)
Testably.Abstractions.AccessControl.Tests.DirectoryInfoAclExtensionsTests ‑ SetAccessControl_ShouldNotUpdateTimes(332884ea-4147-463a-bfe4-8b03073fe23f)
Testably.Abstractions.AccessControl.Tests.DirectoryInfoAclExtensionsTests ‑ SetAccessControl_ShouldNotUpdateTimes(7b467c46-d364-4d22-92f0-38812ea655f6)
Testably.Abstractions.AccessControl.Tests.DirectoryInfoAclExtensionsTests ‑ SetAccessControl_ShouldNotUpdateTimes(861085aa-8637-45df-86e6-6af028d3e60e)
Testably.Abstractions.AccessControl.Tests.DirectoryInfoAclExtensionsTests ‑ SetAccessControl_ShouldNotUpdateTimes(932614a8-ee31-406b-b356-e6b49da4f34d)
Testably.Abstractions.AccessControl.Tests.DirectoryInfoAclExtensionsTests ‑ SetAccessControl_ShouldNotUpdateTimes(a55169b7-077d-4a8c-82c0-8c8272eb22a1)
Testably.Abstractions.AccessControl.Tests.DirectoryInfoAclExtensionsTests ‑ SetAccessControl_ShouldNotUpdateTimes(b7dcf365-9a2d-41e7-a3eb-6542f684a49d)
Testably.Abstractions.AccessControl.Tests.DirectoryInfoAclExtensionsTests ‑ SetAccessControl_ShouldNotUpdateTimes(bb1eb49e-f407-48a1-a31f-93a9fad83883)
…
Testably.Abstractions.AccessControl.Tests.DirectoryInfoAclExtensionsTests ‑ SetAccessControl_ShouldNotUpdateTimes(1b456256-f3d9-400d-87d9-800b34c2e48e)
Testably.Abstractions.AccessControl.Tests.DirectoryInfoAclExtensionsTests ‑ SetAccessControl_ShouldNotUpdateTimes(1c2e83a9-d455-4b23-b4e4-36cf09576b0e)
Testably.Abstractions.AccessControl.Tests.DirectoryInfoAclExtensionsTests ‑ SetAccessControl_ShouldNotUpdateTimes(451dcd94-2952-4a22-a614-e296fe0d9550)
Testably.Abstractions.AccessControl.Tests.DirectoryInfoAclExtensionsTests ‑ SetAccessControl_ShouldNotUpdateTimes(4d9ed0ce-77b2-4348-b481-91b3c0114e4e)
Testably.Abstractions.AccessControl.Tests.DirectoryInfoAclExtensionsTests ‑ SetAccessControl_ShouldNotUpdateTimes(4e17dfa2-0760-4306-af6f-1af50a896bb5)
Testably.Abstractions.AccessControl.Tests.DirectoryInfoAclExtensionsTests ‑ SetAccessControl_ShouldNotUpdateTimes(637fd240-f6b0-4fc9-9205-63811a0f7673)
Testably.Abstractions.AccessControl.Tests.DirectoryInfoAclExtensionsTests ‑ SetAccessControl_ShouldNotUpdateTimes(6455a536-a282-4464-8cad-1d195938e052)
Testably.Abstractions.AccessControl.Tests.DirectoryInfoAclExtensionsTests ‑ SetAccessControl_ShouldNotUpdateTimes(7969e7b1-5b49-492b-8b52-0350cbee6c50)
Testably.Abstractions.AccessControl.Tests.DirectoryInfoAclExtensionsTests ‑ SetAccessControl_ShouldNotUpdateTimes(892ca55f-0eec-470f-b6b1-599e53460b7e)
Testably.Abstractions.AccessControl.Tests.DirectoryInfoAclExtensionsTests ‑ SetAccessControl_ShouldNotUpdateTimes(90e8ff36-1ed7-4ef6-8d12-3070c6d6fbf4)
…

@vbreuss
vbreuss marked this pull request as ready for review September 27, 2026 05:02
@vbreuss
vbreuss enabled auto-merge (squash) September 27, 2026 05:02
@vbreuss
vbreuss disabled auto-merge September 27, 2026 05:02
@vbreuss
vbreuss merged commit c83487b into main Sep 27, 2026
9 of 11 checks passed
@vbreuss
vbreuss deleted the fix/filestream-from-handle-keeps-content branch September 27, 2026 05:02
@sonarqubecloud

Copy link
Copy Markdown

@github-actions

Copy link
Copy Markdown

This is addressed in release v7.1.0.

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

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants