Skip to content

Conversation

@vbreuss
Copy link
Member

@vbreuss vbreuss commented Mar 13, 2025

Fixes #763

The file stream used to create the zip file was not disposed correctly.

@vbreuss vbreuss added the bug Something isn't working label Mar 13, 2025
@vbreuss vbreuss self-assigned this Mar 13, 2025
@sonarqubecloud
Copy link

@github-actions
Copy link

Test Results

     52 files  ± 0       52 suites  ±0   23m 7s ⏱️ -3s
 58 635 tests +10   55 605 ✅ +10   3 030 💤 ±0  0 ❌ ±0 
121 629 runs  +47  108 998 ✅ +47  12 631 💤 ±0  0 ❌ ±0 

Results for commit 286843f. ± Comparison against base commit c8d2c32.

This pull request removes 50759 and adds 50769 tests. Note that renamed tests count towards both.
Testably.Abstractions.AccessControl.Tests.DirectoryInfoAclExtensionsTests+MockFileSystemTests ‑ SetAccessControl_ShouldNotUpdateTimes(path: "path29220496-51f0-4b60-b58c-ce6febf43242")
Testably.Abstractions.AccessControl.Tests.DirectoryInfoAclExtensionsTests+MockFileSystemTests ‑ SetAccessControl_ShouldNotUpdateTimes(path: "path3cbb3e34-df65-424a-accf-5c966b8bdb6d")
Testably.Abstractions.AccessControl.Tests.DirectoryInfoAclExtensionsTests+MockFileSystemTests ‑ SetAccessControl_ShouldNotUpdateTimes(path: "path4fa314ad-1c3c-43e5-b659-9cfe59dacfbb")
Testably.Abstractions.AccessControl.Tests.DirectoryInfoAclExtensionsTests+MockFileSystemTests ‑ SetAccessControl_ShouldNotUpdateTimes(path: "path6150778d-fc81-4b0e-a2c6-83cdcb0c83ac")
Testably.Abstractions.AccessControl.Tests.DirectoryInfoAclExtensionsTests+MockFileSystemTests ‑ SetAccessControl_ShouldNotUpdateTimes(path: "path62dec50c-33bc-479f-ae44-15cf145846e3")
Testably.Abstractions.AccessControl.Tests.DirectoryInfoAclExtensionsTests+MockFileSystemTests ‑ SetAccessControl_ShouldNotUpdateTimes(path: "path64e4287f-fdad-42bd-90f4-5a94bd1d01f4")
Testably.Abstractions.AccessControl.Tests.DirectoryInfoAclExtensionsTests+MockFileSystemTests ‑ SetAccessControl_ShouldNotUpdateTimes(path: "path8ef452b0-111b-4837-9189-2b067e2caad0")
Testably.Abstractions.AccessControl.Tests.DirectoryInfoAclExtensionsTests+MockFileSystemTests ‑ SetAccessControl_ShouldNotUpdateTimes(path: "path9102b486-c061-464d-9c9c-18441b6fdbd5")
Testably.Abstractions.AccessControl.Tests.DirectoryInfoAclExtensionsTests+MockFileSystemTests ‑ SetAccessControl_ShouldNotUpdateTimes(path: "pathac33943d-0cd3-4b7c-9965-634b10ebd9bc")
Testably.Abstractions.AccessControl.Tests.DirectoryInfoAclExtensionsTests+MockFileSystemTests ‑ SetAccessControl_ShouldNotUpdateTimes(path: "pathd7fbec20-e3d3-46ec-9729-7457a8f2e3b4")
…
Testably.Abstractions.AccessControl.Tests.DirectoryInfoAclExtensionsTests+MockFileSystemTests ‑ SetAccessControl_ShouldNotUpdateTimes(path: "path1a0ffde9-cbd9-4d07-884f-a52ef9f18371")
Testably.Abstractions.AccessControl.Tests.DirectoryInfoAclExtensionsTests+MockFileSystemTests ‑ SetAccessControl_ShouldNotUpdateTimes(path: "path57496bb6-37a8-4eb3-b746-2450833bbe2b")
Testably.Abstractions.AccessControl.Tests.DirectoryInfoAclExtensionsTests+MockFileSystemTests ‑ SetAccessControl_ShouldNotUpdateTimes(path: "path81ec2d3f-c488-437a-8fae-1a96b886243a")
Testably.Abstractions.AccessControl.Tests.DirectoryInfoAclExtensionsTests+MockFileSystemTests ‑ SetAccessControl_ShouldNotUpdateTimes(path: "path8e4dfcaf-af0f-48a1-817f-defb148ff3f8")
Testably.Abstractions.AccessControl.Tests.DirectoryInfoAclExtensionsTests+MockFileSystemTests ‑ SetAccessControl_ShouldNotUpdateTimes(path: "path9cf0dcf0-a1be-4a80-b22e-7f6f997b305f")
Testably.Abstractions.AccessControl.Tests.DirectoryInfoAclExtensionsTests+MockFileSystemTests ‑ SetAccessControl_ShouldNotUpdateTimes(path: "pathb5c79dff-a9c8-48ce-8e33-eb3e926f2e9c")
Testably.Abstractions.AccessControl.Tests.DirectoryInfoAclExtensionsTests+MockFileSystemTests ‑ SetAccessControl_ShouldNotUpdateTimes(path: "pathbe8cd3f3-c399-4c89-a574-e089a62b5483")
Testably.Abstractions.AccessControl.Tests.DirectoryInfoAclExtensionsTests+MockFileSystemTests ‑ SetAccessControl_ShouldNotUpdateTimes(path: "pathd6f077f1-a945-496b-9b7a-01212b43132a")
Testably.Abstractions.AccessControl.Tests.DirectoryInfoAclExtensionsTests+MockFileSystemTests ‑ SetAccessControl_ShouldNotUpdateTimes(path: "pathe4b63004-915a-4c6b-9e6f-0910baf983b3")
Testably.Abstractions.AccessControl.Tests.DirectoryInfoAclExtensionsTests+MockFileSystemTests ‑ SetAccessControl_ShouldNotUpdateTimes(path: "pathe58bc79a-521d-4bf3-9dbc-1ac11de6f19a")
…
This pull request removes 2411 skipped tests and adds 2411 skipped tests. Note that renamed tests count towards both.
Testably.Abstractions.AccessControl.Tests.DirectoryInfoAclExtensionsTests+MockFileSystemTests ‑ SetAccessControl_ShouldNotUpdateTimes(path: "path29220496-51f0-4b60-b58c-ce6febf43242")
Testably.Abstractions.AccessControl.Tests.DirectoryInfoAclExtensionsTests+MockFileSystemTests ‑ SetAccessControl_ShouldNotUpdateTimes(path: "path3cbb3e34-df65-424a-accf-5c966b8bdb6d")
Testably.Abstractions.AccessControl.Tests.DirectoryInfoAclExtensionsTests+MockFileSystemTests ‑ SetAccessControl_ShouldNotUpdateTimes(path: "path6150778d-fc81-4b0e-a2c6-83cdcb0c83ac")
Testably.Abstractions.AccessControl.Tests.DirectoryInfoAclExtensionsTests+MockFileSystemTests ‑ SetAccessControl_ShouldNotUpdateTimes(path: "path62dec50c-33bc-479f-ae44-15cf145846e3")
Testably.Abstractions.AccessControl.Tests.DirectoryInfoAclExtensionsTests+MockFileSystemTests ‑ SetAccessControl_ShouldNotUpdateTimes(path: "path9102b486-c061-464d-9c9c-18441b6fdbd5")
Testably.Abstractions.AccessControl.Tests.DirectoryInfoAclExtensionsTests+MockFileSystemTests ‑ SetAccessControl_ShouldNotUpdateTimes(path: "pathac33943d-0cd3-4b7c-9965-634b10ebd9bc")
Testably.Abstractions.AccessControl.Tests.DirectoryInfoAclExtensionsTests+RealFileSystemTests ‑ SetAccessControl_ShouldNotUpdateTimes(path: "path02520356-89c0-4879-b499-175e2e990b16")
Testably.Abstractions.AccessControl.Tests.DirectoryInfoAclExtensionsTests+RealFileSystemTests ‑ SetAccessControl_ShouldNotUpdateTimes(path: "path830e5c55-3f74-4070-9936-65dba000f46a")
Testably.Abstractions.AccessControl.Tests.DirectoryInfoAclExtensionsTests+RealFileSystemTests ‑ SetAccessControl_ShouldNotUpdateTimes(path: "path84eb8002-922e-4000-81bf-52be39ac96f0")
Testably.Abstractions.AccessControl.Tests.DirectoryInfoAclExtensionsTests+RealFileSystemTests ‑ SetAccessControl_ShouldNotUpdateTimes(path: "path9d7ded0a-a554-4686-a7bc-1535e29deab6")
…
Testably.Abstractions.AccessControl.Tests.DirectoryInfoAclExtensionsTests+MockFileSystemTests ‑ SetAccessControl_ShouldNotUpdateTimes(path: "path1a0ffde9-cbd9-4d07-884f-a52ef9f18371")
Testably.Abstractions.AccessControl.Tests.DirectoryInfoAclExtensionsTests+MockFileSystemTests ‑ SetAccessControl_ShouldNotUpdateTimes(path: "path57496bb6-37a8-4eb3-b746-2450833bbe2b")
Testably.Abstractions.AccessControl.Tests.DirectoryInfoAclExtensionsTests+MockFileSystemTests ‑ SetAccessControl_ShouldNotUpdateTimes(path: "path81ec2d3f-c488-437a-8fae-1a96b886243a")
Testably.Abstractions.AccessControl.Tests.DirectoryInfoAclExtensionsTests+MockFileSystemTests ‑ SetAccessControl_ShouldNotUpdateTimes(path: "pathd6f077f1-a945-496b-9b7a-01212b43132a")
Testably.Abstractions.AccessControl.Tests.DirectoryInfoAclExtensionsTests+MockFileSystemTests ‑ SetAccessControl_ShouldNotUpdateTimes(path: "pathe4b63004-915a-4c6b-9e6f-0910baf983b3")
Testably.Abstractions.AccessControl.Tests.DirectoryInfoAclExtensionsTests+MockFileSystemTests ‑ SetAccessControl_ShouldNotUpdateTimes(path: "pathe58bc79a-521d-4bf3-9dbc-1ac11de6f19a")
Testably.Abstractions.AccessControl.Tests.DirectoryInfoAclExtensionsTests+RealFileSystemTests ‑ SetAccessControl_ShouldNotUpdateTimes(path: "path37bc1989-0ef3-48c5-a100-914aa5fe066b")
Testably.Abstractions.AccessControl.Tests.DirectoryInfoAclExtensionsTests+RealFileSystemTests ‑ SetAccessControl_ShouldNotUpdateTimes(path: "path3dac0e39-06dd-4b39-b1bd-23fc8e77b602")
Testably.Abstractions.AccessControl.Tests.DirectoryInfoAclExtensionsTests+RealFileSystemTests ‑ SetAccessControl_ShouldNotUpdateTimes(path: "path6a4b1034-a9ab-4a2d-bd0e-3e97a0d38ccc")
Testably.Abstractions.AccessControl.Tests.DirectoryInfoAclExtensionsTests+RealFileSystemTests ‑ SetAccessControl_ShouldNotUpdateTimes(path: "path98ac3a5f-333e-4e2b-b8c8-902037077b8b")
…

@vbreuss vbreuss merged commit 8a96799 into main Mar 13, 2025
15 checks passed
@vbreuss vbreuss deleted the topic/fix-lock-in-zipfile-createfromdirectory branch March 13, 2025 10:02
@github-actions
Copy link

This is addressed in release v4.0.1.

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

Labels

bug Something isn't working state: released

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Zipped file remains locked after ZipFile().CreateFromDirectory

2 participants