Firing show event on the slide

Antonio-Laguna · Antonio-Laguna · commit 5dd1b9c649a4 · 2018-01-01T16:03:32.000+01:00
Fixes webslides#104
diff --git a/src/js/modules/slide.js b/src/js/modules/slide.js
@@ -8,8 +8,9 @@ const CLASSES = {
 const Events = {
   ENTER: 'dom:enter',
   LEAVE: 'dom:leave',
+  DISABLE: 'slide:disable',
   ENABLE: 'slide:enable',
-  DISABLE: 'slide:disable'
+  SHOW: 'slide:show'
 };
 
 /**
@@ -57,6 +58,7 @@ class Slide {
   show() {
     DOM.show(this.el);
     this.el.classList.add(CLASSES.CURRENT);
+    this.fire_(Events.SHOW);
   }
 
   /**
diff --git a/test/modules/slide.test.js b/test/modules/slide.test.js
@@ -71,36 +71,42 @@ describe('Slide module', () => {
       const enter = jest.fn();
       const enable = jest.fn();
       const disable = jest.fn();
+      const show = jest.fn();
 
       slide.el.addEventListener('dom:leave', leave);
       slide.el.addEventListener('dom:enter', enter);
       slide.el.addEventListener('slide:enable', enable);
       slide.el.addEventListener('slide:disable', disable);
+      slide.el.addEventListener('slide:show', show);
 
       expect(enter).not.toHaveBeenCalled();
       expect(leave).not.toHaveBeenCalled();
       expect(enable).not.toHaveBeenCalled();
       expect(disable).not.toHaveBeenCalled();
+      expect(show).not.toHaveBeenCalled();
 
       slide.enable();
       expect(enter).not.toHaveBeenCalled();
       expect(leave).not.toHaveBeenCalled();
       expect(enable).toHaveBeenCalledTimes(1);
       expect(disable).not.toHaveBeenCalled();
+      expect(show).not.toHaveBeenCalled();
       enable.mockClear();
 
       slide.disable();
       expect(enter).not.toHaveBeenCalled();
       expect(leave).not.toHaveBeenCalled();
       expect(enable).not.toHaveBeenCalled();
       expect(disable).toHaveBeenCalledTimes(1);
+      expect(show).not.toHaveBeenCalled();
       disable.mockClear();
 
       slide.moveAfterLast();
       expect(enter).toHaveBeenCalledTimes(1);
       expect(leave).toHaveBeenCalledTimes(1);
       expect(enable).not.toHaveBeenCalled();
       expect(disable).not.toHaveBeenCalled();
+      expect(show).not.toHaveBeenCalled();
       enter.mockClear();
       leave.mockClear();
 
@@ -109,8 +115,12 @@ describe('Slide module', () => {
       expect(leave).toHaveBeenCalledTimes(1);
       expect(enable).not.toHaveBeenCalled();
       expect(disable).not.toHaveBeenCalled();
+      expect(show).not.toHaveBeenCalled();
       enter.mockClear();
       leave.mockClear();
+
+      slide.show();
+      expect(show).toHaveBeenCalled();
     });
 
     test('Move', () => {
