diff --git a/server/tests/middleware/auth.test.js b/server/tests/middleware/auth.test.js index bc28c24..74b68dc 100644 --- a/server/tests/middleware/auth.test.js +++ b/server/tests/middleware/auth.test.js @@ -93,13 +93,9 @@ describe('Authentication Middleware', () => { const user = await createTestUser({ username: 'testuser', role: 'admin' }); console.log('Created user:', user.toJSON()); // Debug log - // Get the actual string value of the UUID - const userId = user.getDataValue('id') || user.id; - console.log('User ID for JWT:', userId, typeof userId); - const token = jwt.sign( { - userId: userId, + userId: user.id, username: user.username, role: user.role, email: user.email @@ -117,7 +113,7 @@ describe('Authentication Middleware', () => { await authenticateToken(req, res, next); expect(req.user).to.exist; - expect(req.user.id).to.equal(userId); + expect(req.user.id).to.equal(user.id); expect(req.user.username).to.equal(user.username); expect(req.user.role).to.equal(user.role); expect(next.errors).to.have.length(0); @@ -127,12 +123,9 @@ describe('Authentication Middleware', () => { const tenant = await createTestTenant({ slug: 'test-tenant' }); const user = await createTestUser({ username: 'testuser', tenant_id: tenant.id }); - // Get the actual string value of the UUID - const userId = user.getDataValue('id') || user.id; - const token = jwt.sign( { - userId: userId, + userId: user.id, username: user.username, role: user.role, tenantId: tenant.slug @@ -181,12 +174,9 @@ describe('Authentication Middleware', () => { is_active: false }); - // Get the actual string value of the UUID - const userId = user.getDataValue('id') || user.id; - const token = jwt.sign( { - userId: userId, + userId: user.id, username: user.username }, process.env.JWT_SECRET || 'test-secret', diff --git a/server/tests/setup.js b/server/tests/setup.js index 8f5d48b..3430def 100644 --- a/server/tests/setup.js +++ b/server/tests/setup.js @@ -129,7 +129,11 @@ async function createTestUser(userData = {}) { }); } + // Generate a simple UUID-like string for testing + const testId = 'test-' + Date.now() + '-' + Math.random().toString(36).substr(2, 9); + const defaultUserData = { + id: testId, // Use explicit test ID username: 'testuser', email: 'test@example.com', password_hash: '$2b$10$example.hash.for.testing.purposes.only',