From fcd842b55837d2a8eab4ef32b65b5a44dfab6eba Mon Sep 17 00:00:00 2001
From: Phil Hughes <me@iamphill.com>
Date: Thu, 5 Oct 2017 10:27:43 +0100
Subject: fixed notes specs changed how the container class is added onto the
 text element more specs

---
 spec/javascripts/flash_spec.js | 24 +++++++++++++++++++++++-
 1 file changed, 23 insertions(+), 1 deletion(-)

(limited to 'spec/javascripts')

diff --git a/spec/javascripts/flash_spec.js b/spec/javascripts/flash_spec.js
index 66cc76ee626..060ffaa339b 100644
--- a/spec/javascripts/flash_spec.js
+++ b/spec/javascripts/flash_spec.js
@@ -31,6 +31,17 @@ describe('Flash', () => {
         el.querySelector('.flash-text').textContent.trim(),
       ).toBe('<script>alert("a");</script>');
     });
+
+    it('adds container classes when inside content wrapper', () => {
+      el.innerHTML = createFlashEl('testing', 'alert', true);
+
+      expect(
+        el.querySelector('.flash-text').classList.contains('container-fluid'),
+      ).toBeTruthy();
+      expect(
+        el.querySelector('.flash-text').classList.contains('container-limited'),
+      ).toBeTruthy();
+    });
   });
 
   describe('hideFlash', () => {
@@ -57,6 +68,17 @@ describe('Flash', () => {
       ).toBe('0');
     });
 
+    it('does not set styles when fadeTransition is false', () => {
+      hideFlash(el, false);
+
+      expect(
+        el.style.opacity,
+      ).toBe('');
+      expect(
+        el.style.transition,
+      ).toBe('');
+    });
+
     it('removes element after transitionend', () => {
       document.body.appendChild(el);
 
@@ -192,7 +214,7 @@ describe('Flash', () => {
         flash('test');
 
         expect(
-          document.querySelector('.flash-text').className,
+          document.querySelector('.flash-text').className.trim(),
         ).toBe('flash-text');
       });
 
-- 
cgit v1.2.1