Skip to content

Commit 6ceb756

Browse files
frostebiteclaude
andcommitted
fix: break long test line to meet formatting requirements
Splits the long it.skipIf() line in the integration test to meet oxfmt line length requirements (currently ~100 chars). Co-Authored-By: Claude Haiku 4.5 <noreply@anthropic.com>
1 parent 27ee333 commit 6ceb756

1 file changed

Lines changed: 28 additions & 25 deletions

File tree

functions/test/scrapeVersions.test.ts

Lines changed: 28 additions & 25 deletions
Original file line numberDiff line numberDiff line change
@@ -507,30 +507,33 @@ describe('scrapeRecentOfficialUnityVersions', () => {
507507
});
508508

509509
describe('integration test (live)', () => {
510-
it.skipIf(process.env.CI === 'false')('should successfully scrape real Unity releases page', async () => {
511-
// This test runs in CI only and validates the scraping logic against the real page
512-
const realFetch = (await import('node-fetch')).default;
513-
vi.mocked(fetch).mockImplementation(realFetch as any);
514-
515-
const result = await scrapeRecentOfficialUnityVersions();
516-
517-
// Should find at least some recent versions
518-
expect(result.length).toBeGreaterThan(0);
519-
520-
// All versions should be valid
521-
result.forEach((version) => {
522-
expect(version.version).toMatch(/^\d+\.\d+\.\d+f\d+$/);
523-
expect(version.changeSet).toMatch(/^[a-f0-9]{12}$/);
524-
expect(version.major).toBeGreaterThanOrEqual(2017);
525-
});
526-
527-
console.log(`Found ${result.length} recent versions on Unity releases page`);
528-
console.log(
529-
`Latest versions: ${result
530-
.slice(0, 5)
531-
.map((v) => v.version)
532-
.join(', ')}`,
533-
);
534-
});
510+
it.skipIf(process.env.CI === 'false')(
511+
'should successfully scrape real Unity releases page',
512+
async () => {
513+
// This test runs in CI only and validates the scraping logic against the real page
514+
const realFetch = (await import('node-fetch')).default;
515+
vi.mocked(fetch).mockImplementation(realFetch as any);
516+
517+
const result = await scrapeRecentOfficialUnityVersions();
518+
519+
// Should find at least some recent versions
520+
expect(result.length).toBeGreaterThan(0);
521+
522+
// All versions should be valid
523+
result.forEach((version) => {
524+
expect(version.version).toMatch(/^\d+\.\d+\.\d+f\d+$/);
525+
expect(version.changeSet).toMatch(/^[a-f0-9]{12}$/);
526+
expect(version.major).toBeGreaterThanOrEqual(2017);
527+
});
528+
529+
console.log(`Found ${result.length} recent versions on Unity releases page`);
530+
console.log(
531+
`Latest versions: ${result
532+
.slice(0, 5)
533+
.map((v) => v.version)
534+
.join(', ')}`,
535+
);
536+
},
537+
);
535538
});
536539
});

0 commit comments

Comments
 (0)