dennisdoomen / fluentassertions / 24312902651

97%

develop: 97%

Build Type

push

github

Committed by dennisdoomen

Commit Message

Cache Node.Depth to avoid repeated string scanning

Node.Depth previously computed the depth on every call by counting '.' characters
in Subject.PathAndName using LINQ. This method is called in ShouldContinueThisDeep
for every AssertEquivalencyOf invocation, meaning it runs thousands of times during
a typical structural comparison.

The result is now memoised in a private int field (initialised to -1) and
recomputed only when AdjustForRemappedSubject changes the underlying path.

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

Coverage Stats

4079 of 4356 branches covered (93.64%)

Branch coverage included in aggregate %.

5 of 5 new or added lines in 1 file covered. (100.0%)

12 existing lines in 6 files now uncovered.

13155 of 13380 relevant lines covered (98.32%)

66164.21 hits per line