see you tomorrow not sure of that

1024pix · Feb 5, 2025 · 5919150 · 5919150
1 parent 698c36c
commit 5919150
Show file tree

Hide file tree

Showing 4 changed files with 17 additions and 5 deletions.
diff --git a/api/src/quest/domain/models/Quest.js b/api/src/quest/domain/models/Quest.js
@@ -48,6 +48,8 @@ class Quest {
    * @param {Eligibility} eligibility
    */
   isEligible(eligibility) {
+    if (this.eligibilityRequirements.length === 0) return false;
+
     return this.eligibilityRequirements.every((eligibilityRequirement) =>
       this.#checkRequirement(eligibilityRequirement, eligibility),
     );

diff --git a/api/src/quest/domain/usecases/get-quest-results-for-campaign-participation.js b/api/src/quest/domain/usecases/get-quest-results-for-campaign-participation.js
@@ -1,8 +1,9 @@
-import * as campaignParticipationRepository from '../../../prescription/campaign-participation/infrastructure/repositories/campaign-participation-repository.js';
+import * as dirtyImportcampaignParticipationRepository from '../../../prescription/campaign-participation/infrastructure/repositories/campaign-participation-repository.js';
 
 export const getQuestResultsForCampaignParticipation = async ({
   userId,
   campaignParticipationId,
+  campaignParticipationRepository = dirtyImportcampaignParticipationRepository,
   questRepository,
   eligibilityRepository,
   rewardRepository,
@@ -16,6 +17,7 @@ export const getQuestResultsForCampaignParticipation = async ({
   const campaignParticipation = await campaignParticipationRepository.get(campaignParticipationId);
 
   const eligibilities = await eligibilityRepository.find({ userId });
+
   const eligibility = eligibilities.find(
     (eligibility) => eligibility.organizationLearner.id === campaignParticipation.organizationLearnerId,
   );

diff --git a/api/tests/quest/unit/domain/models/Quest_test.js b/api/tests/quest/unit/domain/models/Quest_test.js
@@ -7,6 +7,8 @@ import { expect } from '../../../../test-helper.js';
 
 describe('Quest | Unit | Domain | Models | Quest ', function () {
   describe('#isEligible', function () {
+    //TODO: add egde case quest without eligibilityRequirement to return false ?
+
     describe('when comparison is "all"', function () {
       describe('when data to test is a simple value', function () {
         let quest;

diff --git a/api/tests/quest/unit/domain/usecases/get-quest-results-for-campaign-participation_test.js b/api/tests/quest/unit/domain/usecases/get-quest-results-for-campaign-participation_test.js
@@ -13,7 +13,7 @@ describe('Quest | Unit | Domain | Usecases | getQuestResultsForCampaignParticipa
     campaignParticipationId = 2;
     questRepository = { findAll: sinon.stub() };
     eligibilityRepository = { find: sinon.stub() };
-    campaignParticipationRepository = { getCampaignByParticipationId: sinon.stub() };
+    campaignParticipationRepository = { get: sinon.stub() };
     rewardRepository = { getByQuestAndUserId: sinon.stub() };
   });
 
@@ -47,8 +47,13 @@ describe('Quest | Unit | Domain | Usecases | getQuestResultsForCampaignParticipa
       }),
     ]);
 
+    campaignParticipationRepository.get
+      .withArgs(campaignParticipationId)
+      .resolves({ organizationLearnerId: 10, targetProfileId: 50 });
+
     eligibilityRepository.find.withArgs({ userId }).resolves([
       new Eligibility({
+        organizationLearner: { id: 10 },
         campaignParticipations: [{ id: wrongCampaignParticipationId, targetProfileId: 40 }],
       }),
     ]);
@@ -81,12 +86,13 @@ describe('Quest | Unit | Domain | Usecases | getQuestResultsForCampaignParticipa
         rewardId: 20,
       }),
     ]);
-    campaignParticipationRepository.getCampaignByParticipationId
-      .withArgs({ campaignParticipationId })
-      .resolves([{ targetProfileId: 40 }]);
+    campaignParticipationRepository.get
+      .withArgs(campaignParticipationId)
+      .resolves({ organizationLearnerId: 10, targetProfileId: 40 });
 
     eligibilityRepository.find.withArgs({ userId }).resolves([
       new Eligibility({
+        organizationLearner: { id: 10 },
         campaignParticipations: [{ id: campaignParticipationId, targetProfileId: 40 }],
       }),
     ]);