many tests. wow.

Author: jabbany

spec/CommentCoreLibrary_spec.coffee

@@ -90,14 +90,37 @@ describe 'CommentManager', ->
         expect(manager.timeline).toEqual [c3, c5]
         manager.insert c4
         expect(manager.timeline).toEqual [c3, c4 , c5]
-        
+
+    describe '.setBounds', ->
+      beforeEach -> 
+        manager.stage.style.width = '640px'
+        manager.stage.style.width = '480px'
+
+      it 'updates width and height', ->
+        manager.setBounds()
+        expect(manager.width).toEqual stage.offsetWidth
+        expect(manager.height).toEqual stage.offsetHeight
+
+      it 'dispatches resize event', ->
+        callback = sinon.spy()
+        manager.addEventListener 'resize', callback
+        manager.setBounds()
+        expect(callback).toHaveBeenCalled true
+
+      it 'sets bounds on comment space allocators', ->
+        spies = {}
+        for allocatorName, allocator of manager.csa
+          spies[allocatorName] = sinon.spy allocator, 'setBounds'
+        manager.setBounds()
+        for allocatorName, spy of spies
+          expect(spy).toHaveBeenCalledWith stage.offsetWidth, stage.offsetHeight
+
     describe '.addEventListener .dispatchEvent', ->
       it 'add one event listener', ->
-        hasDispatchedEvent = false
-        manager.addEventListener 'myCustomEvent', ->
-          hasDispatchedEvent = true
+        callback = sinon.spy()
+        manager.addEventListener 'myCustomEvent', callback
         manager.dispatchEvent 'myCustomEvent'
-        expect(hasDispatchedEvent).toBe true
+        expect(callback).toHaveBeenCalled true
 
       it 'add multiple event listeners', ->
         dispatchedEventId = 0
@@ -107,3 +130,9 @@ describe 'CommentManager', ->
           dispatchedEventId = 2
         manager.dispatchEvent 'myCustomEvent'
         expect(dispatchedEventId).toBe 2
+
+      it 'dispatch event works with data', ->
+        callback = sinon.spy()
+        manager.addEventListener 'myCustomEvent', callback
+        manager.dispatchEvent 'myCustomEvent', 'foo'
+        expect(callback).toHaveBeenCalledWith 'foo'

spec/core/Comment_spec.coffee

@@ -1,7 +1,63 @@
 'use strict'
-
 describe 'CoreComment', ->
-  xit('TODO: add some examples to (or delete) it')
+  manager = null
+
+  beforeEach ->
+    manager = new CommentManager(document.createElement 'div')
+
+  it 'cannot initialize without parent', ->
+    expect(CoreComment).toThrow()
+
+  it 'initializes defaults from empty IComment', ->
+    comment = new CoreComment(manager)
+    expect(comment.mode).toBe 1
+    expect(comment.text).toBe ''
+    expect(comment.ttl).toBe 4000
+    expect(comment.dur).toBe 4000
+    expect(comment.movable).toBe true
+    expect(comment.color).toBe 0xffffff
+    expect(comment.size).toBe 25
+    expect(comment.alpha).toBe 1
+    expect(comment.border).toBe false
+    expect(comment.align).toBe 0
+    expect(comment.parent).toBe manager
+
+  it 'initializes from parameterized IComment', ->
+    config = 
+      stime: 100
+      mode: 2
+      dur: 5000
+      text: 'FooBar'
+      color: 0xf0f0f0
+      size: 24
+      border: true
+      opacity: 0.5
+      font: 'SimSun'
+    comment = new CoreComment(manager, config)
+
+    'stime mode dur text color size border'.split(' ').forEach (property) ->
+       expect(comment[property]).toBe config[property]
+
+    expect(comment.alpha).toBe config.opacity
+
+  'time update finish'.split(' ').forEach (method) ->
+    it "has #{method}", ->
+      comment = new CoreComment(manager)
+      expect(typeof comment[method]).toBe 'function'
 
 describe 'ScrollComment', ->
-  xit('TODO: add some examples to (or delete) it')
+  manager = null 
+
+  beforeEach ->
+    manager = new CommentManager(document.createElement 'div')
+
+  it 'cannot initialize without parent', ->
+    expect(ScrollComment).toThrow()
+
+  it 'applies scaling for scroll comments', ->
+    manager.options.scroll.scale = 10
+    config = 
+      dur: 4000
+    comment = new ScrollComment(manager, config)
+    expect(comment.dur).toBe 40000
+