Skip to content

Commit 10b162d

Browse files
committed
fix: Improves user list access safety
1 parent 80fa856 commit 10b162d

File tree

1 file changed

+6
-6
lines changed

1 file changed

+6
-6
lines changed

test/unit/useUsers.test.ts

Lines changed: 6 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -101,8 +101,8 @@ describe('useUsers Composable', () => {
101101

102102
updateUser({ id: 1, name: 'Updated User 1' } as User)
103103

104-
expect(users.value[0].name).toBe('Updated User 1')
105-
expect(users.value[1].name).toBe('User 2')
104+
expect(users.value?.[0]?.name).toBe('Updated User 1')
105+
expect(users.value?.[1]?.name).toBe('User 2')
106106
})
107107

108108
it('should ignore updates for non-existent user', () => {
@@ -113,7 +113,7 @@ describe('useUsers Composable', () => {
113113
updateUser({ id: 999, name: 'New User' } as User)
114114

115115
expect(users.value).toHaveLength(1)
116-
expect(users.value[0].name).toBe('User 1')
116+
expect(users.value?.[0]?.name).toBe('User 1')
117117
})
118118
})
119119

@@ -126,8 +126,8 @@ describe('useUsers Composable', () => {
126126
addUser({ id: 2, name: 'User 2' } as User)
127127

128128
expect(users.value).toHaveLength(2)
129-
expect(users.value[0].name).toBe('User 2')
130-
expect(users.value[1].name).toBe('User 1')
129+
expect(users.value?.[0]?.name).toBe('User 2')
130+
expect(users.value?.[1]?.name).toBe('User 1')
131131
})
132132
})
133133

@@ -149,7 +149,7 @@ describe('useUsers Composable', () => {
149149

150150
expect(mockFetch).toHaveBeenCalledWith('/api/nuxt-users/1', { method: 'DELETE' })
151151
expect(users.value).toHaveLength(1)
152-
expect(users.value[0].id).toBe(2)
152+
expect(users.value?.[0]?.id).toBe(2)
153153
expect(loading.value).toBe(false)
154154
expect(error.value).toBeNull()
155155
})

0 commit comments

Comments
 (0)