ensure delete re-execution (#313)

Author: guybedford

src/loader.js

@@ -939,7 +939,10 @@ function logloads(loads) {
     },
     // 26.3.3.3
     'delete': function(name) {
-      return this._loader.modules[name] ? delete this._loader.modules[name] : false;
+      var loader = this._loader;
+      delete loader.importPromises[name];
+      delete loader.moduleRecords[name];
+      return loader.modules[name] ? delete loader.modules[name] : false;
     },
     // 26.3.3.4 entries not implemented
     // 26.3.3.5

test/loader/module.js

@@ -0,0 +1,7 @@
+export var run;
+
+(function(global) {
+  run = global.run ? 'second' : 'first';
+
+  global.run = true;
+})(typeof window == 'undefined' ? global : window);

test/system.spec.js

@@ -46,6 +46,30 @@ describe('System', function () {
 
     });
 
+    describe('System registry methods', function() {
+
+      it('should support set, get and delete', function(done) {
+
+        var testPath = 'test/loader/module';
+  
+        System.import(testPath).then(function(m) {
+          expect(m.run).to.equal('first');
+          System.delete(testPath);
+          return System.import(testPath);
+        })
+        .then(function(m) {
+          expect(m.run).to.equal('second');
+          System.delete('loader.module');
+          System.set(testPath, System.newModule({ custom: 'module' }));
+          return System.import(testPath);
+        })
+        .then(function(m) {
+          expect(m.custom).to.equal('module');
+        })
+        .then(done, done);
+      });
+    });
+
     describe('an ES6 script', function () {
 
       it('should import an ES6 script', function (done) {

test/test.js

@@ -179,6 +179,23 @@ function runTests() {
     assert(System.normalize('../a/b', '../../c/d'), '../../a/b');
   });
 
+  test('Setting & deleting modules', function(assert, err) {
+    System['import']('loader/module').then(function(m1) {
+      System['delete']('loader/module');
+      System['import']('loader/module').then(function(m2) {
+        System['delete']('loader/module');
+        System.set('loader/module', System.newModule({custom: 'module'}));
+        System['import']('loader/module').then(function(m3) {
+          assert(
+            [m1.run, 'first'],
+            [m2.run, 'second'],
+            [m3.custom, 'module']
+          );
+        }, err);
+      }, err);
+    }, err);
+  });
+
   test('Import a script', function(assert, err) {
     System['import']('syntax/script').then(function(m) {
       assert(!!m, true);