You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
This pull request introduces improvements to the handling and testing of property setup snapshots in the MockRegistry class. The main changes include adding a new internal accessor for property setup snapshots, enhancing test coverage for property setup behaviors, and adding comments to clarify code paths for mutation testing tools.
Property setup snapshot improvements
Added a new internal method GetPropertySetupSnapshot(int memberId) to MockRegistry.Interactions.cs for retrieving the current property setup snapshot by member ID. This method is intended for diagnostics and direct verification in tests.
Test coverage enhancements
Added multiple new unit tests in MockRegistrySetupSnapshotTests.cs to verify correct behavior of property setup snapshotting, including:
Ensuring earlier entries are preserved when resizing the property setup table.
Verifying that default setups overwrite previous defaults, but user setups are retained over defaults.
Confirming that setups are correctly published to the snapshot table for both direct and scenario-based registrations.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
coverageAdditional tests for edge cases or improvement of test coverage
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
This pull request introduces improvements to the handling and testing of property setup snapshots in the
MockRegistryclass. The main changes include adding a new internal accessor for property setup snapshots, enhancing test coverage for property setup behaviors, and adding comments to clarify code paths for mutation testing tools.Property setup snapshot improvements
GetPropertySetupSnapshot(int memberId)toMockRegistry.Interactions.csfor retrieving the current property setup snapshot by member ID. This method is intended for diagnostics and direct verification in tests.Test coverage enhancements
MockRegistrySetupSnapshotTests.csto verify correct behavior of property setup snapshotting, including: