chrisp/brackt
1
1
Fork 0
Code Issues 42 Pull requests Projects Releases Packages Wiki Activity Actions

Fix lint: move shouldAppendDraftSlots to outer scope

Browse source 
oxlint (consistent-function-scoping) requires functions that don't close
over any variables to be defined at the outer scope rather than inside a
describe block.

https://claude.ai/code/session_01M3H55gnMxRztJK9KMXqqZo
This commit is contained in:
Claude 2026-05-20 01:11:28 +00:00
parent 4cb7424ae4
commit ff89343795
No known key found for this signature in database
1 changed files with 4 additions and 4 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

8
app/routes/leagues/__tests__/settings-update.test.ts
View file

@ -463,11 +463,11 @@ describe('Settings Update - Team Count Changes', () => {
// Mirrors the server action condition: existingSlots.length === currentTeamCount
// ---------------------------------------------------------------------------
describe('Settings Update - Draft Slot Creation Guard', () => {
function shouldAppendDraftSlots(existingSlotsCount: number, currentTeamCount: number): boolean {
return existingSlotsCount === currentTeamCount;
}
function shouldAppendDraftSlots(existingSlotsCount: number, currentTeamCount: number): boolean {
return existingSlotsCount === currentTeamCount;
}
describe('Settings Update - Draft Slot Creation Guard', () => {
it('does not create slots when no draft order was ever set', () => {
expect(shouldAppendDraftSlots(0, 12)).toBe(false);
});