fix: toucher log test

marmelab · Apr 8, 2020 · c7f2abd · c7f2abd
1 parent 6df2803
commit c7f2abd
Show file tree

Hide file tree

Showing 2 changed files with 10 additions and 7 deletions.
diff --git a/src/species/toucher.js b/src/species/toucher.js
@@ -248,7 +248,7 @@ export default userConfig => (logger, randomizer) => {
                 config.showAction(touches);
             }
             if (logger && config.log) {
-                logger.log('gremlin', 'toucher   ', touchType, 'at', posX, posY, details);
+                logger.log('gremlin', 'toucher', touchType, 'at', posX, posY, details);
             }
         };
         touchTypes[touchType](position, targetElement, log);

diff --git a/src/species/toucher.spec.js b/src/species/toucher.spec.js
@@ -1,12 +1,12 @@
 import toucher from './toucher';
-
 jest.useFakeTimers();
 
 describe('toucher', () => {
     const dispatchEventSpy = jest.fn();
     const initEventSpy = jest.fn();
     let inputText;
     let chanceMock;
+    let consoleMock;
 
     beforeEach(() => {
         chanceMock = {
@@ -15,6 +15,7 @@ describe('toucher', () => {
             integer: ({ max }) => max,
             pick: types => types[0],
         };
+        consoleMock = { log: jest.fn() };
 
         inputText = document.createElement('input');
         inputText.setAttribute('type', 'text');
@@ -44,11 +45,13 @@ describe('toucher', () => {
         document.body.innerHTML = '';
     });
 
-    it.skip('should log the toucher', () => {
-        // const species = toucher({ log: true })(console, chanceMock);
-        // species();
-        // expect(consoleSpy).toHaveBeenCalledTimes(1);
-        // expect(consoleSpy).toHaveBeenCalledWith('gremlin', 'toucher type', 'A', 'at', 10, 10);
+    it('should log the toucher', () => {
+        const species = toucher({ log: true })(consoleMock, chanceMock);
+
+        species();
+        jest.runAllTimers();
+        expect(consoleMock.log).toHaveBeenCalledTimes(1);
+        expect(consoleMock.log).toHaveBeenCalledWith('gremlin', 'toucher', 'tap', 'at', 10, 10, { duration: 700 });
     });
 
     it("should touch the element but don't show element", () => {