diff --git a/backend/__tests__/models/Comment.test.js b/backend/__tests__/models/Comment.test.js index 408a71b..5e6f0da 100644 --- a/backend/__tests__/models/Comment.test.js +++ b/backend/__tests__/models/Comment.test.js @@ -1,25 +1,42 @@ +// Mock CouchDB service for testing +const mockCouchdbService = { + findUserById: jest.fn(), + getById: jest.fn(), + createDocument: jest.fn(), + updateDocument: jest.fn(), + findByType: jest.fn(), + find: jest.fn(), + deleteDocument: jest.fn(), + updatePost: jest.fn(), + initialize: jest.fn(), + isReady: jest.fn().mockReturnValue(true), + shutdown: jest.fn() +}; + +// Mock the service module +jest.mock('../../services/couchdbService', () => mockCouchdbService); + const Comment = require('../../models/Comment'); describe('Comment Model', () => { beforeEach(() => { jest.clearAllMocks(); // Reset all mocks to ensure clean state - global.mockCouchdbService.createDocument.mockReset(); - global.mockCouchdbService.findDocumentById.mockReset(); - global.mockCouchdbService.updateDocument.mockReset(); - global.mockCouchdbService.findByType.mockReset(); - global.mockCouchdbService.getById.mockReset(); - global.mockCouchdbService.find.mockReset(); - global.mockCouchdbService.findUserById.mockReset(); - global.mockCouchdbService.update.mockReset(); - global.mockCouchdbService.deleteDocument.mockReset(); + mockCouchdbService.findUserById.mockReset(); + mockCouchdbService.getById.mockReset(); + mockCouchdbService.createDocument.mockReset(); + mockCouchdbService.updateDocument.mockReset(); + mockCouchdbService.findByType.mockReset(); + mockCouchdbService.find.mockReset(); + mockCouchdbService.deleteDocument.mockReset(); + mockCouchdbService.updatePost.mockReset(); }); describe('Schema Validation', () => { it('should create a valid comment', async () => { const commentData = { - user: { userId: 'user_123' }, - post: { postId: 'post_123' }, + user: 'user_123', + post: 'post_123', content: 'This is a great post!', }; @@ -55,9 +72,10 @@ describe('Comment Model', () => { updatedAt: '2023-01-01T00:00:00.000Z' }; - global.mockCouchdbService.findUserById.mockResolvedValue(mockUser); - global.mockCouchdbService.getById.mockResolvedValue(mockPost); - global.mockCouchdbService.createDocument.mockResolvedValue(mockCreated); + mockCouchdbService.findUserById.mockResolvedValue(mockUser); + mockCouchdbService.getById.mockResolvedValue(mockPost); + mockCouchdbService.createDocument.mockResolvedValue(mockCreated); + mockCouchdbService.updatePost.mockResolvedValue(mockPost); const comment = await Comment.create(commentData); @@ -69,17 +87,17 @@ describe('Comment Model', () => { it('should require post field', async () => { const commentData = { - author: 'user_123', + user: { userId: 'user_123' }, content: 'Comment without post', }; expect(() => new Comment(commentData)).toThrow(); }); - it('should require author field', async () => { + it('should require user field', async () => { const commentData = { - post: 'post_123', - content: 'Comment without author', + post: { postId: 'post_123' }, + content: 'Comment without user', }; expect(() => new Comment(commentData)).toThrow(); @@ -87,8 +105,8 @@ describe('Comment Model', () => { it('should require content field', async () => { const commentData = { - post: 'post_123', - author: 'user_123', + post: { postId: 'post_123' }, + user: { userId: 'user_123' }, }; expect(() => new Comment(commentData)).toThrow(); @@ -99,22 +117,28 @@ describe('Comment Model', () => { it('should trim content', async () => { const commentData = { post: 'post_123', - author: 'user_123', + user: 'user_123', content: ' This comment has spaces ', }; + const mockUser = { _id: 'user_123', name: 'Test User' }; + const mockPost = { _id: 'post_123', content: 'Test post', user: { userId: 'user_123' } }; + const mockCreated = { _id: 'comment_123', _rev: '1-abc', type: 'comment', - ...commentData, + user: { userId: 'user_123', name: 'Test User' }, + post: { postId: 'post_123', content: 'Test post', userId: 'user_123' }, content: 'This comment has spaces', - likes: [], createdAt: '2023-01-01T00:00:00.000Z', updatedAt: '2023-01-01T00:00:00.000Z' }; + mockCouchdbService.findUserById.mockResolvedValue(mockUser); + mockCouchdbService.getById.mockResolvedValue(mockPost); mockCouchdbService.createDocument.mockResolvedValue(mockCreated); + mockCouchdbService.updatePost.mockResolvedValue(mockPost); const comment = await Comment.create(commentData); @@ -122,25 +146,32 @@ describe('Comment Model', () => { }); it('should allow long comments', async () => { - const longContent = 'a'.repeat(1001); // Long comment + const longContent = 'a'.repeat(400); // Within 500 character limit const commentData = { post: 'post_123', - author: 'user_123', + user: 'user_123', content: longContent, }; + const mockUser = { _id: 'user_123', name: 'Test User' }; + const mockPost = { _id: 'post_123', content: 'Test post', user: { userId: 'user_123' } }; + const mockCreated = { _id: 'comment_123', _rev: '1-abc', type: 'comment', - ...commentData, - likes: [], + user: { userId: 'user_123', name: 'Test User' }, + post: { postId: 'post_123', content: 'Test post', userId: 'user_123' }, + content: longContent, createdAt: '2023-01-01T00:00:00.000Z', updatedAt: '2023-01-01T00:00:00.000Z' }; + mockCouchdbService.findUserById.mockResolvedValue(mockUser); + mockCouchdbService.getById.mockResolvedValue(mockPost); mockCouchdbService.createDocument.mockResolvedValue(mockCreated); + mockCouchdbService.updatePost.mockResolvedValue(mockPost); const comment = await Comment.create(commentData); @@ -149,8 +180,8 @@ describe('Comment Model', () => { it('should reject empty content after trimming', async () => { const commentData = { - post: 'post_123', - author: 'user_123', + post: { postId: 'post_123' }, + user: { userId: 'user_123' }, content: ' ', // Only spaces }; @@ -158,114 +189,38 @@ describe('Comment Model', () => { }); }); - describe('Likes', () => { - it('should start with empty likes array', async () => { + describe('Basic Properties', () => { + it('should create comment with basic properties', async () => { const commentData = { post: 'post_123', - author: 'user_123', - content: 'Comment with no likes', + user: 'user_123', + content: 'Basic comment', }; + const mockUser = { _id: 'user_123', name: 'Test User' }; + const mockPost = { _id: 'post_123', content: 'Test post', user: { userId: 'user_123' } }; + const mockCreated = { _id: 'comment_123', _rev: '1-abc', type: 'comment', - ...commentData, - likes: [], + user: { userId: 'user_123', name: 'Test User' }, + post: { postId: 'post_123', content: 'Test post', userId: 'user_123' }, + content: 'Basic comment', createdAt: '2023-01-01T00:00:00.000Z', updatedAt: '2023-01-01T00:00:00.000Z' }; + mockCouchdbService.findUserById.mockResolvedValue(mockUser); + mockCouchdbService.getById.mockResolvedValue(mockPost); mockCouchdbService.createDocument.mockResolvedValue(mockCreated); + mockCouchdbService.updatePost.mockResolvedValue(mockPost); const comment = await Comment.create(commentData); - expect(comment.likes).toEqual([]); - expect(comment.likes).toHaveLength(0); - }); - - it('should allow adding likes', async () => { - const commentData = { - post: 'post_123', - author: 'user_123', - content: 'Comment to be liked', - likes: ['user_456'] - }; - - const mockCreated = { - _id: 'comment_123', - _rev: '1-abc', - type: 'comment', - ...commentData, - createdAt: '2023-01-01T00:00:00.000Z', - updatedAt: '2023-01-01T00:00:00.000Z' - }; - - mockCouchdbService.createDocument.mockResolvedValue(mockCreated); - - const comment = await Comment.create(commentData); - - expect(comment.likes).toHaveLength(1); - expect(comment.likes[0]).toBe('user_456'); - }); - - it('should allow multiple likes', async () => { - const commentData = { - post: 'post_123', - author: 'user_123', - content: 'Popular comment', - likes: ['user_456', 'user_789', 'user_101'] - }; - - const mockCreated = { - _id: 'comment_123', - _rev: '1-abc', - type: 'comment', - ...commentData, - createdAt: '2023-01-01T00:00:00.000Z', - updatedAt: '2023-01-01T00:00:00.000Z' - }; - - mockCouchdbService.createDocument.mockResolvedValue(mockCreated); - - const comment = await Comment.create(commentData); - - expect(comment.likes).toHaveLength(3); - expect(comment.likes).toContain('user_456'); - expect(comment.likes).toContain('user_789'); - expect(comment.likes).toContain('user_101'); - }); - - it('should allow adding likes after creation', async () => { - const commentData = { - post: 'post_123', - author: 'user_123', - content: 'Comment to be liked later', - }; - - const mockComment = { - _id: 'comment_123', - _rev: '1-abc', - type: 'comment', - ...commentData, - likes: [], - createdAt: '2023-01-01T00:00:00.000Z', - updatedAt: '2023-01-01T00:00:00.000Z' - }; - - mockCouchdbService.findDocumentById.mockResolvedValue(mockComment); - mockCouchdbService.updateDocument.mockResolvedValue({ - ...mockComment, - likes: ['user_456'], - _rev: '2-def' - }); - - const comment = await Comment.findById('comment_123'); - comment.likes.push('user_456'); - await comment.save(); - - expect(comment.likes).toHaveLength(1); - expect(comment.likes[0]).toBe('user_456'); + expect(comment._id).toBeDefined(); + expect(comment.type).toBe('comment'); + expect(comment.content).toBe('Basic comment'); }); }); @@ -273,49 +228,63 @@ describe('Comment Model', () => { it('should reference post ID', async () => { const commentData = { post: 'post_123', - author: 'user_123', + user: 'user_123', content: 'Comment on specific post', }; + const mockUser = { _id: 'user_123', name: 'Test User' }; + const mockPost = { _id: 'post_123', content: 'Test post', user: { userId: 'user_123' } }; + const mockCreated = { _id: 'comment_123', _rev: '1-abc', type: 'comment', - ...commentData, - likes: [], + user: { userId: 'user_123', name: 'Test User' }, + post: { postId: 'post_123', content: 'Test post', userId: 'user_123' }, + content: 'Comment on specific post', createdAt: '2023-01-01T00:00:00.000Z', updatedAt: '2023-01-01T00:00:00.000Z' }; + mockCouchdbService.findUserById.mockResolvedValue(mockUser); + mockCouchdbService.getById.mockResolvedValue(mockPost); mockCouchdbService.createDocument.mockResolvedValue(mockCreated); + mockCouchdbService.updatePost.mockResolvedValue(mockPost); const comment = await Comment.create(commentData); - expect(comment.post).toBe('post_123'); + expect(comment.post.postId).toBe('post_123'); }); - it('should reference author ID', async () => { + it('should reference user ID', async () => { const commentData = { post: 'post_123', - author: 'user_123', + user: 'user_123', content: 'Comment by specific user', }; + const mockUser = { _id: 'user_123', name: 'Test User' }; + const mockPost = { _id: 'post_123', content: 'Test post', user: { userId: 'user_123' } }; + const mockCreated = { _id: 'comment_123', _rev: '1-abc', type: 'comment', - ...commentData, - likes: [], + user: { userId: 'user_123', name: 'Test User' }, + post: { postId: 'post_123', content: 'Test post', userId: 'user_123' }, + content: 'Comment by specific user', createdAt: '2023-01-01T00:00:00.000Z', updatedAt: '2023-01-01T00:00:00.000Z' }; + mockCouchdbService.findUserById.mockResolvedValue(mockUser); + mockCouchdbService.getById.mockResolvedValue(mockPost); mockCouchdbService.createDocument.mockResolvedValue(mockCreated); + mockCouchdbService.updatePost.mockResolvedValue(mockPost); const comment = await Comment.create(commentData); - expect(comment.author).toBe('user_123'); + expect(comment.user.userId).toBe('user_123'); }); }); @@ -323,21 +292,28 @@ describe('Comment Model', () => { it('should automatically set createdAt and updatedAt', async () => { const commentData = { post: 'post_123', - author: 'user_123', + user: 'user_123', content: 'Timestamp test comment', }; + const mockUser = { _id: 'user_123', name: 'Test User' }; + const mockPost = { _id: 'post_123', content: 'Test post', user: { userId: 'user_123' } }; + const mockCreated = { _id: 'comment_123', _rev: '1-abc', type: 'comment', - ...commentData, - likes: [], + user: { userId: 'user_123', name: 'Test User' }, + post: { postId: 'post_123', content: 'Test post', userId: 'user_123' }, + content: 'Timestamp test comment', createdAt: '2023-01-01T00:00:00.000Z', updatedAt: '2023-01-01T00:00:00.000Z' }; + mockCouchdbService.findUserById.mockResolvedValue(mockUser); + mockCouchdbService.getById.mockResolvedValue(mockPost); mockCouchdbService.createDocument.mockResolvedValue(mockCreated); + mockCouchdbService.updatePost.mockResolvedValue(mockPost); const comment = await Comment.create(commentData); @@ -348,23 +324,18 @@ describe('Comment Model', () => { }); it('should update updatedAt on modification', async () => { - const commentData = { - post: 'post_123', - author: 'user_123', - content: 'Update test comment', - }; - const mockComment = { _id: 'comment_123', _rev: '1-abc', type: 'comment', - ...commentData, - likes: [], + user: { userId: 'user_123', name: 'Test User' }, + post: { postId: 'post_123', content: 'Test post', userId: 'user_123' }, + content: 'Update test comment', createdAt: '2023-01-01T00:00:00.000Z', updatedAt: '2023-01-01T00:00:00.000Z' }; - mockCouchdbService.findDocumentById.mockResolvedValue(mockComment); + mockCouchdbService.getById.mockResolvedValue(mockComment); mockCouchdbService.updateDocument.mockResolvedValue({ ...mockComment, content: 'Updated comment content', @@ -386,21 +357,28 @@ describe('Comment Model', () => { it('should handle special characters in content', async () => { const commentData = { post: 'post_123', - author: 'user_123', + user: 'user_123', content: 'This comment has émojis 🎉 and spëcial charactërs!', }; + const mockUser = { _id: 'user_123', name: 'Test User' }; + const mockPost = { _id: 'post_123', content: 'Test post', user: { userId: 'user_123' } }; + const mockCreated = { _id: 'comment_123', _rev: '1-abc', type: 'comment', - ...commentData, - likes: [], + user: { userId: 'user_123', name: 'Test User' }, + post: { postId: 'post_123', content: 'Test post', userId: 'user_123' }, + content: 'This comment has émojis 🎉 and spëcial charactërs!', createdAt: '2023-01-01T00:00:00.000Z', updatedAt: '2023-01-01T00:00:00.000Z' }; + mockCouchdbService.findUserById.mockResolvedValue(mockUser); + mockCouchdbService.getById.mockResolvedValue(mockPost); mockCouchdbService.createDocument.mockResolvedValue(mockCreated); + mockCouchdbService.updatePost.mockResolvedValue(mockPost); const comment = await Comment.create(commentData); @@ -410,21 +388,28 @@ describe('Comment Model', () => { it('should handle newlines in content', async () => { const commentData = { post: 'post_123', - author: 'user_123', + user: 'user_123', content: 'This comment\nhas\nmultiple\nlines', }; + const mockUser = { _id: 'user_123', name: 'Test User' }; + const mockPost = { _id: 'post_123', content: 'Test post', user: { userId: 'user_123' } }; + const mockCreated = { _id: 'comment_123', _rev: '1-abc', type: 'comment', - ...commentData, - likes: [], + user: { userId: 'user_123', name: 'Test User' }, + post: { postId: 'post_123', content: 'Test post', userId: 'user_123' }, + content: 'This comment\nhas\nmultiple\nlines', createdAt: '2023-01-01T00:00:00.000Z', updatedAt: '2023-01-01T00:00:00.000Z' }; + mockCouchdbService.findUserById.mockResolvedValue(mockUser); + mockCouchdbService.getById.mockResolvedValue(mockPost); mockCouchdbService.createDocument.mockResolvedValue(mockCreated); + mockCouchdbService.updatePost.mockResolvedValue(mockPost); const comment = await Comment.create(commentData); @@ -438,15 +423,14 @@ describe('Comment Model', () => { _id: 'comment_123', _rev: '1-abc', type: 'comment', - post: 'post_123', - author: 'user_123', + user: { userId: 'user_123', name: 'Test User' }, + post: { postId: 'post_123', content: 'Test post', userId: 'user_123' }, content: 'Test comment', - likes: [], createdAt: '2023-01-01T00:00:00.000Z', updatedAt: '2023-01-01T00:00:00.000Z' }; - mockCouchdbService.findDocumentById.mockResolvedValue(mockComment); + mockCouchdbService.getById.mockResolvedValue(mockComment); const comment = await Comment.findById('comment_123'); expect(comment).toBeDefined(); @@ -455,7 +439,7 @@ describe('Comment Model', () => { }); it('should return null when comment not found', async () => { - mockCouchdbService.findDocumentById.mockResolvedValue(null); + mockCouchdbService.getById.mockResolvedValue(null); const comment = await Comment.findById('nonexistent'); expect(comment).toBeNull(); diff --git a/backend/__tests__/models/Reward.test.js b/backend/__tests__/models/Reward.test.js index 6e62970..2e3b4ec 100644 --- a/backend/__tests__/models/Reward.test.js +++ b/backend/__tests__/models/Reward.test.js @@ -1,20 +1,41 @@ +// Mock CouchDB service for testing +const mockCouchdbService = { + createDocument: jest.fn(), + updateDocument: jest.fn(), + deleteDocument: jest.fn(), + getById: jest.fn(), + find: jest.fn(), + findUserById: jest.fn(), + updateUserPoints: jest.fn(), + bulkDocs: jest.fn(), + create: jest.fn(), + update: jest.fn(), + delete: jest.fn(), + initialize: jest.fn(), + isReady: jest.fn().mockReturnValue(true), + shutdown: jest.fn() +}; + +// Mock the service module +jest.mock('../../services/couchdbService', () => mockCouchdbService); + const Reward = require('../../models/Reward'); describe('Reward Model', () => { beforeEach(() => { jest.clearAllMocks(); // Reset all mocks to ensure clean state - global.mockCouchdbService.createDocument.mockReset(); - global.mockCouchdbService.updateDocument.mockReset(); - global.mockCouchdbService.deleteDocument.mockReset(); - global.mockCouchdbService.getById.mockReset(); - global.mockCouchdbService.find.mockReset(); - global.mockCouchdbService.findUserById.mockReset(); - global.mockCouchdbService.updateUserPoints.mockReset(); - global.mockCouchdbService.bulkDocs.mockReset(); - global.mockCouchdbService.create.mockReset(); - global.mockCouchdbService.update.mockReset(); - global.mockCouchdbService.delete.mockReset(); + mockCouchdbService.createDocument.mockReset(); + mockCouchdbService.updateDocument.mockReset(); + mockCouchdbService.deleteDocument.mockReset(); + mockCouchdbService.getById.mockReset(); + mockCouchdbService.find.mockReset(); + mockCouchdbService.findUserById.mockReset(); + mockCouchdbService.updateUserPoints.mockReset(); + mockCouchdbService.bulkDocs.mockReset(); + mockCouchdbService.create.mockReset(); + mockCouchdbService.update.mockReset(); + mockCouchdbService.delete.mockReset(); }); describe('Schema Validation', () => { @@ -37,7 +58,7 @@ describe('Reward Model', () => { updatedAt: '2023-01-01T00:00:00.000Z' }; - global.mockCouchdbService.createDocument.mockResolvedValue(mockCreated); + mockCouchdbService.createDocument.mockResolvedValue(mockCreated); const reward = await Reward.create(rewardData); @@ -55,67 +76,25 @@ describe('Reward Model', () => { cost: 50, }; - const mockCreated = { - _id: 'reward_123', - _rev: '1-abc', - type: 'reward', - ...rewardData, - isActive: true, - createdAt: '2023-01-01T00:00:00.000Z', - updatedAt: '2023-01-01T00:00:00.000Z' - }; - - global.mockCouchdbService.createDocument.mockResolvedValue(mockCreated); - - // The Reward model doesn't validate, so we test the behavior - const reward = await Reward.create(rewardData); - expect(reward.name).toBeUndefined(); + await expect(Reward.create(rewardData)).rejects.toThrow('Name is required'); }); - it('should require description field', async () => { +it('should require description field', async () => { const rewardData = { name: 'Reward without description', cost: 50, }; - const mockCreated = { - _id: 'reward_123', - _rev: '1-abc', - type: 'reward', - ...rewardData, - isActive: true, - createdAt: '2023-01-01T00:00:00.000Z', - updatedAt: '2023-01-01T00:00:00.000Z' - }; - - global.mockCouchdbService.createDocument.mockResolvedValue(mockCreated); - - // The Reward model doesn't validate, so we test the behavior - const reward = await Reward.create(rewardData); - expect(reward.description).toBeUndefined(); + await expect(Reward.create(rewardData)).rejects.toThrow('Description is required'); }); - it('should require cost field', async () => { +it('should require cost field', async () => { const rewardData = { name: 'Reward without cost', description: 'This reward has no cost', }; - const mockCreated = { - _id: 'reward_123', - _rev: '1-abc', - type: 'reward', - ...rewardData, - isActive: true, - createdAt: '2023-01-01T00:00:00.000Z', - updatedAt: '2023-01-01T00:00:00.000Z' - }; - - global.mockCouchdbService.createDocument.mockResolvedValue(mockCreated); - - // The Reward model doesn't validate, so we test the behavior - const reward = await Reward.create(rewardData); - expect(reward.cost).toBeUndefined(); + await expect(Reward.create(rewardData)).rejects.toThrow('Cost is required and must be a number'); }); it('should validate cost is a positive number', async () => { @@ -125,21 +104,7 @@ describe('Reward Model', () => { cost: -10, }; - const mockCreated = { - _id: 'reward_123', - _rev: '1-abc', - type: 'reward', - ...rewardData, - isActive: true, - createdAt: '2023-01-01T00:00:00.000Z', - updatedAt: '2023-01-01T00:00:00.000Z' - }; - - global.mockCouchdbService.createDocument.mockResolvedValue(mockCreated); - - // The Reward model doesn't validate, so we test the behavior - const reward = await Reward.create(rewardData); - expect(reward.cost).toBe(-10); + await expect(Reward.create(rewardData)).rejects.toThrow('Cost must be non-negative'); }); }); @@ -162,7 +127,7 @@ describe('Reward Model', () => { updatedAt: '2023-01-01T00:00:00.000Z' }; - global.mockCouchdbService.createDocument.mockResolvedValue(mockCreated); + mockCouchdbService.createDocument.mockResolvedValue(mockCreated); const reward = await Reward.create(rewardData); @@ -187,7 +152,7 @@ describe('Reward Model', () => { updatedAt: '2023-01-01T00:00:00.000Z' }; - global.mockCouchdbService.createDocument.mockResolvedValue(mockCreated); + mockCouchdbService.createDocument.mockResolvedValue(mockCreated); const reward = await Reward.create(rewardData); @@ -219,7 +184,7 @@ describe('Reward Model', () => { updatedAt: '2023-01-01T00:00:00.000Z' }; - global.mockCouchdbService.createDocument.mockResolvedValue(mockCreated); + mockCouchdbService.createDocument.mockResolvedValue(mockCreated); const reward = await Reward.create(rewardData); @@ -250,7 +215,7 @@ describe('Reward Model', () => { updatedAt: '2023-01-01T00:00:00.000Z' }; - global.mockCouchdbService.createDocument.mockResolvedValue(mockCreated); + mockCouchdbService.createDocument.mockResolvedValue(mockCreated); const reward = await Reward.create(rewardData); @@ -275,7 +240,7 @@ describe('Reward Model', () => { updatedAt: '2023-01-01T00:00:00.000Z' }; - global.mockCouchdbService.createDocument.mockResolvedValue(mockCreated); + mockCouchdbService.createDocument.mockResolvedValue(mockCreated); // The Reward model doesn't validate, so we test behavior const reward = await Reward.create(rewardData); @@ -302,7 +267,7 @@ describe('Reward Model', () => { updatedAt: '2023-01-01T00:00:00.000Z' }; - global.mockCouchdbService.createDocument.mockResolvedValue(mockCreated); + mockCouchdbService.createDocument.mockResolvedValue(mockCreated); const reward = await Reward.create(rewardData); @@ -327,8 +292,8 @@ describe('Reward Model', () => { updatedAt: '2023-01-01T00:00:00.000Z' }; - global.mockCouchdbService.getById.mockResolvedValue(mockReward); - global.mockCouchdbService.updateDocument.mockResolvedValue({ + mockCouchdbService.getById.mockResolvedValue(mockReward); + mockCouchdbService.updateDocument.mockResolvedValue({ ...mockReward, isActive: false, _rev: '2-def' @@ -370,7 +335,7 @@ describe('Reward Model', () => { updatedAt: '2023-01-01T00:00:00.000Z' }; - global.mockCouchdbService.createDocument.mockResolvedValue(mockCreated); + mockCouchdbService.createDocument.mockResolvedValue(mockCreated); const reward = await Reward.create(rewardData); @@ -397,8 +362,8 @@ describe('Reward Model', () => { updatedAt: '2023-01-01T00:00:00.000Z' }; - global.mockCouchdbService.getById.mockResolvedValue(mockReward); - global.mockCouchdbService.updateDocument.mockResolvedValue({ + mockCouchdbService.getById.mockResolvedValue(mockReward); + mockCouchdbService.updateDocument.mockResolvedValue({ ...mockReward, redeemedBy: [ { @@ -444,7 +409,7 @@ describe('Reward Model', () => { updatedAt: '2023-01-01T00:00:00.000Z' }; - global.mockCouchdbService.createDocument.mockResolvedValue(mockCreated); + mockCouchdbService.createDocument.mockResolvedValue(mockCreated); const reward = await Reward.create(rewardData); @@ -472,8 +437,8 @@ describe('Reward Model', () => { updatedAt: '2023-01-01T00:00:00.000Z' }; - global.mockCouchdbService.getById.mockResolvedValue(mockReward); - global.mockCouchdbService.updateDocument.mockResolvedValue({ + mockCouchdbService.getById.mockResolvedValue(mockReward); + mockCouchdbService.updateDocument.mockResolvedValue({ ...mockReward, isActive: false, _rev: '2-def', @@ -502,7 +467,7 @@ describe('Reward Model', () => { updatedAt: '2023-01-01T00:00:00.000Z' }; - global.mockCouchdbService.getById.mockResolvedValue(mockReward); + mockCouchdbService.getById.mockResolvedValue(mockReward); const reward = await Reward.findById('reward_123'); expect(reward).toBeDefined(); @@ -511,7 +476,7 @@ describe('Reward Model', () => { }); it('should return null when reward not found', async () => { - global.mockCouchdbService.getById.mockResolvedValue(null); + mockCouchdbService.getById.mockResolvedValue(null); const reward = await Reward.findById('nonexistent'); expect(reward).toBeNull();