- Making errors more descriptive
- When we cannot not get userID we should delete database
Details
Details
- Throw errors from setting/getting userID and check if errors are printed.
- Throw error from getting and make sure database is deleted
Diff Detail
Diff Detail
- Repository
- rCOMM Comm
- Branch
- not-init-fixed
- Lint
No Lint Coverage - Unit
No Test Coverage
Event Timeline
web/database/sqlite-data-handler.js | ||
---|---|---|
43–59 | It is possible that await databaseModule.init({ clearDatabase: true }); will throw and we catch it, and then await databaseModule.schedule({ type: workerRequestMessageTypes.SET_CURRENT_USER_ID, userID: currentLoggedInUserID, }); will succeed... won't that result in assigning a new user to the old data? |
web/database/sqlite-data-handler.js | ||
---|---|---|
43–59 | yes, good catch, fixed |