Refactor user-loader-factory to simplify it
Going through the code base I've noticed this unnecessarily complex piece of code.
I first covered the old logic by unit tests and then refactored the userLoaderFactory
.
Edit: And then in a face of failing API tests I went through the usages of userLoader
in the codebase and made sure non of the clients relies on bluebird
promise interface
Edited by Tomas Vik