Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jxl9/ach tracking tests #223

Open
wants to merge 36 commits into
base: master
Choose a base branch
from
Open
Changes from 1 commit
Commits
Show all changes
36 commits
Select commit Hold shift + click to select a range
a285a1a
implement achievements tracking logic
jasmineee-li Apr 28, 2024
4ba5767
Merge branch 'master' into jxl9/achievements-tracking
neketka Apr 28, 2024
47ce5db
Fix achievement service import
neketka Apr 28, 2024
8a509c0
Add cascading delete to achievement tracker
neketka Apr 28, 2024
e82d43b
implement event lists from org
neketka Apr 29, 2024
f41dccd
Made achievement admin frontend
neketka Apr 29, 2024
d781b21
Bug fixes
neketka Apr 29, 2024
22864a3
fixed bugs, still need to pass tests
jasmineee-li Apr 29, 2024
c6b4a9d
Merge branch 'jxl9/achievements-tracking' into nikita/admin-and-data-…
neketka Apr 29, 2024
c89abec
Fix event trackers
neketka Apr 29, 2024
92680c3
Fix flutter build
neketka Apr 29, 2024
071bb8c
Finish achievement tracking
neketka Apr 29, 2024
d920c37
Fix emit order
neketka Apr 29, 2024
5ec6cb5
Fix other emit error
neketka Apr 29, 2024
f31fab2
Fix build errors
neketka Apr 30, 2024
797a559
Connected achievements to backend
neketka Apr 30, 2024
7e4b9fe
Fix inconsistencies
neketka Apr 30, 2024
2ab00ee
Complete integration
neketka Apr 30, 2024
49524af
Remove hardcoded complete
neketka Apr 30, 2024
351156f
Comment failing tests
neketka Apr 30, 2024
7b12a81
Attempt to fix tests
neketka Apr 30, 2024
3cf8e31
Fix more e2e failures
neketka Apr 30, 2024
9518e6a
Add exception for e2e tests in client service
neketka Apr 30, 2024
796f15f
Added extra logging
neketka Apr 30, 2024
11cdb0c
Fix incorrect nextChallenge
neketka Apr 30, 2024
51eeb31
Fix tests
neketka Apr 30, 2024
8782785
Fix nulls in test
neketka Apr 30, 2024
65e9796
Set progress limit
neketka Apr 30, 2024
dc21d2c
Merge branch 'master' into nikita/admin-and-data-fixes
neketka Apr 30, 2024
dbd0f40
Add manager rule for achievements
neketka Apr 30, 2024
a2c8538
Achivement trackers are made for all
neketka Apr 30, 2024
9ea5d39
Clarified prisma query
neketka Apr 30, 2024
1b12b5c
Merge branch 'master' into jxl9/ach-tracking-tests
jasmineee-li May 1, 2024
ce8c44b
Ran updateapi, ran tracker tests (successfully pass)
jasmineee-li May 1, 2024
01495f3
Add achievement tracking tests
jasmineee-li May 6, 2024
4b29753
Fix code style
jasmineee-li May 6, 2024
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev Previous commit
Next Next commit
Merge branch 'master' into jxl9/achievements-tracking
neketka committed Apr 28, 2024
commit 4ba57673820f0f01a87a0dc7b90b88b2b73d7355
36 changes: 24 additions & 12 deletions server/src/challenge/challenge.service.ts
Original file line number Diff line number Diff line change
@@ -101,7 +101,7 @@ export class ChallengeService {
const eventTracker: EventTracker =
await this.eventService.getCurrentEventTrackerForUser(user);

// const achievementTracker : AchievementTracker =
// const achievementTracker : AchievementTracker =
// await this.achievementService.getAchievementsByIdsForAbility(user.ability, [eventTracker.id]);

const alreadyDone =
@@ -159,18 +159,30 @@ export class ChallengeService {
);

// check if the challenge is part of a journey
const isJourney = (await this.prisma.prevChallenge.count({
where: {
userId: user.id,
challengeId: eventTracker.curChallengeId,
trackerId: eventTracker.id,
},
})) === (await this.prisma.eventTracker.count({
where: { id: eventTracker.id }, // CHECK
}));
const isJourney =
(await this.prisma.prevChallenge.count({
where: {
userId: user.id,
challengeId: eventTracker.curChallengeId,
trackerId: eventTracker.id,
},
})) ===
(await this.prisma.eventTracker.count({
where: { id: eventTracker.id }, // CHECK
}));

await this.achievementService.checkAchievementProgress(user, challengeId, isJourney);

await this.achievementService.checkAchievementProgress(
user,
challengeId,
isJourney,
);

await this.eventService.emitUpdateLeaderPosition({
playerId: newUser.id,
newTotalScore: newUser.score,
newEventScore: newEvTracker.score,
eventId: newEvTracker.eventId,
});

return true;
}
You are viewing a condensed version of this merge commit. You can view the full changes here.