Skip to content

Commit 2d20944

Browse files
test(users): add default provider test ✅
1 parent d21b25c commit 2d20944

2 files changed

Lines changed: 2 additions & 3 deletions

File tree

modules/users/services/user.service.js

Lines changed: 1 addition & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -42,9 +42,7 @@ exports.list = async () => {
4242
*/
4343
exports.create = async (user) => {
4444
// Set provider to local
45-
if (!user.provider) {
46-
user.provider = 'local';
47-
}
45+
if (!user.provider) user.provider = 'local';
4846
// confirming to secure password policies
4947
if (user.password) {
5048
// done in model, let this comment for information if one day joi.zxcvbn is not ok / sufficient

modules/users/tests/user.crud.tests.js

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -132,6 +132,7 @@ describe('User CRUD Unit Tests :', () => {
132132

133133
expect(result.body.user._id).toBe(result.body.user.id);
134134
expect(result.body.user.email).toBe(_userEdited.email);
135+
expect(result.body.user.provider).toBe('local');
135136
expect(result.body.user.roles).toBeInstanceOf(Array);
136137
expect(result.body.user.roles).toHaveLength(1);
137138
expect(result.body.user.roles).toEqual(

0 commit comments

Comments
 (0)