Remove the FactoryModule (replaced by Ninject.Extensions.Factory) and fix unit tests

[scott-xu]

[scott-xu]

This would be a breaking change as user need to refer Ninject.Extensions.Factory if they used/will use Func<>. Where can I document this?

[khellang]

I'm not sure I'd classify this a a breaking change. It's just a new dependency and we don't have to change public APIs because of it. The question is; why is it there in the first place? Do we use Func<T> factories anywhere? @thecodejunkie, @grumpydev? If we do, then this gets my :+1:

[scott-xu]

@khellang Func<T> is used at least in RouteCache https://github.com/NancyFx/Nancy/blob/master/src/Nancy/Routing/DefaultRouteCacheProvider.cs#L46

[scott-xu]

Hi @thecodejunkie @grumpydev, what do you think about this?