Merge pull request #1427 from johndoe6345789/copilot/fix-typescript-errors

Fix TypeScript syntax errors in test files
This commit is contained in:
2026-01-08 16:01:08 +00:00
committed by GitHub
2 changed files with 9 additions and 4 deletions

View File

@@ -35,6 +35,7 @@ describe('ItemsPerPageSelector', () => {
const select = container.querySelector('select')
expect(select !== null).toBe(true)
if (select) {
// Create a proper change event with a select element
Object.defineProperty(select, 'value', {
writable: true,

View File

@@ -145,7 +145,8 @@ describe('retry utilities', () => {
describe('retry', () => {
it('should retry async function on failure', async () => {
let callCount = 0
const mockFn = vi.fn(async () => // eslint-disable-line @typescript-eslint/require-await { // eslint-disable-line @typescript-eslint/require-await
// eslint-disable-next-line @typescript-eslint/require-await
const mockFn = vi.fn(async () => {
callCount++
if (callCount < 2) {
throw new Error('Temporary error')
@@ -164,7 +165,8 @@ describe('retry utilities', () => {
})
it('should return result on first success', async () => {
const mockFn = vi.fn(async () => // eslint-disable-line @typescript-eslint/require-await 'success')
// eslint-disable-next-line @typescript-eslint/require-await
const mockFn = vi.fn(async () => 'success')
const result = await retry(mockFn, { maxRetries: 3, initialDelayMs: 10 })
@@ -173,7 +175,8 @@ describe('retry utilities', () => {
})
it('should throw after max retries', async () => {
const mockFn = vi.fn(async () => // eslint-disable-line @typescript-eslint/require-await { // eslint-disable-line @typescript-eslint/require-await
// eslint-disable-next-line @typescript-eslint/require-await
const mockFn = vi.fn(async () => {
throw new Error('Persistent error')
})
@@ -187,7 +190,8 @@ describe('retry utilities', () => {
it('should use exponential backoff', async () => {
let callCount = 0
const mockFn = vi.fn(async () => // eslint-disable-line @typescript-eslint/require-await { // eslint-disable-line @typescript-eslint/require-await
// eslint-disable-next-line @typescript-eslint/require-await
const mockFn = vi.fn(async () => {
callCount++
if (callCount < 4) {
throw new Error('Temporary error')