Adding test for only setting CSS once

lindsayevans · Sep 29, 2018 · 0c5efc6 · 0c5efc6
1 parent e9c6d52
commit 0c5efc6
Show file tree

Hide file tree

Showing 2 changed files with 11 additions and 3 deletions.
diff --git a/src/remove-focus-outline.test.ts b/src/remove-focus-outline.test.ts
@@ -102,7 +102,12 @@ describe('The library should...', () => {
 
     })
 
-    // TODO:
-    // * configuration merges correctly
+    test('only set CSS once for multiple of same event', () => {
+        setCssSpy.mockClear()
+        triggerEvent('mousedown')
+        triggerEvent('mousedown')
+
+        expect(setCssSpy).toHaveReturnedWith(false)
+    })
 
 })
diff --git a/src/remove-focus-outline.ts b/src/remove-focus-outline.ts
@@ -78,7 +78,7 @@ export class RemoveFocusOutline {
     /**
      * Sets the CSS
      */
-    private setCss(css: string) {
+    private setCss(css: string): boolean {
 
         if (css !== this.currentCss) {
             this.currentCss = css;
@@ -89,6 +89,9 @@ export class RemoveFocusOutline {
             } else {
                 this.$style.innerHTML = css;
             }
+            return true;
+        } else {
+            return false;
         }
     }