Foundry376/Mailspring
1
1
Fork 0
mirror of https://github.com/Foundry376/Mailspring.git synced 2025-01-08 01:04:39 +08:00
Code Issues Projects Releases Packages Wiki Activity

Fix MutliSelectSplitInteractionHandler specs

Browse source
This commit is contained in:
Juan Tejada 2016-01-22 13:47:54 -08:00
parent 27ff4d2de2
commit ad05cc05cc
1 changed files with 20 additions and 30 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

50
spec/components/multiselect-split-interaction-handler-spec.coffee
View file

@ -14,6 +14,9 @@ describe "MultiselectSplitInteractionHandler", ->
@itemAfterKeyboardFocus = new Thread(id:'after-keyboard-focus')
data = [@item, @itemFocus, @itemAfterFocus, @itemKeyboardFocus, @itemAfterKeyboardFocus]
@onFocusItem = jasmine.createSpy('onFocusItem')
@onSetCursorPosition = jasmine.createSpy('onSetCursorPosition')
@selection = []
@dataSource =
selection:
@ -34,13 +37,19 @@ describe "MultiselectSplitInteractionHandler", ->
_.findIndex data, (item) -> item.id is id
count: -> data.length
@props =
dataSource: @dataSource
keyboardCursorId: 'keyboard-focus'
focused: @itemFocus
focusedId: 'focus'
onFocusItem: @onFocusItem
onSetCursorPosition: @onSetCursorPosition
@collection = 'threads'
@handler = new MultiselectSplitInteractionHandler(@dataSource, @collection)
@isRootSheet = true
@handler = new MultiselectSplitInteractionHandler(@props)
spyOn(WorkspaceStore, 'topSheet').andCallFake => {root: @isRootSheet}
spyOn(Actions, 'setFocus')
spyOn(Actions, 'setCursorPosition')
it "should always show focus", ->
expect(@handler.shouldShowFocus()).toEqual(true)
@ -56,25 +65,17 @@ describe "MultiselectSplitInteractionHandler", ->
describe "onClick", ->
it "should focus the list item and indicate it was focused via click", ->
@handler.onClick(@item)
expect(Actions.setFocus).toHaveBeenCalledWith({
collection: @collection
item: @item
usingClick: true
})
expect(@onFocusItem).toHaveBeenCalledWith(@item)
describe "onMetaClick", ->
describe "when there is currently a focused item", ->
beforeEach ->
spyOn(FocusedContentStore, 'focused').andCallFake => @itemFocus
spyOn(FocusedContentStore, 'focusedId').andCallFake -> 'focus'
it "should turn the focused item into the first selected item", ->
@handler.onMetaClick(@item)
expect(@dataSource.selection.add).toHaveBeenCalledWith(@itemFocus)
it "should clear the focus", ->
@handler.onMetaClick(@item)
expect(Actions.setFocus).toHaveBeenCalledWith({collection: @collection, item: null})
expect(@onFocusItem).toHaveBeenCalledWith(null)
it "should toggle selection", ->
@handler.onMetaClick(@item)
@ -87,9 +88,6 @@ describe "MultiselectSplitInteractionHandler", ->
describe "onShiftClick", ->
describe "when there is currently a focused item", ->
beforeEach ->
spyOn(FocusedContentStore, 'focused').andCallFake => @itemFocus
spyOn(FocusedContentStore, 'focusedId').andCallFake -> 'focus'
it "should turn the focused item into the first selected item", ->
@handler.onMetaClick(@item)
@ -97,7 +95,7 @@ describe "MultiselectSplitInteractionHandler", ->
it "should clear the focus", ->
@handler.onMetaClick(@item)
expect(Actions.setFocus).toHaveBeenCalledWith({collection: @collection, item: null})
expect(@onFocusItem).toHaveBeenCalledWith(null)
it "should expand selection", ->
@handler.onShiftClick(@item)
@ -124,41 +122,33 @@ describe "MultiselectSplitInteractionHandler", ->
describe "when the select option is passed", ->
it "should turn the existing focused item into a selected item", ->
spyOn(FocusedContentStore, 'focused').andCallFake => @itemFocus
spyOn(FocusedContentStore, 'focusedId').andCallFake -> 'focus'
@handler.onShift(1, {select: true})
expect(@dataSource.selection.add).toHaveBeenCalledWith(@itemFocus)
it "should walk the selection to the shift target", ->
spyOn(FocusedContentStore, 'focused').andCallFake => @itemFocus
spyOn(FocusedContentStore, 'focusedId').andCallFake -> 'focus'
@handler.onShift(1, {select: true})
expect(@dataSource.selection.walk).toHaveBeenCalledWith({current: @itemFocus, next: @itemAfterFocus})
describe "when one or more items is selected", ->
it "should move the keyboard cursor", ->
@selection = [@itemFocus, @itemAfterFocus, @itemKeyboardFocus]
spyOn(FocusedContentStore, 'keyboardCursor').andCallFake => @itemKeyboardFocus
spyOn(FocusedContentStore, 'keyboardCursorId').andCallFake -> 'keyboard-focus'
@handler.onShift(1, {})
expect(Actions.setCursorPosition).toHaveBeenCalledWith({collection: @collection, item: @itemAfterKeyboardFocus})
expect(@onSetCursorPosition).toHaveBeenCalledWith(@itemAfterKeyboardFocus)
describe "when no items are selected", ->
it "should move the focus", ->
spyOn(FocusedContentStore, 'focused').andCallFake => @itemFocus
spyOn(FocusedContentStore, 'focusedId').andCallFake -> 'focus'
@handler.onShift(1, {})
expect(Actions.setFocus).toHaveBeenCalledWith({collection: @collection, item: @itemAfterFocus})
expect(@onFocusItem).toHaveBeenCalledWith(@itemAfterFocus)
describe "_checkSelectionAndFocusConsistency", ->
describe "when only one item is selected", ->
beforeEach ->
spyOn(FocusedContentStore, 'focused').andCallFake -> null
spyOn(FocusedContentStore, 'focusedId').andCallFake -> null
@selection = [@item]
@props.focused = null
@handler = new MultiselectSplitInteractionHandler(@props)
it "should clear the selection and make the item focused", ->
@handler._checkSelectionAndFocusConsistency()
expect(@dataSource.selection.clear).toHaveBeenCalled()
expect(Actions.setFocus).toHaveBeenCalledWith({collection: @collection, item: @item})
expect(@onFocusItem).toHaveBeenCalledWith(@item)