fixed notes specs

changed how the container class is added onto the text element
more specs

1 files changed, 23 insertions, 1 deletions

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');
       });