Skip to content

Fix #2474: DirectoryInfo/FileInfo display property map prevents infinite loop#2692

Merged
nohwnd merged 1 commit intopester:mainfrom
nohwnd:fix-2474-directoryinfo
Apr 4, 2026
Merged

Fix #2474: DirectoryInfo/FileInfo display property map prevents infinite loop#2692
nohwnd merged 1 commit intopester:mainfrom
nohwnd:fix-2474-directoryinfo

Conversation

@nohwnd
Copy link
Copy Markdown
Member

@nohwnd nohwnd commented Apr 3, 2026

Fix #2474

Added System.IO.DirectoryInfo and System.IO.FileInfo to the display property map in Get-DisplayProperty2 to prevent infinite recursion from circular Root/Directory properties.

Copilot-generated fix.

Co-authored-by: Copilot 223556219+Copilot@users.noreply.github.com

@nohwnd
Copy link
Copy Markdown
Member Author

nohwnd commented Apr 3, 2026

enforce this by adding tests. there should be already some for process, if not, add also for process.

… infinite loop

Copilot-generated fix.

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
@nohwnd nohwnd force-pushed the fix-2474-directoryinfo branch from d8ce204 to 90e4958 Compare April 3, 2026 10:13
@nohwnd nohwnd marked this pull request as ready for review April 3, 2026 11:44
@nohwnd nohwnd added the 🚢 Ship it! When copilot cli helps to develop, I cannot approve. So this is sign that this is good to go. label Apr 3, 2026
@nohwnd
Copy link
Copy Markdown
Member Author

nohwnd commented Apr 3, 2026

🤖 Tests added in latest push — includes DirectoryInfo, FileInfo, and Process display properties. Force-pushed.

@nohwnd nohwnd merged commit 2f5333a into pester:main Apr 4, 2026
13 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

🚢 Ship it! When copilot cli helps to develop, I cannot approve. So this is sign that this is good to go.

Projects

None yet

1 participant