fix tests

sadiqmmm · May 8, 2014 · 5e60376 · 5e60376
1 parent e5259e0
commit 5e60376
Show file tree

Hide file tree

Showing 2 changed files with 23 additions and 35 deletions.
diff --git a/test/unit/lib/color-picker.coffee b/test/unit/lib/color-picker.coffee
@@ -14,14 +14,14 @@ describe('ColorPicker', ->
     picker = new Quill.Lib.ColorPicker(select)
     expect(container.querySelector('.sc-color-picker').outerHTML).toEqualHTML('
       <div title="Text Color" class="sc-color sc-picker sc-color-picker">
-        <div class="sc-picker-label"></div>
+        <div data-value="rgb(0, 0, 0)" class="sc-picker-label sc-format-button"></div>
         <div class="sc-picker-options">
-          <div class="sc-picker-item sc-selected" style="background-color: rgb(0, 0, 0);"></div>
-          <div class="sc-picker-item" style="background-color: rgb(230, 0, 0);"></div>
-          <div class="sc-picker-item" style="background-color: rgb(255, 153, 0);"></div>
-          <div class="sc-picker-item" style="background-color: rgb(255, 255, 0);"></div>
-          <div class="sc-picker-item" style="background-color: rgb(0, 138, 0);"></div>
-          <div class="sc-picker-item" style="background-color: rgb(0, 102, 204);"></div>
+          <div data-value="rgb(0, 0, 0)" class="sc-picker-item sc-selected" style="background-color: rgb(0, 0, 0);"></div>
+          <div data-value="rgb(230, 0, 0)" class="sc-picker-item" style="background-color: rgb(230, 0, 0);"></div>
+          <div data-value="rgb(255, 153, 0)" class="sc-picker-item" style="background-color: rgb(255, 153, 0);"></div>
+          <div data-value="rgb(255, 255, 0)" class="sc-picker-item" style="background-color: rgb(255, 255, 0);"></div>
+          <div data-value="rgb(0, 138, 0)" class="sc-picker-item" style="background-color: rgb(0, 138, 0);"></div>
+          <div data-value="rgb(0, 102, 204)" class="sc-picker-item" style="background-color: rgb(0, 102, 204);"></div>
         </div>
       </div>
     ')

diff --git a/test/unit/lib/picker.coffee b/test/unit/lib/picker.coffee
@@ -14,11 +14,11 @@ describe('Picker', ->
   it('constructor', ->
     expect(@container.querySelector('.sc-picker').outerHTML).toEqualHTML('
       <div title="Font" class="sc-font sc-picker">
-        <div class="sc-picker-label">Sans Serif</div>
+        <div data-value="sans-serif" class="sc-picker-label">Sans Serif</div>
         <div class="sc-picker-options">
-          <div class="sc-picker-item sc-selected">Sans Serif</div>
-          <div class="sc-picker-item">Serif</div>
-          <div class="sc-picker-item">Monospace</div>
+          <div data-value="sans-serif" class="sc-picker-item sc-selected">Sans Serif</div>
+          <div data-value="serif" class="sc-picker-item">Serif</div>
+          <div data-value="monospace" class="sc-picker-item">Monospace</div>
         </div>
       </div>
     ')
@@ -35,37 +35,25 @@ describe('Picker', ->
 
   it('select picker item', ->
     Quill.DOM.triggerEvent(@container.querySelector('.sc-picker-options').lastChild, 'click')
-    expect(@container.querySelector('.sc-picker')).toEqualHTML('
-      <div class="sc-picker-label sc-active">Monospace</div>
-      <div class="sc-picker-options">
-        <div class="sc-picker-item">Sans Serif</div>
-        <div class="sc-picker-item">Serif</div>
-        <div class="sc-picker-item sc-selected">Monospace</div>
-      </div>
-    ')
+    expect(Quill.DOM.getText(@picker.label)).toEqual('Monospace')
+    _.each(@container.querySelectorAll('.sc-picker-item'), (item, i) ->
+      expect(Quill.DOM.hasClass(item, 'sc-selected')).toBe(i == 2)
+    )
   )
 
   it('select option', ->
     Quill.DOM.selectOption(@select, 'serif')
-    expect(@container.querySelector('.sc-picker')).toEqualHTML('
-      <div class="sc-picker-label sc-active">Serif</div>
-      <div class="sc-picker-options">
-        <div class="sc-picker-item">Sans Serif</div>
-        <div class="sc-picker-item sc-selected">Serif</div>
-        <div class="sc-picker-item">Monospace</div>
-      </div>
-    ')
+    expect(Quill.DOM.getText(@picker.label)).toEqual('Serif')
+    _.each(@container.querySelectorAll('.sc-picker-item'), (item, i) ->
+      expect(Quill.DOM.hasClass(item, 'sc-selected')).toBe(i == 1)
+    )
   )
 
   it('select option mixed', ->
     Quill.DOM.selectOption(@select, '')
-    expect(@container.querySelector('.sc-picker')).toEqualHTML('
-      <div class="sc-picker-label sc-active">&nbsp;</div>
-      <div class="sc-picker-options">
-        <div class="sc-picker-item">Sans Serif</div>
-        <div class="sc-picker-item">Serif</div>
-        <div class="sc-picker-item">Monospace</div>
-      </div>
-    ')
+    expect(Quill.DOM.getText(@picker.label).trim()).toEqual('')
+    _.each(@container.querySelectorAll('.sc-picker-item'), (item, i) ->
+      expect(Quill.DOM.hasClass(item, 'sc-selected')).toBe(false)
+    )
   )
 )