Skip to content

Commit 2f2f8d3

Browse files
committedNov 28, 2024
feat(api): create user api method to update assessment instructions info
1 parent b70a6d9 commit 2f2f8d3

File tree

4 files changed

+29
-6
lines changed

4 files changed

+29
-6
lines changed
 
Original file line numberDiff line numberDiff line change
@@ -1,5 +1,5 @@
11
const rememberUserHasSeenAssessmentInstructions = function ({ userId, userRepository }) {
2-
return userRepository.updateHasSeenAssessmentInstructionsToTrue(userId);
2+
return userRepository.updateAssessmentInstructionsInfoAsSeen({ userId });
33
};
44

55
export { rememberUserHasSeenAssessmentInstructions };

Diff for: ‎api/src/evaluation/infrastructure/repositories/user-repository.js

+21-2
Original file line numberDiff line numberDiff line change
@@ -1,7 +1,16 @@
11
/**
2-
* @typedef {import ("./index.js").UserApi} UserApi
2+
* @typedef {import ('./index.js').UserApi} UserApi
33
*/
44

5+
/**
6+
* @function
7+
* @param {Object} params
8+
* @param {UserApi} params.userApi
9+
*/
10+
const updateMarkLevelSevenInfoAsSeen = async function ({ userId, userApi }) {
11+
return userApi.markLevelSevenInfoAsSeen({ userId });
12+
};
13+
514
/**
615
* @function
716
* @param {Object} params
@@ -12,4 +21,14 @@ const updateHasSeenNewDashboardInfo = async function ({ userId, userApi }) {
1221
return userApi.markNewDashboardInfoAsSeen({ userId });
1322
};
1423

15-
export { updateHasSeenNewDashboardInfo };
24+
/**
25+
* @function
26+
* @param {Object} params
27+
* @param {UserApi} params.userApi
28+
* @param {number} params.userId
29+
*/
30+
const updateAssessmentInstructionsInfoAsSeen = async function ({ userId, userApi }) {
31+
return userApi.markAssessmentInstructionsInfoAsSeen({ userId });
32+
};
33+
34+
export { updateAssessmentInstructionsInfoAsSeen, updateHasSeenNewDashboardInfo, updateMarkLevelSevenInfoAsSeen };

Diff for: ‎api/src/identity-access-management/application/api/users-api.js

+4
Original file line numberDiff line numberDiff line change
@@ -8,6 +8,10 @@ export const markNewDashboardInfoAsSeen = async ({ userId }) => {
88
return usecases.markUserHasSeenNewDashboardInfo({ userId });
99
};
1010

11+
export const markAssessmentInstructionsInfoAsSeen = async ({ userId }) => {
12+
return usecases.markAssessmentInstructionsInfoAsSeen({ userId });
13+
};
14+
1115
export const getUserDetailsByUserIds = async ({ userIds }) => {
1216
const users = await usecases.getUserDetailsByUserIds({ userIds });
1317
return users.map((user) => new UserDTO(user));

Diff for: ‎api/tests/evaluation/unit/domain/usecases/remember-user-has-seen-assessment-instructions_test.js

+3-3
Original file line numberDiff line numberDiff line change
@@ -5,20 +5,20 @@ describe('Unit | UseCase | remember-user-has-seen-assessment-instructions', func
55
let userRepository;
66

77
beforeEach(function () {
8-
userRepository = { updateHasSeenAssessmentInstructionsToTrue: sinon.stub() };
8+
userRepository = { updateAssessmentInstructionsInfoAsSeen: sinon.stub() };
99
});
1010

1111
it('should update has seen assessment instructions', async function () {
1212
// given
1313
const userId = 'userId';
1414
const updatedUser = Symbol('updateduser');
15-
userRepository.updateHasSeenAssessmentInstructionsToTrue.resolves(updatedUser);
15+
userRepository.updateAssessmentInstructionsInfoAsSeen.resolves(updatedUser);
1616

1717
// when
1818
const actualUpdatedUser = await rememberUserHasSeenAssessmentInstructions({ userId, userRepository });
1919

2020
// then
21-
expect(userRepository.updateHasSeenAssessmentInstructionsToTrue).to.have.been.calledWithExactly(userId);
21+
expect(userRepository.updateAssessmentInstructionsInfoAsSeen).to.have.been.calledWithExactly({ userId });
2222
expect(actualUpdatedUser).to.equal(updatedUser);
2323
});
2424
});

0 commit comments

Comments
 (0)