Exception that a constructor parameter doesn't exist when it does

Question

I am trying to inject through the use of Unity two objects based on an interface into a class constructor.

I'm currently receiving the following error when unit testing:

Result Message:
Test method TestProject.TFStests.Check_Interface_CheckOut_Method threw exception: System.InvalidOperationException: The type Adp.Tools.VersionControl.TfsVersionControl.TfsVcPromotionManager does not have a constructor that takes the parameters (TfsVcQaCheckoutWorker).

The following code is my Unity class, this is used to register and resolve the TfsVCPromotionManager object:

public class UnityClass
{
    public static ITfsVcPromotionManager returnNewPromotionManager(
       VersionControlServer tfServer)
    {
        var container = new UnityContainer();

        ITfsVcQaCheckinWorker test1 = CreateUnityCheckInWorker();
        ITfsVcQaCheckoutWorker test2 = CreateUnityCheckOutWorker(tfServer);

        container.RegisterType<ITfsVcPromotionManager, TfsVcPromotionManager>(
            new InjectionConstructor(test2), new InjectionConstructor(test1));
        return container.Resolve<TfsVcPromotionManager>();
    }

    private static ITfsVcQaCheckinWorker CreateUnityCheckInWorker()
    {
        var container = new UnityContainer();

        container.RegisterType<ITfsVcQaCheckinWorker, ITfsVcQaCheckinWorker>();
        return container.Resolve<TfsVcQaCheckinWorker>();
    }

    private static ITfsVcQaCheckoutWorker CreateUnityCheckOutWorker(
        VersionControlServer passedServer)
    {
        var container = new UnityContainer();

        container.RegisterType<ITfsVcQaCheckoutWorker, TfsVcQaCheckoutWorker>(
            new InjectionConstructor(passedServer));
        return container.Resolve<TfsVcQaCheckoutWorker>();
    }
}

This is the constrcutor os the TfsVcPromotionManager class. note that it clearly takes in an instance based on the interfaces ITfsVcQaCheckoutworker and ITfsVcCheckinWorker.

 private ITfsVcQaCheckoutWorker _checkOutWorker;

    private ITfsVcQaCheckinWorker _checkInWorker;

    public TfsVcPromotionManager(ITfsVcQaCheckoutWorker checkOutWorker,
                                 ITfsVcQaCheckinWorker checkInWorker)
    {
        if (checkOutWorker == null || checkInWorker == null)
        {
            throw new NullReferenceException();
        }

        _checkOutWorker = checkOutWorker;
        _checkInWorker = checkInWorker;
    }

Can anyone give me an indication of what I'm doing wrong.

Jeff B · Accepted Answer · 2014-01-03 18:05:28Z

5

It says it can't find a "constructor that takes the parameters (TfsVcQaCheckoutWorker)", which is indeed a true statement. There is no constructor that takes just one parameter. I'm not terrible familiar with Unity, but I bet your problem would be solved by changing this:

container.RegisterType<ITfsVcPromotionManager, TfsVcPromotionManager>(
    new InjectionConstructor(test2),
    new InjectionConstructor(test1))

To this:

container.RegisterType<ITfsVcPromotionManager, TfsVcPromotionManager>(
    new InjectionConstructor(test2, test1))

See the MSDN Reference for the InjectionConstructor's constructor.

edited Jan 3, 2014 at 18:05

answered Jan 3, 2014 at 17:22

Jeff B

9,14220 gold badges68 silver badges146 bronze badges

Sign up to request clarification or add additional context in comments.

1 Comment

user1352057 Over a year ago

Many thanks Jeff Bridgman. That worked perfectly. I'm a little embarrassed it missed that.

Olivier Jacot-Descombes · Accepted Answer · 2014-01-03 18:07:54Z

4

I believe you should register container with several params the other way. Have you tried this?

container.RegisterType<ITfsVcPromotionManager, TfsVcPromotionManager>(
    new InjectionConstructor(test2, test1)
);

edited Jan 3, 2014 at 18:07

Olivier Jacot-Descombes

114k14 gold badges149 silver badges202 bronze badges

answered Jan 3, 2014 at 17:22

Andrei

56.8k10 gold badges92 silver badges114 bronze badges

Collectives™ on Stack Overflow

Exception that a constructor parameter doesn't exist when it does

2 Answers 2

1 Comment

Comments

Your Answer

Hot Network Questions

Collectives™ on Stack Overflow

2 Answers 2

1 Comment

Comments

Your Answer

Sign up or log in

Post as a guest

Related