Merge pull request #336 from BackburnerJS/dropped-event

Fix for 332

Author: rwjblue

lib/backburner/deferred-action-queues.ts

@@ -37,6 +37,8 @@ export default class DeferredActionQueues {
       throw new Error(`You attempted to schedule an action in a queue (${queueName}) for a method that doesn\'t exist`);
     }
 
+    this.queueNameIndex = 0;
+
     if (onceFlag) {
       return queue.pushUnique(target, method, args, stack);
     } else {
@@ -66,7 +68,6 @@ export default class DeferredActionQueues {
         if (queue.flush(false /* async */) === QUEUE_STATE.Pause) {
           return QUEUE_STATE.Pause;
         }
-        this.queueNameIndex = 0; // only reset to first queue if non-pause break
       }
     }
   }

tests/autorun-test.ts

@@ -104,22 +104,28 @@ QUnit.test('autorun interleaved with microtasks do not get dropped [GH#332]', fu
 
   bb.schedule('render', function() {
     actual.push('first');
+    bb.schedule('actions', () => {
+      actual.push('action1');
+    });
 
     Promise.resolve().then(() => {
       actual.push('second');
+      bb.schedule('actions', () => {
+        actual.push('action2');
+      });
 
       return Promise.resolve().then(() => {
         actual.push('third');
 
         bb.schedule('actions', () => {
-          actual.push('fourth');
+          actual.push('action3');
         });
       });
     });
   });
 
   setTimeout(function() {
-    assert.deepEqual(actual, ['first', 'second', 'third', 'fourth']);
+    assert.deepEqual(actual, ['first', 'action1', 'second', 'action2', 'third', 'action3']);
 
     done();
   });

tests/debounce-test.ts

@@ -49,7 +49,7 @@ QUnit.test('debounce', function(assert) {
   }, 110);
 
   // great, we've made it this far, there's one more thing
-  // we need to QUnit.test. we want to make sure we can call `debounce`
+  // we need to test. we want to make sure we can call `debounce`
   // again with the same target/method after it has executed
 
   // at the 120ms mark, let's schedule another call to `debounce`