Mailspring/spec/mailbox-perspective-spec.es6
Juan Tejada 37af2ba42c fix(remove-from-view): Fix logic for delete/remove-from-view behavior:
Summary:
- Separate gmail's remove-from-view and delete behaviors and write logic
  for each of those
  - Remove MailboxPerspective::{canArchiveThreads, canTrashThreads,
    removeThreads} and some unecessary code in TaskFactory
  - Instead, add MailboxPerspective::tasksForRemovingFromPerspective (I
    know its a bit of a mouthful)
  - I initially tried to put all of the logic for each execution path
    inside the TaskFactory by checking perspective types, but it made
    more sense to use the polymorphism already in place for the different
    perspective types.
  - There is a default delete/remove-from-view behavior which is
    configurable via simple ruleset objects. The gmail behavior is
    configured in this way.
- Update swipe css classes based on destination of threads
- Fixes #1460:
  - Update logic to display archive/trash buttons and context menu options correctly
    when selected threads can be archived/trashed (not based on
    perspective)
  - Same for swiping
- Add a bunch of specs
- Convert some code to ES6
- TODO write some docs for new functions

Test Plan: Unit tests

Reviewers: drew, evan, bengotow

Reviewed By: bengotow

Differential Revision: https://phab.nylas.com/D2682
2016-03-07 18:16:37 -08:00

275 lines
9.8 KiB
JavaScript

import {
AccountStore,
MailboxPerspective,
TaskFactory,
Category,
CategoryStore,
} from 'nylas-exports'
import {Default} from '../internal_packages/thread-list/lib/category-removal-target-rulesets'
describe('MailboxPerspective', ()=> {
beforeEach(()=> {
this.accountIds = ['a1', 'a2']
this.accounts = {
'a1': {
id: 'a1',
defaultFinishedCategory: () => ({displayName: 'archive'}),
categoryIcon: ()=> null,
},
'a2': {
id: 'a2',
defaultFinishedCategory: () => ({displayName: 'trash2'}),
categoryIcon: ()=> null,
},
}
this.perspective = new MailboxPerspective(this.accountIds)
spyOn(AccountStore, 'accountForId').andCallFake((accId) => this.accounts[accId])
});
describe('isEqual', ()=> {
// TODO
});
describe('canArchiveThreads', ()=> {
it('returns false if the perspective is archive', ()=> {
const accounts = [
{canArchiveThreads: () => true},
{canArchiveThreads: () => true},
]
spyOn(AccountStore, 'accountsForItems').andReturn(accounts)
spyOn(this.perspective, 'isArchive').andReturn(true)
expect(this.perspective.canArchiveThreads()).toBe(false)
});
it('returns false if one of the accounts associated with the threads cannot archive', ()=> {
const accounts = [
{canArchiveThreads: () => true},
{canArchiveThreads: () => false},
]
spyOn(AccountStore, 'accountsForItems').andReturn(accounts)
spyOn(this.perspective, 'isArchive').andReturn(false)
expect(this.perspective.canArchiveThreads()).toBe(false)
});
it('returns true otherwise', ()=> {
const accounts = [
{canArchiveThreads: () => true},
{canArchiveThreads: () => true},
]
spyOn(AccountStore, 'accountsForItems').andReturn(accounts)
spyOn(this.perspective, 'isArchive').andReturn(false)
expect(this.perspective.canArchiveThreads()).toBe(true)
});
});
describe('canTrashThreads', ()=> {
it('returns false if the perspective is trash', ()=> {
const accounts = [
{canTrashThreads: () => true},
{canTrashThreads: () => true},
]
spyOn(AccountStore, 'accountsForItems').andReturn(accounts)
spyOn(this.perspective, 'isTrash').andReturn(true)
expect(this.perspective.canTrashThreads()).toBe(false)
});
it('returns false if one of the accounts associated with the threads cannot archive', ()=> {
const accounts = [
{canTrashThreads: () => true},
{canTrashThreads: () => false},
]
spyOn(AccountStore, 'accountsForItems').andReturn(accounts)
spyOn(this.perspective, 'isTrash').andReturn(false)
expect(this.perspective.canTrashThreads()).toBe(false)
});
it('returns true otherwise', ()=> {
const accounts = [
{canTrashThreads: () => true},
{canTrashThreads: () => true},
]
spyOn(AccountStore, 'accountsForItems').andReturn(accounts)
spyOn(this.perspective, 'isTrash').andReturn(false)
expect(this.perspective.canTrashThreads()).toBe(true)
});
});
describe('canReceiveThreadsFromAccountIds', ()=> {
it('returns true if the thread account ids are included in the current account ids', ()=> {
expect(this.perspective.canReceiveThreadsFromAccountIds(['a1'])).toBe(true)
});
it('returns false otherwise', ()=> {
expect(this.perspective.canReceiveThreadsFromAccountIds(['a4'])).toBe(false)
expect(this.perspective.canReceiveThreadsFromAccountIds([])).toBe(false)
expect(this.perspective.canReceiveThreadsFromAccountIds()).toBe(false)
});
});
describe('tasksForRemovingItems', ()=> {
beforeEach(()=> {
this.categories = {
'a1': {
'archive': new Category({name: 'archive', displayName: 'archive', accountId: 'a1'}),
'inbox': new Category({name: 'inbox', displayName: 'inbox1', accountId: 'a1'}),
'trash': new Category({name: 'trash', displayName: 'trash1', accountId: 'a1'}),
'category': new Category({name: null, displayName: 'folder1', accountId: 'a1'}),
},
'a2': {
'archive': new Category({name: 'all', displayName: 'all', accountId: 'a2'}),
'inbox': new Category({name: 'inbox', displayName: 'inbox2', accountId: 'a2'}),
'trash': new Category({name: 'trash', displayName: 'trash2', accountId: 'a2'}),
'category': new Category({name: null, displayName: 'label2', accountId: 'a2'}),
},
}
this.threads = [
{accountId: 'a1'},
{accountId: 'a2'},
]
spyOn(TaskFactory, 'tasksForApplyingCategories')
spyOn(CategoryStore, 'getTrashCategory').andCallFake((accId)=> {
return this.categories[accId].trash
})
});
function assertMoved(accId) {
expect(TaskFactory.tasksForApplyingCategories).toHaveBeenCalled()
const {args} = TaskFactory.tasksForApplyingCategories.calls[0]
const {categoriesToRemove, categoriesToAdd} = args[0]
const assertor = {
from(originName) {
expect(categoriesToRemove(accId)[0].displayName).toEqual(originName)
return assertor
},
to(destinationName) {
expect(categoriesToAdd(accId)[0].displayName).toEqual(destinationName)
return assertor
},
}
return assertor
}
it('moves to finished category if viewing inbox', ()=> {
const perspective = MailboxPerspective.forCategories([
this.categories.a1.inbox,
this.categories.a2.inbox,
])
perspective.tasksForRemovingItems(this.threads, Default)
assertMoved('a1').from('inbox1').to('archive')
assertMoved('a2').from('inbox2').to('trash2')
});
it('moves to trash if viewing archive', ()=> {
const perspective = MailboxPerspective.forCategories([
this.categories.a1.archive,
this.categories.a2.archive,
])
perspective.tasksForRemovingItems(this.threads, Default)
assertMoved('a1').from('archive').to('trash1')
assertMoved('a2').from('all').to('trash2')
})
it('deletes permanently if viewing trash', ()=> {
// TODO
// Not currently possible
});
it('moves to default finished category if viewing category', ()=> {
const perspective = MailboxPerspective.forCategories([
this.categories.a1.category,
this.categories.a2.category,
])
perspective.tasksForRemovingItems(this.threads, Default)
assertMoved('a1').from('folder1').to('archive')
assertMoved('a2').from('label2').to('trash2')
})
it('moves to default finished perspective if viewing search', ()=> {
const perspective = MailboxPerspective.forSearch(this.accountIds, '')
perspective.tasksForRemovingItems(this.threads, Default)
assertMoved('a1').to('archive')
assertMoved('a2').to('trash2')
});
it('unstars if viewing starred', ()=> {
spyOn(TaskFactory, 'taskForInvertingStarred').andReturn({some: 'task'})
const perspective = MailboxPerspective.forStarred(this.accountIds)
const tasks = perspective.tasksForRemovingItems(this.threads, Default)
expect(tasks).toEqual([{some: 'task'}])
});
it('does nothing when viewing spam or sent', ()=> {
['spam', 'sent'].forEach((invalid)=> {
const perspective = MailboxPerspective.forCategories([
new Category({name: invalid, accountId: 'a1'}),
new Category({name: invalid, accountId: 'a2'}),
])
const tasks = perspective.tasksForRemovingItems(this.threads, Default)
expect(TaskFactory.tasksForApplyingCategories).not.toHaveBeenCalled()
expect(tasks).toEqual([])
})
});
describe('when perspective is category perspective', ()=> {
it('overrides default ruleset', ()=> {
const customRuleset = {
all: ()=> ({displayName: 'my category'}),
}
const perspective = MailboxPerspective.forCategories([
this.categories.a1.category,
])
spyOn(perspective, 'categoriesSharedName').andReturn('all')
perspective.tasksForRemovingItems(this.threads, customRuleset)
assertMoved('a1').to('my category')
});
it('does not create tasks if any name in the ruleset is null', ()=> {
const customRuleset = {
all: null,
}
const perspective = MailboxPerspective.forCategories([
this.categories.a1.category,
])
spyOn(perspective, 'categoriesSharedName').andReturn('all')
const tasks = perspective.tasksForRemovingItems(this.threads, customRuleset)
expect(tasks).toEqual([])
});
});
});
describe('CategoryMailboxPerspective', ()=> {
beforeEach(()=> {
this.categories = [
new Category({displayName: 'c1', accountId: 'a1'}),
new Category({displayName: 'c2', accountId: 'a2'}),
new Category({displayName: 'c3', accountId: 'a2'}),
]
this.perspective = MailboxPerspective.forCategories(this.categories)
});
describe('canReceiveThreadsFromAccountIds', ()=> {
it('returns true if the thread account ids are included in the current account ids', ()=> {
expect(this.perspective.canReceiveThreadsFromAccountIds(['a1'])).toBe(true)
});
it('returns false otherwise', ()=> {
expect(this.perspective.canReceiveThreadsFromAccountIds(['a4'])).toBe(false)
expect(this.perspective.canReceiveThreadsFromAccountIds([])).toBe(false)
expect(this.perspective.canReceiveThreadsFromAccountIds()).toBe(false)
});
it('returns false if it is a locked category', ()=> {
this.perspective._categories.push(
new Category({name: 'sent', displayName: 'c4', accountId: 'a1'})
)
expect(this.perspective.canReceiveThreadsFromAccountIds(['a2'])).toBe(false)
});
});
describe('receiveThreads', ()=> {
// TODO
});
});
});