Skip to content

Commit

Permalink
ISSUE #5412 fix test file
Browse files Browse the repository at this point in the history
  • Loading branch information
carmenfan committed Mar 9, 2025
1 parent 647ce01 commit 173f36d
Show file tree
Hide file tree
Showing 2 changed files with 3 additions and 9 deletions.
2 changes: 1 addition & 1 deletion backend/src/v5/services/sessions.js
Original file line number Diff line number Diff line change
Expand Up @@ -19,11 +19,11 @@ const { USERS_DB_NAME } = require('../models/users.constants');
const config = require('../utils/config');
const db = require('../handler/db');

const { ensureIndicesExist } = require('../models/users');
const { events } = require('./eventsManager/eventsManager.constants');
const expressSession = require('express-session');
const { generateUUID } = require('../utils/helper/uuids');
const { publish } = require('./eventsManager/eventsManager');
const { ensureIndicesExist } = require('../models/users');

const Sessions = { };
const initialiseSession = async () => {
Expand Down
10 changes: 2 additions & 8 deletions backend/tests/v5/unit/middleware/sessions.test.js
Original file line number Diff line number Diff line change
Expand Up @@ -16,7 +16,6 @@
*/

const { cloneDeep } = require('lodash');

const { src } = require('../../helper/path');
const { determineTestGroup, generateRandomString } = require('../../helper/services');

Expand All @@ -36,18 +35,14 @@ const { SOCKET_HEADER } = require(`${src}/services/chat/chat.constants`);

// Need to mock these 2 to ensure we are not trying to create a real session configuration
jest.mock('express-session', () => () => { });
jest.mock('../../../../src/v5/handler/db', () => ({
...jest.requireActual('../../../../src/v5/handler/db'),
getSessionStore: () => { },
}));

jest.mock('../../../../src/v5/services/sessions');
jest.mock('../../../../src/v5/handler/db');

jest.mock('../../../../src/v5/utils/helper/strings', () => ({
...jest.requireActual('../../../../src/v5/utils/helper/strings'),
getURLDomain: () => 'abc.com',
}));

jest.mock('../../../../src/v5/services/sessions');
const SessionService = require(`${src}/services/sessions`);

const sessionMiddleware = jest.fn().mockImplementation((req, res, next) => next());
Expand All @@ -63,7 +58,6 @@ const pluginAgent = generateRandomString();
const webBrowserUserAgent = generateRandomString();

UserAgentHelper.isFromWebBrowser.mockImplementation((userAgent) => userAgent === webBrowserUserAgent);

const testDestroySession = () => {
const req = {
session: { destroy: (callback) => { callback(); }, user: { username: 'user1' } },
Expand Down

0 comments on commit 173f36d

Please sign in to comment.