Skip to content

Commit c523f8c

Browse files
fix(executor): 🚨 fix biome formatting in loader test files (#221)
## Summary Fix 3 biome formatting violations in executor test files introduced by #219 that weren't caught before merge, causing CI lint failure on main. ## Highlights - `loader-json-hint.integration.test.ts`: array join wrapping reformatted - `loader.test.ts`: expect call wrapping collapsed to single line ## Test plan - [x] `task lint` passes locally - [ ] CI green 🤖 Generated with [Claude Code](https://claude.com/claude-code) Co-authored-by: Claude Opus 4.6 <noreply@anthropic.com>
1 parent 2c91e18 commit c523f8c

2 files changed

Lines changed: 5 additions & 9 deletions

File tree

executor-node/test/loader-json-hint.integration.test.ts

Lines changed: 4 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -9,7 +9,7 @@
99
import { type ChildProcess, spawn } from 'node:child_process'
1010
import { mkdtempSync, rmSync, writeFileSync } from 'node:fs'
1111
import { tmpdir } from 'node:os'
12-
import { join, resolve, dirname } from 'node:path'
12+
import { dirname, join, resolve } from 'node:path'
1313
import { fileURLToPath } from 'node:url'
1414
import { afterAll, beforeAll, describe, expect, it } from 'vitest'
1515

@@ -27,11 +27,9 @@ beforeAll(() => {
2727
// A script with a bare JSON import (missing `with { type: 'json' }`)
2828
writeFileSync(
2929
join(tmpDir, 'bare-import.mjs'),
30-
[
31-
"import data from './data.json'",
32-
'export default async function() { return data }',
33-
''
34-
].join('\n')
30+
["import data from './data.json'", 'export default async function() { return data }', ''].join(
31+
'\n'
32+
)
3533
)
3634
})
3735

executor-node/test/loader.test.ts

Lines changed: 1 addition & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -122,9 +122,7 @@ describe('loadScript hook validation', () => {
122122
const result = annotateImportError(nodeError)
123123

124124
expect(result).toContain(nodeError)
125-
expect(result).toContain(
126-
'Hint: Node ESM requires an import attribute for JSON modules.'
127-
)
125+
expect(result).toContain('Hint: Node ESM requires an import attribute for JSON modules.')
128126
expect(result).toContain("with { type: 'json' }")
129127
})
130128

0 commit comments

Comments
 (0)