Skip to content

Commit

Permalink
fix(user-update): fix user update error introduced in #577
Browse files Browse the repository at this point in the history
  • Loading branch information
landonreed committed May 26, 2020
1 parent 7ed815f commit c984474
Show file tree
Hide file tree
Showing 2 changed files with 3 additions and 2 deletions.
3 changes: 2 additions & 1 deletion __tests__/end-to-end.js
Original file line number Diff line number Diff line change
Expand Up @@ -915,7 +915,8 @@ describe('end-to-end', () => {
await waitForAndClick(`[data-test-id="edit-user-${testUserSlug}"]`)
// $FlowFixMe cryptic error that is hard to resolve :(
const adminCheckbox = await page.$(`[data-test-id="app-admin-checkbox-${testUserSlug}"]`)
expect(await (await adminCheckbox.getProperty('checked')).jsonValue()).toBe(true)
const isAdmin = await (await adminCheckbox.getProperty('checked')).jsonValue()
expect(isAdmin).toBe(true)
}, defaultTestTimeout, 'should allow admin user to create another user')

makeTestPostLogin('should delete a user', async () => {
Expand Down
2 changes: 1 addition & 1 deletion lib/manager/actions/user.js
Original file line number Diff line number Diff line change
Expand Up @@ -232,7 +232,7 @@ export function updateUserData (user: any, userData: any) {
const url = `/api/manager/secure/user/${user.user_id}`
const payload = {
user_id: profile.user_id,
app_metadata: metadata.datatools
data: metadata.datatools
}
let status
// Make request to server
Expand Down

0 comments on commit c984474

Please # to comment.