brackt/app/routes/leagues/__tests__
Chris Parsons 1ba50828f7
Claude/redesign autodraft queue c4 kp r (#40)
* Redesign autodraft queue system with three-state control and queue-only constraint

Core Logic & Database:
- Add `queue_only` boolean column to `autodraft_settings` (migration 0031)
- Rename autodraft UI states: Off / Next Pick / All Picks (while_on mode maps to All Picks)
- `autoPickForTeam`: respects new `queueOnly` param — skips EV fallback when enabled
- `executeAutoPick`: auto-disables autodraft + emits socket event when queue empties with queueOnly ON (AC3)
- `autodraft-updated` socket event now includes `queueOnly` field

Mobile UI Overhaul:
- Rename "Lobby" tab → "Available" (AC6)
- Add new "Queue" tab to mobile bottom nav with drag-reorder, per-item Draft buttons, and autodraft controls (AC5)
- Controls tab retains commissioner tools, notifications, exit; queue controls moved to Queue tab
- Turn indicator appears on both Available and Queue tabs

Components:
- `AutodraftSettings`: replaces toggle+radio with three-state button group (Off | Next Pick | All Picks) + "Only autodraft from queue" switch (AC1, AC2)
- `QueueSection`: adds `canPick` prop + per-item Draft buttons for instant drafting when on the clock

Desktop (AC4):
- Sidebar QueueSection unchanged in position; gains same three-state controls and Draft buttons

Tests (AC7):
- `autodraft.test.ts`: updated for queueOnly field and socket event shape
- `timer-autodraft.test.ts`: new tests for queue-only constraint, auto-shutoff transitions, and all three autodraft states

https://claude.ai/code/session_01PYhJicAStoJ2u6q6dV1naB

* fix: remove erroneous ?? fallbacks in autodraft socket emissions and add autoPickForTeam tests

- Remove `?? true` default on queueOnly in queue-empty auto-disable socket emit
  (line 488) — was dead code since the column is NOT NULL, but semantically wrong
  and would have caused client-side UI desync if the type ever relaxed
- Remove `?? false` default on the next_pick auto-disable path for consistency
- Add app/models/__tests__/auto-pick.test.ts with 6 tests covering the queueOnly
  constraint: empty queue, all items drafted, partial queue skip, and EV fallback

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>

* fix: add missing queueOnly prop to AutodraftSettings test fixtures

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>

* test: rewrite AutodraftSettings tests for three-state button group UI

The component was redesigned from a switch + radio buttons to Off/Next Pick/All Picks
buttons with a separate queue-only Switch toggle. Updated 17 stale tests and added 5
new tests covering the queue-only toggle and the All Picks/Off button interactions.

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>

---------

Co-authored-by: Claude <noreply@anthropic.com>
2026-02-27 22:16:26 -08:00
..
autodraft.test.ts Claude/redesign autodraft queue c4 kp r (#40) 2026-02-27 22:16:26 -08:00
draft-board-access.test.ts Fix public draft board access not working when setting is enabled (#23) 2026-02-22 16:31:24 -08:00
draft-reset.test.ts feat: Add comprehensive scoring system database schema 2025-10-28 23:40:11 -07:00
README.md feat: add admin draft reset functionality with queue and pick deletion 2025-10-21 22:34:26 -07:00
settings-update.test.ts Fix public draft board access not working when setting is enabled (#23) 2026-02-22 16:31:24 -08:00
team-management.test.ts feat: prevent users from owning multiple teams in same league 2025-10-21 22:27:14 -07:00

League Settings Integration Tests

Overview

This directory contains integration tests for league settings features. The tests are written using Vitest and cover the following functionality:

Team Management

  • Remove Team Owner: Commissioners can remove owners from teams
  • Assign Team Owner: Admins can assign users to teams via dropdown
  • Authorization: Proper access control for commissioners vs admins
  • Edge Cases: Handling invalid inputs and error scenarios
  • Integration Scenarios: Complete workflows and multi-team operations

Draft Reset

  • Admin-Only Access: Only admins can reset drafts
  • Delete Operations: Clearing picks and queues
  • Status Updates: Resetting season to pre_draft
  • Draft Order Preservation: Ensuring draft slots remain intact
  • Data Integrity: Maintaining referential integrity during reset

Test Structure

Test Files

  • team-management.test.ts - Team management feature tests
  • draft-reset.test.ts - Draft reset feature tests

Fixtures

  • app/test/fixtures/user.ts - Mock user data including admin and regular users
  • app/test/fixtures/team.ts - Mock team data
  • app/test/fixtures/league.ts - Mock league data

Running Tests

Run all tests

npm test

Run tests in watch mode

npm run dev:test

Run tests with UI

npm run test:ui

Run tests with coverage

npm run test:coverage

Run only team management tests

npm test team-management

Test Coverage

The test suite covers:

1. Remove Owner Functionality (4 tests)

  • Successfully remove an owner from a team
  • Handle errors when removal fails
  • Allow commissioners to remove owners
  • Verify proper database calls

2. Assign Owner Functionality (5 tests)

  • Successfully assign an owner to a team
  • Handle errors when assignment fails
  • Only allow admins to assign owners
  • Prevent non-admins from assigning owners
  • Replace existing owner when assigning new owner

3. Admin User List (3 tests)

  • Fetch all users for admin dropdown
  • Only fetch users when user is admin
  • Return empty array for non-admin users

4. Authorization (3 tests)

  • Verify admin status before allowing assignment
  • Reject assignment for non-admin users
  • Allow commissioners to remove owners regardless of admin status

5. Edge Cases (6 tests)

  • Prevent assigning user who already owns a team in the league
  • Handle removing owner from team with no owner
  • Handle assigning owner to team that already has owner
  • Handle invalid team ID gracefully
  • Handle invalid user clerk ID gracefully
  • Proper error messages

6. Integration Scenarios (3 tests)

  • Complete workflow: assign then remove owner
  • Reassigning owner from one user to another
  • Handle multiple teams with different ownership states

Total: 24 tests

Draft Reset Test Coverage

1. Authorization (3 tests)

  • Only allow admins to reset draft
  • Reject non-admin users from resetting draft
  • Reject commissioners who are not admins

2. Delete Operations (4 tests)

  • Delete all draft picks for a season
  • Clear all draft queues for a season
  • Handle errors when deleting draft picks fails
  • Handle errors when clearing queues fails

3. Season Status Update (2 tests)

  • Update season status to pre_draft
  • Handle errors when updating season status fails

4. Status Validation (4 tests)

  • Allow reset when season status is draft
  • Allow reset when season status is completed
  • Allow reset when season status is active
  • Not allow reset when season status is pre_draft

5. Complete Workflow (3 tests)

  • Execute complete reset workflow in correct order
  • Not proceed with reset if admin check fails
  • Handle partial failure gracefully

6. Draft Order Preservation (2 tests)

  • Not delete or modify draft slots during reset
  • Preserve draft order after reset

7. Edge Cases (4 tests)

  • Handle reset when no draft picks exist
  • Handle reset when no draft queues exist
  • Handle invalid season ID gracefully
  • Handle concurrent reset attempts

8. Data Integrity (2 tests)

  • Ensure all picks are deleted before updating status
  • Maintain referential integrity after reset

Total: 24 tests

Key Features Tested

Commissioner Capabilities

  • Remove owners from any team in their league
  • No ability to assign owners (admin-only)

Admin Capabilities

  • Remove owners from any team
  • Assign any user to any team via dropdown
  • Access to full user list for assignment
  • Reset draft (admin-only, not commissioners)
    • Delete all draft picks
    • Clear all draft queues
    • Reset season to pre_draft status
    • Preserve draft order

Security

  • Admin-only check enforced for assignments and draft resets
  • Commissioner access verified for removals
  • Proper error handling for unauthorized access
  • Status validation for draft reset operations

Mocking Strategy

The tests use Vitest's mocking capabilities to:

  • Mock database operations (removeTeamOwner, assignTeamOwner)
  • Mock user authentication (isUserAdminByClerkId)
  • Mock user data fetching (findAllUsers)
  • Simulate various success and error scenarios

Future Enhancements

Potential additions to the test suite:

  • Test UI component rendering
  • Test form validation
  • Test loading states
  • Test success/error toast messages
  • Test concurrent operations
  • Test with real database (integration tests)
  • Performance tests for large user lists