Lazy loading of imported packages has been implemented

divmgl · Oct 6, 2015 · 84f30d7 · 84f30d7
1 parent a467e4e
commit 84f30d7
Show file tree

Hide file tree

Showing 3 changed files with 11 additions and 7 deletions.
diff --git a/gulpfile.js b/gulpfile.js
@@ -11,5 +11,5 @@ gulp.task('test', function() {
 });
 
 gulp.task('watch:test', function(){
- gulp.watch('index.js', ["test"]);
-});
+ gulp.watch(['index.js', "tests/**/*"], ["test"]);
+});
diff --git a/index.js b/index.js
@@ -64,7 +64,11 @@ module.exports = function nwire(config, callback) {
 
  if (!skipNeeds) {
  self.packages[dependencyName] = load(dependencyName, skip);
- imports[dependencyName] = self.packages[dependencyName];
+ Object.defineProperty(imports, dependencyName, {
+ get: function(){
+ return self.packages[dependencyName];
+ }
+ });
  }
  });
 
@@ -91,4 +95,4 @@ module.exports = function nwire(config, callback) {
  if (!callback) throw err;
  callback(err);
  }
-}
+}
diff --git a/tests/nwire.js b/tests/nwire.js
@@ -81,9 +81,9 @@ describe('nwire application', function() {
 });
 
 describe('leech package', function(){
- it('has a consumed package', function(){
+ it('should not have packages', function(){
  var app = wire(configurationFixture);
- expect(_.size(app.packages.leech.imports)).to.equal(1);
+ expect(_.size(app.packages.leech.imports)).to.equal(0);
  });
 
  it('has the consumed seed package', function(){
@@ -100,4 +100,4 @@ describe('leech package', function(){
  var app = wire(circularConfigurationFixture);
  expect(app.packages.leech.imports.seed.dummyFn).to.not.be.undefined;
  });
-});
+});