Skip to content

[00036] Add VeryLargeStateTree to JsonApplyBenchmarks params and update tests#15

Merged
rorychatt merged 2 commits into
mainfrom
plan-00036-Ivy.NativeJsonDiff
Apr 12, 2026
Merged

[00036] Add VeryLargeStateTree to JsonApplyBenchmarks params and update tests#15
rorychatt merged 2 commits into
mainfrom
plan-00036-Ivy.NativeJsonDiff

Conversation

@rorychatt

Copy link
Copy Markdown
Collaborator

Summary

Added VeryLargeStateTree to JsonApplyBenchmarks.cs [Params] attribute (was missing because plan 00026 created the file after plan 00025 added the category). Updated JsonPayloadGeneratorTests.cs with a new [InlineData] entry, a size assertion test for ~1MB payloads (adjusted from plan's ~2MB estimate to match actual ~944KB generator output), and corrected the AllCategories.Length assertion from 8 to 9.

Files Modified

  • Benchmarks: src/Ivy.NativeJsonDiff.Benchmarks/Benchmarks/JsonApplyBenchmarks.cs — added VeryLargeStateTree to [Params]
  • Tests: src/Ivy.NativeJsonDiff.Tests/JsonPayloadGeneratorTests.cs — added [InlineData("VeryLargeStateTree")], Generate_VeryLargeStateTree_HasExpectedSize test, Assert.Contains("VeryLargeStateTree", ...), updated count from 8 to 9

Commits

  • c5cba0f [00036] Add VeryLargeStateTree to JsonApplyBenchmarks params and update tests
  • cfea59b [00036] Fix VeryLargeStateTree size assertion range to match actual ~1MB payload

@rorychatt
rorychatt merged commit 000ee37 into main Apr 12, 2026
2 checks passed
@rorychatt
rorychatt deleted the plan-00036-Ivy.NativeJsonDiff branch April 12, 2026 10:55
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant