Skip to content

Commit ff806e6

Browse files
committed
Do not need to wrap return in del as a Promise
backend.del should do this by default
1 parent 34d3a63 commit ff806e6

File tree

2 files changed

+3
-4
lines changed

2 files changed

+3
-4
lines changed

src/user-store.js

Lines changed: 1 addition & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -205,8 +205,7 @@ class UserStore {
205205
deleteUser (user) {
206206
let userKey = UserStore.normalizeIdKey(user.id)
207207

208-
return Promise.resolve()
209-
.then(() => this.backend.del('users', userKey))
208+
return this.backend.del('users', userKey)
210209
}
211210

212211
/**

test/unit/user-store-test.js

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -225,8 +225,8 @@ describe('UserStore', () => {
225225

226226
store.backend.del = sinon.stub()
227227

228-
return store.deleteUser({ id: userId })
229-
.then(() => expect(store.backend.del).to.have.been.calledWith('users', UserStore.normalizeIdKey(userId)))
228+
store.deleteUser({ id: userId })
229+
expect(store.backend.del).to.have.been.calledWith('users', UserStore.normalizeIdKey(userId))
230230
})
231231
})
232232
})

0 commit comments

Comments
 (0)