dotnet / dotnet/aspnetcore

Add missing ExpandoObject test coverage in ObjectVisitorTest

Open Beginner friendly
#68,933 1 comment 0 reactions 0 assignees View on GitHub
area-networking feature-json-patch
Dominant language
C#
Stars
38.4k
Forks
10.9k
Avg merge
2d 10h
Merged PRs (30d)
281

Description

### Is there an existing issue for this?

- [x] I have searched the existing issues

### Is your feature request related to a problem? Please describe the problem.

ObjectVisitorTest defines the ReturnsExpandoAdapterData data source, but there is currently no corresponding test method executing it to validate ExpandoObject traversal paths correctly.

### Describe the solution you'd like

Add a new unit test method utilizing [MemberData(nameof(ReturnsExpandoAdapterData))] to ensure proper test coverage for ExpandoObject scenarios in ObjectVisitorTest.

### Additional context

"Adding test coverage for existing ReturnsExpandoAdapterData data source in ObjectVisitorTest to ensure robust validation of dynamic types."

Contributor guide

Open the contributing guide

Research direction

Start in ObjectVisitorTest and inspect the existing ReturnsExpandoAdapterData data source. Add a unit test method using [MemberData(nameof(ReturnsExpandoAdapterData))] and confirm that the ExpandoObject traversal scenarios execute successfully.

Written by the indexing model from the issue text.

Assessment

Tech stack
csharp
Domain
testing
Issue type
Feature
Difficulty
2/5
Estimated time
1-3 hours
Activity status
Active
Clarity
Clearly specified
Newbie friendliness
85/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.