◐ Shell
clean mode source ↗

Add comprehensive PowerShell class tests for ConvertTo-Json by yotsuda · Pull Request #26769 · PowerShell/PowerShell

@yotsuda

AI review requested due to automatic review settings

February 10, 2026 08:07

@iSazonov iSazonov added the CL-Test

Indicates that a PR should be marked as a test change in the Change Log

label

Feb 10, 2026

iSazonov

Replace multiple small classes with a single SimpleClass covering all property
types (primitive, BigInt, Guid, IPAddress, array, dictionary, hidden, ETS).
Remove constructor, method, and separate ETS tests. 14 tests -> 7 tests.

iSazonov

Change all Phase 5 assertions from Match to BeExactly. Use OrderedDictionary
for deterministic key order. Add ETS NoteProperty to nested IPVal member.

iSazonov

@iSazonov iSazonov added the Approved-LowRisk

Indicate a PR has been approved and can be merged after a quick review of another maintainer.

label

Feb 19, 2026

daxian-dbw

@yotsuda yotsuda deleted the add-convertto-json-class-tests branch

February 20, 2026 06:23

JustinGrote pushed a commit to JustinGrote/PowerShell that referenced this pull request

Jun 2, 2026