Skip to content

Commit 28f7553

Browse files
changes after randhir review
1 parent 146c81f commit 28f7553

File tree

2 files changed

+9
-6
lines changed

2 files changed

+9
-6
lines changed

test/fixtures/userStatus/userStatus.js

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -13,12 +13,12 @@ const userStatusDataForNewUser = {
1313
};
1414

1515
const userStatusDataAfterSignup = {
16-
currentStatus: { state: "ONBOARDING", until: "" },
16+
currentStatus: { state: "ONBOARDING" },
1717
monthlyHours: { committed: 0 },
1818
};
1919

20-
const userStatusDataAfterFillJoin = {
21-
currentStatus: { state: "ONBOARDING", until: "" },
20+
const userStatusDataAfterFillingJoinSection = {
21+
currentStatus: { state: "ONBOARDING" },
2222
monthlyHours: { committed: 40 },
2323
};
2424

@@ -64,7 +64,7 @@ const generateUserStatusData = (state, updatedAt, from, until = "", message = ""
6464
module.exports = {
6565
userStatusDataForNewUser,
6666
userStatusDataAfterSignup,
67-
userStatusDataAfterFillJoin,
67+
userStatusDataAfterFillingJoinSection,
6868
userStatusDataForOooState,
6969
oooStatusDataForShortDuration,
7070
generateUserStatusData,

test/integration/users.test.js

Lines changed: 5 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -16,7 +16,10 @@ const searchParamValues = require("../fixtures/user/search")();
1616

1717
const config = require("config");
1818
const joinData = require("../fixtures/user/join");
19-
const { userStatusDataAfterSignup, userStatusDataAfterFillJoin } = require("../fixtures/userStatus/userStatus");
19+
const {
20+
userStatusDataAfterSignup,
21+
userStatusDataAfterFillingJoinSection,
22+
} = require("../fixtures/userStatus/userStatus");
2023
const { addJoinData, addOrUpdate } = require("../../models/users");
2124
const userStatusModel = require("../../models/userStatus");
2225

@@ -695,7 +698,7 @@ describe("Users", function () {
695698
let userStatusData;
696699
beforeEach(async function () {
697700
await userStatusModel.updateUserStatus(userId, userStatusDataAfterSignup);
698-
const updateStatus = await userStatusModel.updateUserStatus(userId, userStatusDataAfterFillJoin);
701+
const updateStatus = await userStatusModel.updateUserStatus(userId, userStatusDataAfterFillingJoinSection);
699702
userStatusData = (await firestore.collection("usersStatus").doc(updateStatus.id).get()).data();
700703
});
701704
it("should return 409 if the data already present", function (done) {

0 commit comments

Comments
 (0)