Browser with.Dependencies(..) broken?

[PureKrome]

Notice: I'm not sure if this issue is related to #1862 or this is a TinyIOC issue and should be raised there.

When testing a route using Nancy.Testing and using with.Dependencies(someCollectionOfDependencies); it fails to correctly set them up.

As a work around, I need to iterate through each one and with.Dependency(someDependency); and that works.

for example ..

Fails

var browser = new Browser(with =>
{
    with.Module<MyModule>();
    with.Dependencies(dependencies); // a collection of `x` dep's.
});

Succeeds

var browser = new Browser(with =>
{
    with.Module<MyModule>();
    foreach(var dependency in dependencies)
    {
        with.Dependency(dependency);
    }
});

btw, the collection dependencies are identical.

Lastly, this is with version 1.2 of Nancy, etc.

[horsdal]

On a quick first glance it doesn't seem related to #1862. It could be another issue in ConfigurableBootsttrapper though.

[PureKrome]

👍 Ok, is this enough info for you @horsdal (or anyone else on the team), right now?

[khellang]

@PureKrome I don't know how that method has ever worked. The method takes a params object[] dependencies, loops over them and calls this.Dependency<T>(dependency) where T is inferred to object 😕 I'll send a PR to fix it. In the meantime you should explicitly invoke the Dependencies<T> overload.

[PureKrome]

where T is inferred to object

AH! I was hoping that was happening ... based on the error message :) Awesome!

Sweet as candy!