Merge pull request #8 from RobotlegsJS/coverage

Improve code coverage

Author: tiagoschenkel

test/org/osflash/signals/SlotListTest.test.ts

@@ -44,6 +44,14 @@ describe("SlotListTest", () => {
         assert.equal(0, SlotList.NIL.length);
     });
 
+    it("NIL_can_not_append_a_new_slot", () => {
+        let list: SlotList = SlotList.NIL.append(slotA);
+        // list should be a new SlotList with only one element
+        assert.equal(1, list.length);
+        // append a element to NIL should not change it's length
+        assert.equal(0, SlotList.NIL.length);
+    });
+
     it("tail_defaults_to_NIL_if_omitted_in_constructor", () => {
         const noTail: SlotList = new SlotList(slotA);
         assert.equal(SlotList.NIL, noTail.tail);

test/org/osflash/signals/SlotTest.test.ts

@@ -69,6 +69,13 @@ describe("SlotTest", () => {
         assert.deepEqual(received, expected);
     });
 
+    it("execute0_with_once_enabled_should_remove_listener", () => {
+        expected = [];
+        slot = new Slot(listener0, signal, true);
+        slot.execute0();
+        assert.deepEqual(received, expected);
+    });
+
     it("execute1_should_not_call_listener_when_disabled", () => {
         slot = new Slot(listener1, signal);
         slot.enabled = false;
@@ -100,6 +107,13 @@ describe("SlotTest", () => {
         assert.deepEqual(received, expected);
     });
 
+    it("execute1_with_once_enabled_should_remove_listener", () => {
+        expected = [42];
+        slot = new Slot(listener1, signal, true);
+        slot.execute1(expected[0]);
+        assert.deepEqual(received, expected);
+    });
+
     it("execute_should_not_call_listener_when_disabled", () => {
         slot = new Slot(listener0, signal);
         slot.enabled = false;