VSoftTechnologies/DUnitX

Incorrect XML report generated when registering more than one fixture declared in a non-namespaced unit (fix included in the issue)

Opened this issue · 2 comments

if not tmpFixtures.TryGetValue(fixtureNamespace,fixture) then

This

if not tmpFixtures.TryGetValue(fixtureNamespace,parentFixture) then

should fix it (the parentFixture will be filled with an existing fixture after the first iteration for a fixture declared in a non-namespaced unit).

Possibly also replace the following check:

if parentFixture = nil then

By an assertion like

System.Assert(Assigned(parentFixture));

since I believe it would always be assigned.

Thanks, fixed.