Make `new MutationContext<T>` injectable. about dado.componentmodel.mutations HOT 3 OPEN

Please forgive me. Dependency injection has come around since I've had any major dotnet projects, so I'm not very familiar with the concept. May ask why this isn't acceptable?

public class CustomerRepository
{
	// public MutationContext<Customer> customerMutationContext { get; set; } // line 3

	// public CustomerRepository(MutationContext<Customer> customerMutationContext) // line 5
	// {
	// 	this.customerMutationContext = customerMutationContext;
	// }

	public void SaveCustomer(Customer cust)
	{
		(new MutationContext(cust)).Mutate();
	}
}

Since line 3 and 5 explicitly define a MutationContext of Customer, why not have the constructor implicitly form that context from the parameter?

from dado.componentmodel.mutations.

Because "new is glue" and it should be the job of the IoC container to new up the dependencies of a class. Since MutationContext takes an instance of the customer, its not possible for the IoC container to provide it.

In my example, I could mock MutationContext and provide one that did anything I wanted in order to test my project. In your example new MutationContext is forever baked into the code even though its an outside dependency.

Ideally id inject IMutationContext and not MutationContext.

from dado.componentmodel.mutations.

Okay. I see what you're going after. You're welcome to submit a pull request.

from dado.componentmodel.mutations.