Skip to content

Commit

Permalink
Ensure home view is reloaded after changing stats interval
Browse files Browse the repository at this point in the history
  • Loading branch information
zozs committed Apr 23, 2023
1 parent 9c6477b commit 85ee013
Show file tree
Hide file tree
Showing 3 changed files with 10 additions and 3 deletions.
4 changes: 3 additions & 1 deletion __tests__/settings.test.js
Original file line number Diff line number Diff line change
Expand Up @@ -206,7 +206,9 @@ describe('weekday setting', () => {
expect(db.setWeekdays.mock.calls[0][1]).toBe(0b0000000)
expect(res.send).toHaveBeenCalledAfter(db.setWeekdays)
})
})

describe('userSetting', () => {
test('can set stats interval correctly', async () => {
const instanceRef = 'T1234'
const userRef = 'U1'
Expand All @@ -224,7 +226,7 @@ describe('weekday setting', () => {
}
const res = { send: jest.fn() }

await settings.setUserSetting(res, instanceRef, action, userRef, 'statsInterval')
await settings.setUserSetting(res, instanceRef, action, userRef, 'statsInterval', asyncEventHandler)
expect(db.setUserSetting).toHaveBeenCalledTimes(1)
expect(db.setUserSetting.mock.calls[0][0]).toBe(instanceRef)
expect(db.setUserSetting.mock.calls[0][1]).toBe(userRef)
Expand Down
2 changes: 1 addition & 1 deletion routes.js
Original file line number Diff line number Diff line change
Expand Up @@ -64,7 +64,7 @@ module.exports = (app, asyncEventHandler) => {
await settings.setEndTime(res, instanceRef, action, asyncEventHandler)
break
case 'user_stats_interval':
await settings.setUserSetting(res, instanceRef, action, userRef, 'statsInterval')
await settings.setUserSetting(res, instanceRef, action, userRef, 'statsInterval', asyncEventHandler)
break
case 'wild_button':
await clickCommand(res, payload, asyncEventHandler)
Expand Down
7 changes: 6 additions & 1 deletion settings.js
Original file line number Diff line number Diff line change
Expand Up @@ -47,9 +47,14 @@ module.exports = {
res.send('')
},

async setUserSetting (res, instanceRef, action, userRef, name) {
async setUserSetting (res, instanceRef, action, userRef, name, asyncEventHandler) {
const value = parseInt(action.selected_option.value)
await db.setUserSetting(instanceRef, userRef, name, value)
await asyncEventHandler({
method: 'home',
instanceRef,
user: userRef
})
res.send('')
},

Expand Down

0 comments on commit 85ee013

Please sign in to comment.