JsFileLoader not working with module.exports

[harpcio]

Example:

// repositories/UserRepository.js
class UserRepository 
    get model() {
        return User;
    }

    getAll() {
        return this.findAll();
    }
}

module.exports = UserRepository;

This is working fine:

const di = new ContainerBuilder();
di.register('UserRepository', UserRepository);

But this is not working at all:

// di/repositories.js
module.exports = {
    services: {
        "repositories.user": {
            class: "../repositories/UserRepository"
        },
    }
};

const container = new ContainerBuilder();
const loader = new JsFileLoader(container);
loader.load('./repositories.js');

Then trying to get this object:

container.get('repositories.user');

I received this:

node_modules/node-dependency-injection/dist/lib/InstanceManager.js:122
      throw new _ServiceNotFoundException["default"](id);

The problem is in FileLoader.js, line: 283, I will do PR

[rafal-ksiazek-rmtcfm-com]

https://github.com/zazoomauro/node-dependency-injection/pull/146

[rafal-ksiazek-rmtcfm-com]

Here you have my repository with full code: https://bitbucket.org/remotecraftsmen/erp-system/src/master/server/src/di/services.js

Right now is not working because this tiny bug.

[zazoomauro]

@rafal-ksiazek-rmtcfm-com thanks!

[rafal-ksiazek-rmtcfm-com]

@zazoomauro It's still not working - the change is not updated in npm registry. Maybe you need to update the version of package to 2.6.4?

[zazoomauro]

@rafal-ksiazek-rmtcfm-com build is failing! https://travis-ci.org/github/zazoomauro/node-dependency-injection/jobs/695776915

[zazoomauro]

Looks like npm run test:standardis failing

[harpcio]

Sorry for that: https://github.com/zazoomauro/node-dependency-injection/pull/147

[harpcio]

@zazoomauro Can you merge it?

[zazoomauro]

Merged