Skip to content

Commit d82040b

Browse files
sebastianwesselsebastianwessel
authored
Merge pull request #78 from sebastianwessel/fix-lint
chore: fix lint
2 parents f5d79c4 + 2979b37 commit d82040b

5 files changed

Lines changed: 123 additions & 123 deletions

File tree

‎src/sandbox/provide/provideTimingFunctions.ts‎

Lines changed: 65 additions & 65 deletions
Original file line numberDiff line numberDiff line change
@@ -7,13 +7,13 @@ export const provideTimingFunctions = (
77
maxIntervalCount: number
88
},
99
) => {
10-
const scope = new Scope()
10+
const scope = new Scope()
1111

12-
const timeouts = new Map<number, ReturnType<typeof setTimeout>>()
13-
let timeoutCounter = 0
12+
const timeouts = new Map<number, ReturnType<typeof setTimeout>>()
13+
let timeoutCounter = 0
1414

15-
const immediates = new Map<number, ReturnType<typeof setTimeout>>()
16-
let immediateCounter = 0
15+
const immediates = new Map<number, ReturnType<typeof setTimeout>>()
16+
let immediateCounter = 0
1717

1818
const intervals = new Map<number, ReturnType<typeof setTimeout>>()
1919
let intervalCounter = 0
@@ -59,49 +59,49 @@ export const provideTimingFunctions = (
5959
})
6060

6161
scope.manage(_clearTimeout)
62-
ctx.setProp(ctx.global, 'clearTimeout', _clearTimeout)
63-
64-
const _setImmediate = ctx.newFunction('setImmediate', vmFnHandle => {
65-
const currentCounter = immediateCounter++
66-
if (timeouts.size + 1 > max.maxTimeoutCount) {
67-
throw new Error(
68-
`Client tries to use setImmediate, which exceeds the limit of max ${max.maxTimeoutCount} concurrent running timeout functions`,
69-
)
70-
}
71-
72-
const vmFnHandleCopy = vmFnHandle.dup()
73-
scope.manage(vmFnHandleCopy)
74-
75-
const timeoutID = setTimeout(() => {
76-
const t = immediates.get(currentCounter)
77-
if (t) {
78-
clearTimeout(t)
79-
immediates.delete(currentCounter)
80-
}
81-
ctx.callFunction(vmFnHandleCopy, ctx.undefined)
82-
}, 0)
83-
84-
immediates.set(currentCounter, timeoutID)
85-
86-
return ctx.newNumber(currentCounter)
87-
})
88-
89-
scope.manage(_setImmediate)
90-
ctx.setProp(ctx.global, 'setImmediate', _setImmediate)
91-
92-
const _clearImmediate = ctx.newFunction('clearImmediate', idHandle => {
93-
const id: number = ctx.dump(idHandle)
94-
idHandle.dispose()
95-
96-
const t = immediates.get(id)
97-
if (t) {
98-
clearTimeout(t)
99-
immediates.delete(id)
100-
}
101-
})
102-
103-
scope.manage(_clearImmediate)
104-
ctx.setProp(ctx.global, 'clearImmediate', _clearImmediate)
62+
ctx.setProp(ctx.global, 'clearTimeout', _clearTimeout)
63+
64+
const _setImmediate = ctx.newFunction('setImmediate', vmFnHandle => {
65+
const currentCounter = immediateCounter++
66+
if (timeouts.size + 1 > max.maxTimeoutCount) {
67+
throw new Error(
68+
`Client tries to use setImmediate, which exceeds the limit of max ${max.maxTimeoutCount} concurrent running timeout functions`,
69+
)
70+
}
71+
72+
const vmFnHandleCopy = vmFnHandle.dup()
73+
scope.manage(vmFnHandleCopy)
74+
75+
const timeoutID = setTimeout(() => {
76+
const t = immediates.get(currentCounter)
77+
if (t) {
78+
clearTimeout(t)
79+
immediates.delete(currentCounter)
80+
}
81+
ctx.callFunction(vmFnHandleCopy, ctx.undefined)
82+
}, 0)
83+
84+
immediates.set(currentCounter, timeoutID)
85+
86+
return ctx.newNumber(currentCounter)
87+
})
88+
89+
scope.manage(_setImmediate)
90+
ctx.setProp(ctx.global, 'setImmediate', _setImmediate)
91+
92+
const _clearImmediate = ctx.newFunction('clearImmediate', idHandle => {
93+
const id: number = ctx.dump(idHandle)
94+
idHandle.dispose()
95+
96+
const t = immediates.get(id)
97+
if (t) {
98+
clearTimeout(t)
99+
immediates.delete(id)
100+
}
101+
})
102+
103+
scope.manage(_clearImmediate)
104+
ctx.setProp(ctx.global, 'clearImmediate', _clearImmediate)
105105

106106
const _setInterval = ctx.newFunction('setInterval', (vmFnHandle, intervalHandle) => {
107107
const currentCounter = intervalCounter++
@@ -140,23 +140,23 @@ export const provideTimingFunctions = (
140140
scope.manage(_clearInterval)
141141
ctx.setProp(ctx.global, 'clearInterval', _clearInterval)
142142

143-
const dispose = () => {
144-
for (const [_key, value] of timeouts) {
145-
clearTimeout(value)
146-
}
147-
timeouts.clear()
148-
timeoutCounter = 0
149-
150-
for (const [_key, value] of immediates) {
151-
clearTimeout(value)
152-
}
153-
immediates.clear()
154-
immediateCounter = 0
155-
156-
for (const [_key, value] of intervals) {
157-
clearInterval(value)
158-
}
159-
intervals.clear()
143+
const dispose = () => {
144+
for (const [_key, value] of timeouts) {
145+
clearTimeout(value)
146+
}
147+
timeouts.clear()
148+
timeoutCounter = 0
149+
150+
for (const [_key, value] of immediates) {
151+
clearTimeout(value)
152+
}
153+
immediates.clear()
154+
immediateCounter = 0
155+
156+
for (const [_key, value] of intervals) {
157+
clearInterval(value)
158+
}
159+
intervals.clear()
160160
intervalCounter = 0
161161

162162
scope.dispose()

‎src/test/async/core-timers.test.ts‎

Lines changed: 8 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -67,7 +67,7 @@ describe('async - core - timers', () => {
6767
expect((result as OkResponse).data).toBe('interval reached')
6868
})
6969

70-
it('clearInterval works correctly', async () => {
70+
it('clearInterval works correctly', async () => {
7171
const code = `
7272
export default await new Promise((resolve) => {
7373
let count = 0
@@ -87,19 +87,19 @@ describe('async - core - timers', () => {
8787
// but it should be around 3 if intervals are 100ms and we clear after 500ms.
8888
expect((result as OkResponse).data).toBeGreaterThanOrEqual(3)
8989
expect((result as OkResponse).data).toBeLessThanOrEqual(5)
90-
})
90+
})
9191

92-
it('setImmediate works correctly', async () => {
93-
const code = `
92+
it('setImmediate works correctly', async () => {
93+
const code = `
9494
export default await new Promise((resolve) => {
9595
setImmediate(() => {
9696
resolve('immediate reached')
9797
})
9898
})
9999
`
100100

101-
const result = await runCode(code)
102-
expect(result.ok).toBeTrue()
103-
expect((result as OkResponse).data).toBe('immediate reached')
104-
})
101+
const result = await runCode(code)
102+
expect(result.ok).toBeTrue()
103+
expect((result as OkResponse).data).toBe('immediate reached')
104+
})
105105
})

‎src/test/async/timers-promises.test.ts‎

Lines changed: 21 additions & 21 deletions
Original file line numberDiff line numberDiff line change
@@ -3,37 +3,37 @@ import { loadAsyncQuickJs } from '../../loadAsyncQuickJs.js'
33
import type { OkResponse } from '../../types/OkResponse.js'
44

55
describe('async - node:timers/promises', () => {
6-
let runtime: Awaited<ReturnType<typeof loadAsyncQuickJs>>
6+
let runtime: Awaited<ReturnType<typeof loadAsyncQuickJs>>
77

8-
beforeAll(async () => {
9-
runtime = await loadAsyncQuickJs()
10-
})
8+
beforeAll(async () => {
9+
runtime = await loadAsyncQuickJs()
10+
})
1111

12-
const runCode = async (code: string) => {
13-
return await runtime.runSandboxed(async ({ evalCode }) => {
14-
return await evalCode(code)
15-
})
16-
}
12+
const runCode = async (code: string) => {
13+
return await runtime.runSandboxed(async ({ evalCode }) => {
14+
return await evalCode(code)
15+
})
16+
}
1717

18-
it('setTimeout resolves', async () => {
19-
const code = `
18+
it('setTimeout resolves', async () => {
19+
const code = `
2020
import { setTimeout } from 'node:timers/promises'
2121
export default await setTimeout(100, 'done')
2222
`
2323

24-
const result = await runCode(code)
25-
expect(result.ok).toBeTrue()
26-
expect((result as OkResponse).data).toBe('done')
27-
})
24+
const result = await runCode(code)
25+
expect(result.ok).toBeTrue()
26+
expect((result as OkResponse).data).toBe('done')
27+
})
2828

29-
it('setImmediate resolves', async () => {
30-
const code = `
29+
it('setImmediate resolves', async () => {
30+
const code = `
3131
import { setImmediate } from 'node:timers/promises'
3232
export default await setImmediate('immediate')
3333
`
3434

35-
const result = await runCode(code)
36-
expect(result.ok).toBeTrue()
37-
expect((result as OkResponse).data).toBe('immediate')
38-
})
35+
const result = await runCode(code)
36+
expect(result.ok).toBeTrue()
37+
expect((result as OkResponse).data).toBe('immediate')
38+
})
3939
})

‎src/test/sync/core-timers.test.ts‎

Lines changed: 8 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -67,7 +67,7 @@ describe('sync - core - timers', () => {
6767
expect((result as OkResponse).data).toBe('interval reached')
6868
})
6969

70-
it('clearInterval works correctly', async () => {
70+
it('clearInterval works correctly', async () => {
7171
const code = `
7272
export default await new Promise((resolve) => {
7373
let count = 0
@@ -87,19 +87,19 @@ describe('sync - core - timers', () => {
8787
// but it should be around 3 if intervals are 100ms and we clear after 500ms.
8888
expect((result as OkResponse).data).toBeGreaterThanOrEqual(3)
8989
expect((result as OkResponse).data).toBeLessThanOrEqual(5)
90-
})
90+
})
9191

92-
it('setImmediate works correctly', async () => {
93-
const code = `
92+
it('setImmediate works correctly', async () => {
93+
const code = `
9494
export default await new Promise((resolve) => {
9595
setImmediate(() => {
9696
resolve('immediate reached')
9797
})
9898
})
9999
`
100100

101-
const result = await runCode(code)
102-
expect(result.ok).toBeTrue()
103-
expect((result as OkResponse).data).toBe('immediate reached')
104-
})
101+
const result = await runCode(code)
102+
expect(result.ok).toBeTrue()
103+
expect((result as OkResponse).data).toBe('immediate reached')
104+
})
105105
})

‎src/test/sync/timers-promises.test.ts‎

Lines changed: 21 additions & 21 deletions
Original file line numberDiff line numberDiff line change
@@ -3,37 +3,37 @@ import { loadQuickJs } from '../../loadQuickJs.js'
33
import type { OkResponse } from '../../types/OkResponse.js'
44

55
describe('sync - node:timers/promises', () => {
6-
let runtime: Awaited<ReturnType<typeof loadQuickJs>>
6+
let runtime: Awaited<ReturnType<typeof loadQuickJs>>
77

8-
beforeAll(async () => {
9-
runtime = await loadQuickJs()
10-
})
8+
beforeAll(async () => {
9+
runtime = await loadQuickJs()
10+
})
1111

12-
const runCode = async (code: string) => {
13-
return await runtime.runSandboxed(async ({ evalCode }) => {
14-
return await evalCode(code)
15-
})
16-
}
12+
const runCode = async (code: string) => {
13+
return await runtime.runSandboxed(async ({ evalCode }) => {
14+
return await evalCode(code)
15+
})
16+
}
1717

18-
it('setTimeout resolves', async () => {
19-
const code = `
18+
it('setTimeout resolves', async () => {
19+
const code = `
2020
import { setTimeout } from 'node:timers/promises'
2121
export default await setTimeout(100, 'done')
2222
`
2323

24-
const result = await runCode(code)
25-
expect(result.ok).toBeTrue()
26-
expect((result as OkResponse).data).toBe('done')
27-
})
24+
const result = await runCode(code)
25+
expect(result.ok).toBeTrue()
26+
expect((result as OkResponse).data).toBe('done')
27+
})
2828

29-
it('setImmediate resolves', async () => {
30-
const code = `
29+
it('setImmediate resolves', async () => {
30+
const code = `
3131
import { setImmediate } from 'node:timers/promises'
3232
export default await setImmediate('immediate')
3333
`
3434

35-
const result = await runCode(code)
36-
expect(result.ok).toBeTrue()
37-
expect((result as OkResponse).data).toBe('immediate')
38-
})
35+
const result = await runCode(code)
36+
expect(result.ok).toBeTrue()
37+
expect((result as OkResponse).data).toBe('immediate')
38+
})
3939
})

0 commit comments

Comments
 (0)