ObjectContainer.IsRegistered<T>() does not walk up the inheritance chain, even though Resolve<T>() does, giving misleading results

[DrEsteban]

SpecFlow Version

3.9.74

Which test runner are you using?

xUnit

Test Runner Version Number

2.4.1

.NET Implementation

.NET 8.0

Project Format of the SpecFlow project

Sdk-style project format

.feature.cs files are generated using

SpecFlow.Tools.MsBuild.Generation NuGet package

Test Execution Method

Visual Studio Test Explorer

SpecFlow Section in app.config or content of specflow.json

{
"$schema": "https://specflow.org/specflow-config.json",
"stepAssemblies": [
{
"assembly": "<my_common_assembly>"
}
]
}

Issue Description

I am registering services in the Global container in a [BeforeTestRun] method due to Issue #2580.

[BeforeTestRun(Order = 10)]
public static void SetEnvironmentConfiguration(ObjectContainer container)
{
    // Registering to the base container is necessary due to an integration issue with XUnit:
    // https://github.com/SpecFlowOSS/SpecFlow/issues/2580
    container.BaseContainer.RegisterInstanceAs<IEnvironmentConstantsProvider>(EnvironmentConstants.Instance);
}

In another portion of my project, I am attempting to proactively check whether this dependency has been registered so I can give a helpful error message:

if (!container.IsRegistered<IEnvironmentConstantsProvider>())
{
    throw new Exception($"{nameof(IEnvironmentConstantsProvider)} not registered in container. See <docs_on_how_to_configure_correctly>");
}

var envProvider = container.Resolve<IEnvironmentConstantsProvider>();

This call to IsRegistered() is returning false, even though the object is registered, just not at the hierarchy level of the current ObjectContainer... If I comment out my if check, the object resolves as expected because Resolve() actually walks up the inheritance chain to find an object. But it seems as if IsRegistered() simply stops and returns false if it isn't in the immediate scope of the current ObjectContainer - which is not what I'd expect as a user.

NOTE: I've devised a workaround by introducing an extension method as follows, which works:

public static bool IsRegisteredAtAnyLevel<T>(this IObjectContainer container)
{
    do
    {
        if (container.IsRegistered<T>())
        {
            return true;
        }
    } while (container is ObjectContainer c && (container = c.BaseContainer) != null);

    return false;
}

Steps to Reproduce

1. Register an object at the TestThread or Feature level.
2. Call container.IsRegistered<MyObj>() at the scenario level => FAIL
3. Call container.Resolve<MyObj>() at the scenario level => SUCCESS

My issue description also provides a decent description of the repro.

Link to Repro Project

Closed source project

[DrEsteban]

FYI - I'd also recommend updating your issue template to include .NET 8.0. The max it allows right now is .NET 6.0

[DrEsteban]

Existing issue on BoDi: SpecFlowOSS/BoDi#18

[gkalnytskyi-woolworthslimited]

@DrEsteban, I would suggest checking Reqnroll for a compatible alternative and check if this bug is reproducible there, as this project hasn't been updated in more than a year.

[DrEsteban]

😯 I didn't realize! Thanks for the recommendation @gkalnytskyi-woolworthslimited

[DeLaphante]

I would advised all to not use Specflow+ Runner or LivingDoc as its currently not being supported or developed - migrate to MsTest or Nunit and also update to latest .net 8+ and use third party reporting libraries such as extent report.

Specflow is working perfectly fine for me with the above changes in the following daily github actions repo -> https://github.com/DeLaphante/CynkyAutomation/actions