Compare commits
35 Commits
v0.1.3
..
1ec51f9c6a
| Author | SHA1 | Date | |
|---|---|---|---|
| 1ec51f9c6a | |||
| b2876d1e49 | |||
| a24782f4bb | |||
| 1bece10df2 | |||
| b96932270b | |||
| 58857af5ca | |||
| 39d3250114 | |||
| 18418adf9e | |||
| d2824695c0 | |||
| 3d87f3e1dc | |||
| 04dfdfa378 | |||
| 2cf832eeac | |||
| 967bdc1b89 | |||
| a3cd46e39a | |||
| 42902106e6 | |||
| 1503420519 | |||
| f8f9c205be | |||
| d8a8931bc3 | |||
| e921f17d2c | |||
| c993852147 | |||
| 96a7454d2f | |||
| ed43399b24 | |||
| e8f0fd2538 | |||
| 2e98e9eb51 | |||
| 3a7a9b4b8e | |||
| 66ee38f456 | |||
| 4e3b25e2fc | |||
| e4c4333b40 | |||
| 09302fd911 | |||
| 0d9707ea27 | |||
| b90ec08966 | |||
| 1cd2cbd0a6 | |||
| 24db43eb7f | |||
| 1c9f70c3ed | |||
| 5caa284b3c |
@@ -0,0 +1,15 @@
|
||||
# Development environment configuration for EuchreCamp
|
||||
# Copy this file to .env.development and fill in your values
|
||||
|
||||
# Database Configuration
|
||||
DATABASE_PROVIDER=postgresql
|
||||
# Development database URL - must contain "_dev" to pass safety checks
|
||||
DATABASE_URL="postgresql://euchre_camp:password@localhost:5432/euchre_camp_dev"
|
||||
|
||||
# Authentication
|
||||
BETTER_AUTH_SECRET="your-secret-key-change-this"
|
||||
BETTER_AUTH_URL="http://localhost:3000"
|
||||
|
||||
# Application Configuration
|
||||
NODE_ENV=development
|
||||
TRUSTED_ORIGINS="http://localhost:3000,http://127.0.0.1:3000"
|
||||
@@ -0,0 +1,54 @@
|
||||
# EuchreCamp Environment Configuration
|
||||
# Copy this file to .env and fill in your values
|
||||
|
||||
# ============================================
|
||||
# Database Configuration
|
||||
# ============================================
|
||||
# PostgreSQL connection string
|
||||
# Format: postgresql://username:password@host:port/database
|
||||
DATABASE_URL=postgresql://euchre:euchrepassword@localhost:5432/euchre_camp
|
||||
|
||||
# Shadow database for Prisma migrations (optional for PostgreSQL)
|
||||
DATABASE_SHADOW_URL=postgresql://euchre:euchrepassword@localhost:5432/euchre_camp_shadow
|
||||
|
||||
# Database provider (postgresql, mysql, sqlite, etc.)
|
||||
DATABASE_PROVIDER=postgresql
|
||||
|
||||
# ============================================
|
||||
# Better Auth Configuration
|
||||
# ============================================
|
||||
# Secret key for session encryption (generate a strong random string)
|
||||
# Run: openssl rand -base64 32
|
||||
BETTER_AUTH_SECRET=your-secret-key-change-in-production
|
||||
|
||||
# Base URL for authentication callbacks
|
||||
# For production: https://your-domain.com
|
||||
BETTER_AUTH_URL=http://localhost:3000
|
||||
|
||||
# ============================================
|
||||
# Application Configuration
|
||||
# ============================================
|
||||
# Environment: development, production, test
|
||||
NODE_ENV=production
|
||||
|
||||
# Trusted origins for CORS and authentication
|
||||
# Add your domain(s) for production
|
||||
TRUSTED_ORIGINS=http://localhost:3000,http://127.0.0.1:3000
|
||||
|
||||
# ============================================
|
||||
# Optional: External Services
|
||||
# ============================================
|
||||
# If using external database (e.g., Supabase, Railway)
|
||||
# DATABASE_URL=postgresql://user:pass@host:port/db
|
||||
|
||||
# If using external auth provider
|
||||
# BETTER_AUTH_URL=https://your-app.com
|
||||
|
||||
# ============================================
|
||||
# CasaOS Deployment Notes
|
||||
# ============================================
|
||||
# When deploying to CasaOS, set these via the UI:
|
||||
# 1. DATABASE_URL: Your PostgreSQL connection string
|
||||
# 2. BETTER_AUTH_SECRET: Generate with: openssl rand -base64 32
|
||||
# 3. BETTER_AUTH_URL: Your app's public URL
|
||||
# 4. TRUSTED_ORIGINS: Your app's public URL(s)
|
||||
+11
-1
@@ -1,5 +1,15 @@
|
||||
{
|
||||
"extends": ["next/core-web-vitals", "next/typescript"],
|
||||
"env": {
|
||||
"browser": true,
|
||||
"es2021": true,
|
||||
"node": true
|
||||
},
|
||||
"parser": "@typescript-eslint/parser",
|
||||
"parserOptions": {
|
||||
"ecmaVersion": "latest",
|
||||
"sourceType": "module"
|
||||
},
|
||||
"plugins": ["@typescript-eslint"],
|
||||
"ignorePatterns": [
|
||||
".next",
|
||||
"out",
|
||||
|
||||
@@ -0,0 +1,71 @@
|
||||
name: Build CI Images
|
||||
|
||||
on:
|
||||
push:
|
||||
branches:
|
||||
- main
|
||||
paths:
|
||||
- 'Dockerfile.ci-base'
|
||||
- 'package.json'
|
||||
- 'bun.lockb'
|
||||
- '.gitea/workflows/build-ci-images.yml'
|
||||
schedule:
|
||||
# Weekly rebuild to get latest Playwright/Bun versions
|
||||
- cron: '0 2 * * 0' # Every Sunday at 2 AM
|
||||
workflow_dispatch: # Manual trigger
|
||||
|
||||
env:
|
||||
REGISTRY: docker.notsosm.art
|
||||
IMAGE_NAME: euchre-camp
|
||||
|
||||
jobs:
|
||||
build-ci-base:
|
||||
runs-on: ubuntu-latest
|
||||
permissions:
|
||||
contents: read
|
||||
packages: write
|
||||
|
||||
steps:
|
||||
- name: Checkout code
|
||||
uses: actions/checkout@v4
|
||||
|
||||
- name: Set up Docker Buildx
|
||||
uses: docker/setup-buildx-action@v3
|
||||
|
||||
- name: Login to Registry
|
||||
run: |
|
||||
echo "${{ secrets.DOCKER_PASSWORD }}" | docker login ${{ env.REGISTRY }} -u ${{ secrets.DOCKER_LOGIN }} --password-stdin
|
||||
|
||||
- name: Extract metadata for CI base image
|
||||
id: meta
|
||||
run: |
|
||||
# Get Playwright version from package.json
|
||||
PLAYWRIGHT_VERSION=$(grep -o '"@playwright/test": "[^"]*"' package.json | cut -d'"' -f4)
|
||||
echo "playwright_version=${PLAYWRIGHT_VERSION}" >> $GITHUB_OUTPUT
|
||||
|
||||
# Get Bun version (latest)
|
||||
BUN_VERSION=$(bun --version 2>/dev/null || echo "latest")
|
||||
echo "bun_version=${BUN_VERSION}" >> $GITHUB_OUTPUT
|
||||
|
||||
# Set tags
|
||||
echo "tags=${{ env.REGISTRY }}/${{ env.IMAGE_NAME }}/ci-base:latest,${{ env.REGISTRY }}/${{ env.IMAGE_NAME }}/ci-base:playwright-${PLAYWRIGHT_VERSION},${{ env.REGISTRY }}/${{ env.IMAGE_NAME }}/ci-base:${{ github.sha }}" >> $GITHUB_OUTPUT
|
||||
|
||||
- name: Build and push CI base image
|
||||
run: |
|
||||
# Build with multiple tags
|
||||
docker build \
|
||||
--file Dockerfile.ci-base \
|
||||
--tag ${{ env.REGISTRY }}/${{ env.IMAGE_NAME }}/ci-base:latest \
|
||||
--tag ${{ env.REGISTRY }}/${{ env.IMAGE_NAME }}/ci-base:playwright-${{ steps.meta.outputs.playwright_version }} \
|
||||
--tag ${{ env.REGISTRY }}/${{ env.IMAGE_NAME }}/ci-base:${{ github.sha }} \
|
||||
.
|
||||
|
||||
# Push all tags
|
||||
docker push ${{ env.REGISTRY }}/${{ env.IMAGE_NAME }}/ci-base:latest
|
||||
docker push ${{ env.REGISTRY }}/${{ env.IMAGE_NAME }}/ci-base:playwright-${{ steps.meta.outputs.playwright_version }}
|
||||
docker push ${{ env.REGISTRY }}/${{ env.IMAGE_NAME }}/ci-base:${{ github.sha }}
|
||||
|
||||
- name: Clean up
|
||||
if: always()
|
||||
run: |
|
||||
docker logout ${{ env.REGISTRY }}
|
||||
+6
-42
@@ -9,7 +9,7 @@ jobs:
|
||||
unit-tests:
|
||||
runs-on: ubuntu-latest
|
||||
container:
|
||||
image: node:20-alpine
|
||||
image: docker.notsosm.art/euchre-camp/ci-base:latest
|
||||
options: --user root
|
||||
|
||||
steps:
|
||||
@@ -17,55 +17,19 @@ jobs:
|
||||
uses: actions/checkout@v4
|
||||
|
||||
- name: Install dependencies
|
||||
run: npm ci
|
||||
|
||||
- name: Run unit tests
|
||||
run: npm run test:run
|
||||
|
||||
acceptance-tests:
|
||||
runs-on: ubuntu-latest
|
||||
needs: unit-tests
|
||||
container:
|
||||
image: mcr.microsoft.com/playwright:v1.58.0-jammy
|
||||
options: --user root
|
||||
env:
|
||||
DATABASE_PROVIDER: sqlite
|
||||
DATABASE_URL: file:./prisma/ci.db
|
||||
BETTER_AUTH_SECRET: test-secret-key-for-ci-only
|
||||
|
||||
steps:
|
||||
- name: Checkout code
|
||||
uses: actions/checkout@v4
|
||||
|
||||
- name: Setup Node.js
|
||||
uses: actions/setup-node@v4
|
||||
with:
|
||||
node-version: '20'
|
||||
|
||||
- name: Install dependencies
|
||||
run: npm ci
|
||||
run: bun install
|
||||
|
||||
- name: Generate Prisma client
|
||||
run: npx prisma generate
|
||||
run: bun x prisma generate
|
||||
env:
|
||||
DATABASE_URL: postgresql://user:pass@localhost:5432/dummy
|
||||
|
||||
- name: Setup SQLite database
|
||||
run: |
|
||||
# Create SQLite database file
|
||||
mkdir -p prisma
|
||||
npx prisma migrate deploy
|
||||
|
||||
- name: Run acceptance tests
|
||||
run: npm run test:acceptance
|
||||
env:
|
||||
DATABASE_PROVIDER: sqlite
|
||||
DATABASE_URL: file:./prisma/ci.db
|
||||
BETTER_AUTH_SECRET: test-secret-key-for-ci-only
|
||||
- name: Run unit tests
|
||||
run: bun test src/__tests__/unit/ src/__tests__/*.test.tsx src/__tests__/auth-simple.test.ts
|
||||
|
||||
analyze-bump-type:
|
||||
runs-on: ubuntu-latest
|
||||
needs: acceptance-tests
|
||||
needs: unit-tests
|
||||
|
||||
steps:
|
||||
- name: Checkout code
|
||||
|
||||
@@ -14,6 +14,9 @@ jobs:
|
||||
runs-on: ubuntu-latest
|
||||
# Skip if this is an auto-generated version bump commit
|
||||
if: "!contains(github.event.head_commit.message, 'chore: bump version')"
|
||||
container:
|
||||
image: docker.notsosm.art/euchre-camp/ci-base:latest
|
||||
options: --user root
|
||||
|
||||
steps:
|
||||
- name: Checkout code
|
||||
@@ -70,10 +73,10 @@ jobs:
|
||||
echo "Bumping version: $BUMP"
|
||||
|
||||
# Run the bump script
|
||||
node scripts/bump-version.js "$BUMP" --yes
|
||||
bun run scripts/bump-version.js "$BUMP" --yes
|
||||
|
||||
# Get new version
|
||||
NEW_VERSION=$(node -p "require('./package.json').version")
|
||||
NEW_VERSION=$(bun -e "console.log(require('./package.json').version)")
|
||||
echo "new_version=$NEW_VERSION" >> $GITHUB_OUTPUT
|
||||
echo "New version: $NEW_VERSION"
|
||||
|
||||
@@ -104,10 +107,6 @@ jobs:
|
||||
git push origin "$TAG_NAME"
|
||||
fi
|
||||
|
||||
- name: Set up Docker Buildx
|
||||
if: steps.commit.outputs.committed == 'true'
|
||||
uses: docker/setup-buildx-action@v3
|
||||
|
||||
- name: Build test-capable image
|
||||
if: steps.commit.outputs.committed == 'true'
|
||||
run: |
|
||||
@@ -123,7 +122,7 @@ jobs:
|
||||
docker run --rm \
|
||||
-e DATABASE_URL="postgresql://user:pass@localhost:5432/dummy" \
|
||||
${{ env.IMAGE_NAME }}-test:${{ steps.version.outputs.new_version }} \
|
||||
npm run test:run
|
||||
bun test 'src/__tests__/unit/**' 'src/__tests__/*.test.tsx' 'src/__tests__/auth-simple.test.ts'
|
||||
|
||||
- name: Build production image
|
||||
if: steps.commit.outputs.committed == 'true'
|
||||
@@ -160,8 +159,36 @@ jobs:
|
||||
echo " docker push ${{ env.REGISTRY }}/${{ env.IMAGE_NAME }}:latest"
|
||||
fi
|
||||
|
||||
- name: Deploy to dev (placeholder)
|
||||
- name: Deploy to dev environment
|
||||
if: steps.commit.outputs.committed == 'true'
|
||||
run: |
|
||||
echo "Deploying version ${{ steps.version.outputs.new_version }} to dev environment..."
|
||||
# TODO: Add actual deployment steps
|
||||
|
||||
# Update docker-compose.yml with new image tag using full registry path
|
||||
# The registry is docker.notsosm.art and image is euchre-camp
|
||||
IMAGE_TAG="${{ steps.version.outputs.new_version }}"
|
||||
sed -i "s|image: docker.notsosm.art/euchre-camp:[0-9.]*|image: docker.notsosm.art/euchre-camp:${IMAGE_TAG}|" docker-compose.yml
|
||||
|
||||
# Copy the updated docker-compose.yml to the deployment location
|
||||
# The runners are on the same Docker server where the container is running
|
||||
sudo mkdir -p /home/euchre_camp
|
||||
sudo cp docker-compose.yml /home/euchre_camp/
|
||||
sudo chown -R euchre:euchre /home/euchre_camp
|
||||
|
||||
# Pull and restart the dev container
|
||||
cd /home/euchre_camp
|
||||
docker-compose pull app
|
||||
docker-compose up -d app
|
||||
|
||||
# Wait for container to be healthy
|
||||
echo "Waiting for container to start..."
|
||||
sleep 10
|
||||
|
||||
# Check if container is running
|
||||
if docker ps --filter "name=euchre-camp-app" --format "{{.Status}}" | grep -q "Up"; then
|
||||
echo "✅ Dev environment successfully deployed with version ${{ steps.version.outputs.new_version }}"
|
||||
else
|
||||
echo "❌ Dev environment deployment failed"
|
||||
docker-compose logs app
|
||||
exit 1
|
||||
fi
|
||||
|
||||
@@ -1,25 +0,0 @@
|
||||
name: Test
|
||||
|
||||
on:
|
||||
push:
|
||||
branches:
|
||||
- '**'
|
||||
|
||||
jobs:
|
||||
unit-tests:
|
||||
runs-on: ubuntu-latest
|
||||
container:
|
||||
image: node:20-alpine
|
||||
options: --user root
|
||||
# Skip if this is an auto-generated version bump commit (handled by release workflow)
|
||||
if: "!contains(github.event.head_commit.message, 'chore: bump version')"
|
||||
|
||||
steps:
|
||||
- name: Checkout code
|
||||
uses: actions/checkout@v4
|
||||
|
||||
- name: Install dependencies
|
||||
run: npm ci
|
||||
|
||||
- name: Run unit tests
|
||||
run: npm run test:run
|
||||
+8
-1
@@ -34,7 +34,14 @@ yarn-error.log*
|
||||
.pnpm-debug.log*
|
||||
|
||||
# env files (can opt-in for committing if needed)
|
||||
.env*
|
||||
.env
|
||||
.env.local
|
||||
.env.development.local
|
||||
.env.test.local
|
||||
.env.production.local
|
||||
# Allow example env files to be tracked
|
||||
!.env.example
|
||||
!.env.development.example
|
||||
|
||||
# vercel
|
||||
.vercel
|
||||
|
||||
@@ -9,11 +9,12 @@ EuchreCamp is a Next.js 14+ application for managing Euchre tournaments and trac
|
||||
### Key Technologies
|
||||
- **Next.js 14+** (App Router)
|
||||
- **TypeScript**
|
||||
- **Prisma ORM** (SQLite)
|
||||
- **Prisma ORM** (SQLite/PostgreSQL)
|
||||
- **Tailwind CSS**
|
||||
- **Better Auth** (Authentication)
|
||||
- **Vitest** (Unit Testing)
|
||||
- **Bun** (Package Manager & Test Runner)
|
||||
- **Playwright** (Acceptance Testing)
|
||||
- **Vitest** (Legacy - migrated to Bun test runner)
|
||||
|
||||
## Architecture Patterns
|
||||
|
||||
@@ -37,13 +38,55 @@ EuchreCamp is a Next.js 14+ application for managing Euchre tournaments and trac
|
||||
|
||||
## Common Tasks
|
||||
|
||||
### Package Manager: Bun
|
||||
|
||||
This project uses **Bun** as the package manager and test runner:
|
||||
|
||||
```bash
|
||||
# Install dependencies
|
||||
bun install
|
||||
|
||||
# Run development server
|
||||
bun run dev
|
||||
|
||||
# Build the application
|
||||
bun run build
|
||||
|
||||
# Run unit/component tests
|
||||
bun test
|
||||
|
||||
# Run unit tests only
|
||||
bun run test:unit
|
||||
|
||||
# Run component tests only
|
||||
bun run test:component
|
||||
|
||||
# Run acceptance tests (Playwright)
|
||||
bun run test:acceptance
|
||||
|
||||
# Run linting
|
||||
bun run lint
|
||||
```
|
||||
|
||||
**Note**: E2E tests still use Playwright, as Bun's test runner doesn't support browser automation. Unit and component tests have been migrated to Bun's native test runner for faster execution. E2E tests are located in the `e2e/` directory (not `src/__tests__/e2e/`).
|
||||
|
||||
### CI/CD with Bun
|
||||
|
||||
All CI/CD workflows have been updated to use Bun:
|
||||
|
||||
- **Dockerfile**: Uses `oven/bun:alpine` base image for all stages
|
||||
- **PR Workflow**: Uses `oven/setup-bun` action and `bun install`, `bun test`
|
||||
- **Release Workflow**: Uses Bun for version bumping and Docker builds
|
||||
|
||||
**Note**: The `test.yml` workflow has been removed as it's redundant with the PR workflow.
|
||||
|
||||
### Admin User Creation
|
||||
|
||||
To create an admin user, use the provided scripts:
|
||||
|
||||
**Option 1: Using Better Auth API (Recommended)**
|
||||
```bash
|
||||
node scripts/create-admin-via-api.js
|
||||
bun run scripts/create-admin-via-api.js
|
||||
```
|
||||
This creates the admin user `david@dhg.lol` with password `adminadmin` using Better Auth's internal API.
|
||||
|
||||
|
||||
@@ -1,3 +1,31 @@
|
||||
## [0.1.4] - 2026-04-02
|
||||
|
||||
### Patch Changes
|
||||
|
||||
- ci: remove acceptance tests and add dev deployment
|
||||
- fix(tests): resolve password validation, csv upload, and admin auth issues
|
||||
- fix(ci): correct playwright config paths and add env example files
|
||||
- fix(ci): use existing PostgreSQL server at dhg.lol
|
||||
- fix(ci): use PostgreSQL in acceptance-tests with dev credentials
|
||||
- feat(ci): build custom Docker images for Gitea Actions compatibility
|
||||
- fix(tests): add @prisma/client mock for test isolation
|
||||
- fix(ci): add prisma generate step to unit-tests job
|
||||
- fix: downgrade ESLint to v8.57.1 for LSP compatibility
|
||||
- fix: improve TypeScript types for better IDE code hinting
|
||||
- feat: migrate to ESLint flat config (eslint.config.js)
|
||||
- Revert "fix: downgrade ESLint to v8.x for .eslintrc.json compatibility"
|
||||
- fix: downgrade ESLint to v8.x for .eslintrc.json compatibility
|
||||
- fix: update PR workflow to exclude e2e tests from unit test phase
|
||||
- fix: avoid global mock clearing in EditTournamentForm tests
|
||||
- fix: avoid global mock clearing in Navigation tests
|
||||
- fix: add DOM cleanup to bun-setup.ts
|
||||
- fix: disable test isolation in bunfig.toml
|
||||
- refactor: improve test structure for Bun compatibility
|
||||
- feat: update CI/CD to use Bun
|
||||
- feat: migrate from npm to Bun
|
||||
- docs: update TODO list with recent fix
|
||||
- fix: add defensive checks to prisma.ts to prevent build failures
|
||||
|
||||
## [0.1.3] - 2026-04-01
|
||||
|
||||
### Patch Changes
|
||||
|
||||
+13
-13
@@ -1,9 +1,9 @@
|
||||
# Multi-stage build for EuchreCamp Next.js application
|
||||
|
||||
# Stage 1: Builder
|
||||
FROM node:20-alpine AS builder
|
||||
FROM oven/bun:alpine AS builder
|
||||
|
||||
# Install dependencies
|
||||
# Install dependencies (needed for native modules)
|
||||
RUN apk add --no-cache python3 make g++
|
||||
|
||||
# Set working directory
|
||||
@@ -13,21 +13,21 @@ WORKDIR /app
|
||||
COPY package*.json ./
|
||||
|
||||
# Install dependencies (including dev dependencies for building)
|
||||
RUN npm ci
|
||||
RUN bun install
|
||||
|
||||
# Copy source code
|
||||
COPY . .
|
||||
|
||||
# Generate Prisma client (with dummy PostgreSQL DATABASE_URL for build-time generation)
|
||||
# Note: A dummy URL is used since the real database is not available during build
|
||||
RUN DATABASE_URL="postgresql://user:pass@localhost:5432/dummy" npx prisma generate
|
||||
RUN DATABASE_PROVIDER=postgresql DATABASE_URL="postgresql://user:pass@localhost:5432/dummy" bun x prisma generate
|
||||
|
||||
# Build the application (with dummy DATABASE_URL for static page generation and git commit)
|
||||
ARG GIT_COMMIT=unknown
|
||||
RUN DATABASE_URL="postgresql://user:pass@localhost:5432/dummy" NEXT_PUBLIC_GIT_COMMIT=$GIT_COMMIT npm run build
|
||||
RUN DATABASE_PROVIDER=postgresql DATABASE_URL="postgresql://user:pass@localhost:5432/dummy" NEXT_PUBLIC_GIT_COMMIT=$GIT_COMMIT bun run build
|
||||
|
||||
# Stage 2: Test runner (includes dev dependencies for testing)
|
||||
FROM node:20-alpine AS test-runner
|
||||
FROM oven/bun:alpine AS test-runner
|
||||
|
||||
# Install dependencies
|
||||
RUN apk add --no-cache python3 make g++ git
|
||||
@@ -39,16 +39,16 @@ WORKDIR /app
|
||||
COPY package*.json ./
|
||||
|
||||
# Install ALL dependencies (including dev dependencies for testing)
|
||||
RUN npm ci
|
||||
RUN bun install
|
||||
|
||||
# Copy source code
|
||||
COPY . .
|
||||
|
||||
# Generate Prisma client
|
||||
RUN DATABASE_URL="postgresql://user:pass@localhost:5432/dummy" npx prisma generate
|
||||
RUN DATABASE_PROVIDER=postgresql DATABASE_URL="postgresql://user:pass@localhost:5432/dummy" bun x prisma generate
|
||||
|
||||
# Stage 3: Production runner
|
||||
FROM node:20-alpine AS runner
|
||||
FROM oven/bun:alpine AS runner
|
||||
|
||||
# Install dumb-init for proper signal handling
|
||||
RUN apk add --no-cache dumb-init
|
||||
@@ -64,15 +64,15 @@ WORKDIR /app
|
||||
COPY --from=builder --chown=euchre:euchre /app/.next ./.next
|
||||
COPY --from=builder --chown=euchre:euchre /app/public ./public
|
||||
COPY --from=builder --chown=euchre:euchre /app/package.json ./package.json
|
||||
COPY --from=builder --chown=euchre:euchre /app/package-lock.json ./package-lock.json
|
||||
COPY --from=builder --chown=euchre:euchre /app/bun.lockb ./bun.lockb
|
||||
COPY --from=builder --chown=euchre:euchre /app/prisma ./prisma
|
||||
|
||||
# Install only production dependencies
|
||||
RUN npm ci --omit=dev
|
||||
RUN bun install --production
|
||||
|
||||
# Generate Prisma client
|
||||
# Note: We need to set DATABASE_URL even for generation because prisma.config.ts requires it
|
||||
RUN DATABASE_URL="postgresql://user:pass@localhost:5432/dummy" npx prisma generate
|
||||
RUN DATABASE_PROVIDER=postgresql DATABASE_URL="postgresql://user:pass@localhost:5432/dummy" bun x prisma generate
|
||||
|
||||
# Switch to non-root user
|
||||
USER euchre
|
||||
@@ -86,4 +86,4 @@ HEALTHCHECK --interval=30s --timeout=3s --start-period=5s --retries=3 \
|
||||
|
||||
# Start command
|
||||
ENTRYPOINT ["dumb-init", "--"]
|
||||
CMD ["npm", "start"]
|
||||
CMD ["bun", "run", "start"]
|
||||
|
||||
@@ -0,0 +1,32 @@
|
||||
# Base CI image with Bun, Node.js, Playwright, and build tools
|
||||
# Used for Gitea Actions CI workflows
|
||||
# Uses Microsoft Playwright image as base (Ubuntu-based) with Bun added
|
||||
|
||||
FROM mcr.microsoft.com/playwright:v1.58.0-jammy AS base
|
||||
|
||||
# Install unzip (required for Bun installation) and other tools
|
||||
RUN apt-get update && apt-get install -y unzip && rm -rf /var/lib/apt/lists/*
|
||||
|
||||
# Install Bun (latest version)
|
||||
# Note: The playwright image already has Node.js pre-installed
|
||||
RUN curl -fsSL https://bun.sh/install | bash
|
||||
|
||||
# Add Bun to PATH for subsequent commands
|
||||
ENV PATH="/root/.bun/bin:$PATH"
|
||||
|
||||
# Verify installations
|
||||
RUN echo "=== Bun Version ===" && bun --version && \
|
||||
echo "=== Node.js Version ===" && node --version && \
|
||||
echo "=== Playwright Version ===" && bun x playwright --version
|
||||
|
||||
WORKDIR /app
|
||||
|
||||
# Set default environment variables
|
||||
ENV DATABASE_PROVIDER=sqlite
|
||||
ENV DATABASE_URL=file:./prisma/ci.db
|
||||
ENV BETTER_AUTH_SECRET=test-secret-key-for-ci-only
|
||||
ENV NODE_ENV=test
|
||||
|
||||
# Health check command (can be overridden)
|
||||
HEALTHCHECK --interval=30s --timeout=3s --start-period=5s --retries=3 \
|
||||
CMD bun --version
|
||||
@@ -0,0 +1,4 @@
|
||||
[test]
|
||||
preload = ["./src/__tests__/bun-setup.ts"]
|
||||
exclude = ["e2e/**", "**/e2e/**"]
|
||||
# isolation = true
|
||||
+1
-1
@@ -3,7 +3,7 @@
|
||||
|
||||
services:
|
||||
app:
|
||||
image: euchre-camp/euchre-camp:0.1.0.dev
|
||||
image: docker.notsosm.art/euchre-camp:0.1.0.dev
|
||||
container_name: euchre-camp-app
|
||||
ports:
|
||||
- "3000:3000"
|
||||
|
||||
@@ -30,6 +30,23 @@
|
||||
- [ ] Update MatchEditor to use tournament-specific target score
|
||||
- [ ] Run tests and verify variant scoring implementation
|
||||
|
||||
### Recently Completed ✅
|
||||
- [x] Update CI/CD workflows to use Bun (PR, release)
|
||||
- [x] Update Dockerfile to use Bun Alpine image
|
||||
- [x] Update PR workflow to use Bun
|
||||
- [x] Update Release workflow to use Bun
|
||||
- [x] Remove test.yml workflow (redundant)
|
||||
- [x] Verify Docker build with Bun
|
||||
|
||||
### Recently Completed ✅
|
||||
- [x] Fix Prisma build error in CI pipeline (Docker build failure due to missing DATABASE_URL validation)
|
||||
- [x] Add defensive checks to src/lib/prisma.ts to prevent build failures
|
||||
- [x] Migrate from npm to Bun package manager
|
||||
- [x] Migrate unit tests (Vitest → Bun test runner)
|
||||
- [x] Migrate component tests (Vitest → Bun test runner)
|
||||
- [x] Configure Bun with DOM environment for React Testing Library
|
||||
- [x] Keep Playwright for E2E tests (hybrid approach)
|
||||
|
||||
### Recently Completed ✅
|
||||
- [x] Add OpenSkill rating system support (src/lib/openskill-utils.ts)
|
||||
- [x] Add Glicko2 rating system support (src/lib/glicko2-utils.ts)
|
||||
|
||||
+3
-3
@@ -79,7 +79,7 @@ test.describe('CSV Upload Player Deduplication', () => {
|
||||
formData.append('eventId', testTournamentId.toString());
|
||||
|
||||
const response = await request.post('http://localhost:3000/api/matches/upload', {
|
||||
data: formData,
|
||||
multipart: formData,
|
||||
});
|
||||
|
||||
expect(response.ok()).toBeTruthy();
|
||||
@@ -133,7 +133,7 @@ test.describe('CSV Upload Player Deduplication', () => {
|
||||
formData.append('eventId', testTournamentId.toString());
|
||||
|
||||
const response = await request.post('http://localhost:3000/api/matches/upload', {
|
||||
data: formData,
|
||||
multipart: formData,
|
||||
});
|
||||
|
||||
expect(response.ok()).toBeTruthy();
|
||||
@@ -190,7 +190,7 @@ test.describe('CSV Upload Player Deduplication', () => {
|
||||
formData.append('eventId', testTournamentId.toString());
|
||||
|
||||
const response = await request.post('http://localhost:3000/api/matches/upload', {
|
||||
data: formData,
|
||||
multipart: formData,
|
||||
});
|
||||
|
||||
expect(response.ok()).toBeTruthy();
|
||||
@@ -0,0 +1,45 @@
|
||||
/**
|
||||
* Bridge file to run Cucumber tests through Playwright's test runner
|
||||
*
|
||||
* This allows Cucumber tests to benefit from Playwright's:
|
||||
* - Dev server management
|
||||
* - Browser lifecycle management
|
||||
* - Test reporting
|
||||
* - CI/CD integration
|
||||
*/
|
||||
|
||||
import { test } from '@playwright/test';
|
||||
import { execSync } from 'child_process';
|
||||
|
||||
// This test file doesn't contain actual tests
|
||||
// It just runs Cucumber CLI which executes the feature files
|
||||
test.describe('Cucumber E2E Tests', () => {
|
||||
test('Run all Cucumber feature files', async () => {
|
||||
// This test is a placeholder that triggers Cucumber execution
|
||||
// In practice, Cucumber should be run directly via CLI
|
||||
console.log('Cucumber tests should be run via: bun cucumber-js');
|
||||
});
|
||||
});
|
||||
|
||||
/**
|
||||
* Alternative approach: Programmatic execution
|
||||
*
|
||||
* If you want to run Cucumber programmatically from within Playwright:
|
||||
*/
|
||||
/*
|
||||
import { execSync } from 'child_process';
|
||||
|
||||
export default async function runCucumberTests() {
|
||||
try {
|
||||
const output = execSync(
|
||||
'bun cucumber-js --config e2e/cucumber/cucumber.config.ts',
|
||||
{ encoding: 'utf-8', stdio: 'inherit' }
|
||||
);
|
||||
console.log(output);
|
||||
return true;
|
||||
} catch (error) {
|
||||
console.error('Cucumber tests failed:', error);
|
||||
return false;
|
||||
}
|
||||
}
|
||||
*/
|
||||
@@ -0,0 +1,182 @@
|
||||
# Gherkin-Style E2E Tests - Feature Summary
|
||||
|
||||
## Overview
|
||||
|
||||
This directory contains Gherkin-style acceptance tests for the EuchreCamp application, written in Cucumber syntax and executed via Playwright.
|
||||
|
||||
## Coverage Summary
|
||||
|
||||
### Feature Files (6 files, 16 scenarios)
|
||||
|
||||
| Feature File | Issue | Scenarios | Status |
|
||||
|--------------|-------|-----------|--------|
|
||||
| authentication.feature | - | 5 | ✅ Complete |
|
||||
| password-reset.feature | #10 | 4 | ⚠️ @wip (needs UI) |
|
||||
| player-schedule.feature | #9 | 3 | ⚠️ @wip (needs data) |
|
||||
| tournament-schedule.feature | #7 | 4 | ⚠️ @wip (needs data) |
|
||||
| user-registration.feature | - | 3 | ✅ Complete |
|
||||
| wordmark-navigation.feature | #24 | 3 | ✅ Complete |
|
||||
|
||||
### Step Definitions
|
||||
|
||||
- **Total steps defined:** 75
|
||||
- **Common steps:** Navigation, forms, assertions
|
||||
- **Auth steps:** Registration, login, logout, schedule navigation
|
||||
- **All steps verified:** ✅ Dry-run passes with no undefined steps
|
||||
|
||||
## Issues Covered
|
||||
|
||||
### Issue #24: Wordmark Navigation ✅
|
||||
**Scenario:** Clicking EuchreCamp wordmark navigates to appropriate page
|
||||
- Unauthenticated users → Public homepage
|
||||
- Players → Their profile page
|
||||
- Admins → Admin dashboard
|
||||
|
||||
### Issue #10: Password Reset ⚠️
|
||||
**Scenarios:**
|
||||
- Access password reset page
|
||||
- Request reset with valid email
|
||||
- Request reset with invalid email
|
||||
- Submit empty email field
|
||||
|
||||
**Status:** Marked @wip because password reset UI is not yet implemented
|
||||
|
||||
### Issue #9: Player Schedule ⚠️
|
||||
**Scenarios:**
|
||||
- View empty schedule (no matches)
|
||||
- View schedule with upcoming matches
|
||||
- Navigate to match details
|
||||
|
||||
**Status:** Marked @wip because data setup requires tournament creation
|
||||
|
||||
### Issue #7: Tournament Schedule Tab ⚠️
|
||||
**Scenarios:**
|
||||
- View schedule page for tournament
|
||||
- Generate round-robin schedule
|
||||
- View schedule with bye rounds
|
||||
- Click matchup to enter results
|
||||
|
||||
**Status:** Marked @wip because data setup requires tournament creation
|
||||
|
||||
## Running Tests
|
||||
|
||||
### Command Line
|
||||
|
||||
```bash
|
||||
# Run all Cucumber tests
|
||||
bun run test:acceptance:cucumber
|
||||
|
||||
# Run with pretty output
|
||||
bun run test:acceptance:cucumber:pretty
|
||||
|
||||
# Run specific feature
|
||||
bun cucumber-js e2e/cucumber/features/user-registration.feature
|
||||
|
||||
# Dry run (verify all steps defined)
|
||||
bun cucumber-js --config e2e/cucumber/cucumber.config.ts --dry-run
|
||||
```
|
||||
|
||||
### Tags
|
||||
|
||||
Filter tests by tag:
|
||||
```bash
|
||||
# Run only @happy-path tests
|
||||
bun cucumber-js --tags "@happy-path"
|
||||
|
||||
# Skip @wip tests
|
||||
bun cucumber-js --tags "not @wip"
|
||||
|
||||
# Run specific issue tests
|
||||
bun cucumber-js --tags "@issue-24"
|
||||
```
|
||||
|
||||
## Feature Examples
|
||||
|
||||
### Wordmark Navigation (Issue #24)
|
||||
```gherkin
|
||||
Scenario: Authenticated player clicks wordmark goes to their profile
|
||||
Given I am logged in as a player
|
||||
When I click the "EuchreCamp" wordmark
|
||||
Then I should be redirected to my profile page
|
||||
And I should see "Welcome"
|
||||
```
|
||||
|
||||
### User Registration
|
||||
```gherkin
|
||||
Scenario: Successful registration with valid data
|
||||
Given I am on the registration page
|
||||
When I fill in "name" with "Test User"
|
||||
And I fill in "email" with "test@example.com"
|
||||
And I fill in "password" with "TestPassword123!"
|
||||
And I click the "Create Account" button
|
||||
Then I should be redirected to my profile page
|
||||
And my user account should exist
|
||||
```
|
||||
|
||||
### Tournament Schedule (Issue #7)
|
||||
```gherkin
|
||||
Scenario: Tournament admin generates round-robin schedule
|
||||
Given I am logged in as a tournament admin
|
||||
And a tournament exists with 4 teams
|
||||
When I go to the tournament schedule page
|
||||
And I click the "Generate Schedule" button
|
||||
Then I should see "Schedule generated successfully"
|
||||
And I should see round 1 matchups
|
||||
```
|
||||
|
||||
## Key Principles
|
||||
|
||||
1. **Browser-only interactions** - Tests interact with UI only, no database access
|
||||
2. **Gherkin syntax** - Plain English Given-When-Then format
|
||||
3. **Happy path focus** - Tests common user workflows
|
||||
4. **Issue tracking** - Each feature file linked to specific GitHub issues
|
||||
5. **@wip markers** - New/incomplete features marked for development
|
||||
|
||||
## Next Steps
|
||||
|
||||
### Immediate
|
||||
1. Run tests against dev server: `bun run test:acceptance:cucumber`
|
||||
2. Review @wip scenarios and implement missing UI
|
||||
3. Set up test data creation via API or fixtures
|
||||
|
||||
### Short-term
|
||||
1. Add more feature files for other issues:
|
||||
- Issue #8: Tournament bracket visualization
|
||||
- Issue #11: Club admin dashboard
|
||||
- Issue #15: Admin view-as-user functionality
|
||||
- Issue #22: Team configuration options
|
||||
|
||||
2. Add more step definitions for:
|
||||
- Tournament creation
|
||||
- Team management
|
||||
- Match result entry
|
||||
- Calendar view
|
||||
|
||||
### Long-term
|
||||
1. Integrate with CI/CD pipeline
|
||||
2. Generate HTML reports
|
||||
3. Add visual regression testing
|
||||
4. Create test data factory for complex scenarios
|
||||
|
||||
## Useful Commands
|
||||
|
||||
```bash
|
||||
# List all feature files
|
||||
ls e2e/cucumber/features/
|
||||
|
||||
# Count scenarios
|
||||
grep -c "Scenario:" e2e/cucumber/features/*.feature
|
||||
|
||||
# List all step definitions
|
||||
grep -r "Given\|When\|Then" e2e/cucumber/step-definitions/ | wc -l
|
||||
|
||||
# Check for @wip scenarios
|
||||
grep -n "@wip" e2e/cucumber/features/*.feature
|
||||
```
|
||||
|
||||
## Documentation
|
||||
|
||||
- [Cucumber.js Documentation](https://github.com/cucumber/cucumber-js)
|
||||
- [Gherkin Reference](https://cucumber.io/docs/gherkin/)
|
||||
- [Playwright Documentation](https://playwright.dev)
|
||||
- [Feature README](./README.md)
|
||||
@@ -0,0 +1,209 @@
|
||||
# Cucumber Gherkin-Style E2E Tests
|
||||
|
||||
This directory contains Gherkin-style acceptance tests using Cucumber and Playwright for testing the EuchreCamp application.
|
||||
|
||||
## Overview
|
||||
|
||||
These tests follow the **Given-When-Then** syntax for behavior-driven development (BDD) and test the application from a user's perspective by interacting with the browser UI only.
|
||||
|
||||
## Key Principles
|
||||
|
||||
1. **Browser-only interactions**: Tests interact with the application via the browser (click, type, navigate)
|
||||
2. **No direct database access**: All data is created/modified through the UI
|
||||
3. **Dev site testing**: Tests run against a running development server
|
||||
4. **Happy path focus**: Tests verify common user workflows
|
||||
5. **Gherkin syntax**: Tests are written in plain English using Given-When-Then
|
||||
|
||||
## Running Tests
|
||||
|
||||
### Run all Cucumber tests
|
||||
```bash
|
||||
bun run test:acceptance:cucumber
|
||||
```
|
||||
|
||||
### Run with pretty formatter (visible output)
|
||||
```bash
|
||||
bun run test:acceptance:cucumber:pretty
|
||||
```
|
||||
|
||||
### Run specific feature file
|
||||
```bash
|
||||
bun cucumber-js e2e/cucumber/features/user-registration.feature
|
||||
```
|
||||
|
||||
### Run specific scenario
|
||||
```bash
|
||||
bun cucumber-js --name "Successful registration with valid data"
|
||||
```
|
||||
|
||||
### Dry run (check for undefined steps)
|
||||
```bash
|
||||
bun cucumber-js --config e2e/cucumber/cucumber.config.ts --dry-run
|
||||
```
|
||||
|
||||
## Test Structure
|
||||
|
||||
```
|
||||
e2e/cucumber/
|
||||
├── features/ # Gherkin feature files (6 files, 16 scenarios)
|
||||
│ ├── authentication.feature # Login/logout flows
|
||||
│ ├── password-reset.feature # Password reset (issue #10)
|
||||
│ ├── player-schedule.feature # Player schedule view (issue #9)
|
||||
│ ├── tournament-schedule.feature # Tournament schedule tab (issue #7)
|
||||
│ ├── user-registration.feature # User registration flows
|
||||
│ └── wordmark-navigation.feature # Wordmark navigation (issue #24)
|
||||
├── step-definitions/ # Step implementations (75 steps defined)
|
||||
│ ├── common-steps.ts # Navigation, forms, assertions
|
||||
│ └── auth-steps.ts # Login, logout, registration, schedule
|
||||
├── support/ # Test infrastructure
|
||||
│ ├── world.ts # Shared test context
|
||||
│ └── hooks.ts # Before/After hooks
|
||||
├── cucumber.config.ts # Cucumber configuration
|
||||
└── README.md # This file
|
||||
```
|
||||
|
||||
## Feature Files
|
||||
|
||||
### 1. user-registration.feature
|
||||
User registration scenarios with validation
|
||||
|
||||
### 2. authentication.feature
|
||||
Login/logout flows for authenticated users
|
||||
|
||||
### 3. wordmark-navigation.feature
|
||||
Tests for Issue #24: Wordmark navigation based on user role
|
||||
|
||||
### 4. password-reset.feature
|
||||
Tests for Issue #10: Password reset flow (marked @wip - needs implementation)
|
||||
|
||||
### 5. player-schedule.feature
|
||||
Tests for Issue #9: Player schedule view with upcoming matches
|
||||
|
||||
### 6. tournament-schedule.feature
|
||||
Tests for Issue #7: Tournament admin schedule tab and round-robin generation
|
||||
|
||||
## Example Test
|
||||
|
||||
### Feature File (user-registration.feature)
|
||||
```gherkin
|
||||
Feature: User Registration
|
||||
As a new user
|
||||
I want to register for an account
|
||||
So that I can participate in Euchre tournaments
|
||||
|
||||
Background:
|
||||
Given I am on the registration page
|
||||
|
||||
Scenario: Successful registration with valid data
|
||||
When I fill in "name" with "Test User"
|
||||
And I fill in "email" with "test@example.com"
|
||||
And I fill in "password" with "TestPassword123!"
|
||||
And I click the "Create Account" button
|
||||
Then I should be redirected to my profile page
|
||||
And my user account should exist
|
||||
```
|
||||
|
||||
### Step Definition (auth-steps.ts)
|
||||
```typescript
|
||||
Given('I am logged in as a player', async function () {
|
||||
const credentials = generateTestCredentials();
|
||||
world.user = credentials;
|
||||
|
||||
await world.page.goto(`${world.baseURL}/auth/register`);
|
||||
await world.page.fill('input[name="name"]', credentials.name);
|
||||
await world.page.fill('input[name="email"]', credentials.email);
|
||||
await world.page.fill('input[name="password"]', credentials.password);
|
||||
await world.page.click('button[type="submit"]');
|
||||
await world.page.waitForURL(/\/players\/\d+\/profile/);
|
||||
});
|
||||
```
|
||||
|
||||
## Tags
|
||||
|
||||
Use tags to organize and filter tests:
|
||||
|
||||
- `@happy-path` - Tests successful user workflows
|
||||
- `@negative-test` - Tests error cases and validation
|
||||
- `@authentication` - Tests related to login/registration
|
||||
- `@wip` - Work in progress (skipped by default)
|
||||
- `@skip` - Temporarily skipped tests
|
||||
|
||||
## Configuration
|
||||
|
||||
The Cucumber configuration is in `cucumber.config.ts`:
|
||||
|
||||
- Feature files: `e2e/cucumber/features/**/*.feature`
|
||||
- Step definitions: `e2e/cucumber/step-definitions/**/*.ts`
|
||||
- TypeScript loader: tsx (for path alias support)
|
||||
- Tags: Skips `@wip` and `@skip` by default
|
||||
- Retries: 2 retries in CI environment
|
||||
|
||||
## World Context
|
||||
|
||||
The `world.ts` file provides a shared context for tests:
|
||||
|
||||
```typescript
|
||||
interface WorldState {
|
||||
page: Page; // Playwright page
|
||||
context: BrowserContext;
|
||||
baseURL: string; // Dev server URL
|
||||
user?: { // Current test user
|
||||
email: string;
|
||||
name: string;
|
||||
password: string;
|
||||
};
|
||||
tournament?: any; // Current test tournament
|
||||
player?: any; // Current test player
|
||||
}
|
||||
```
|
||||
|
||||
## Hooks
|
||||
|
||||
Hooks run before/after tests:
|
||||
|
||||
- `BeforeAll` - Launch browser once before all tests
|
||||
- `AfterAll` - Close browser and cleanup after all tests
|
||||
- `Before` - Create new page context before each scenario
|
||||
- `After` - Close page and cleanup test data after each scenario
|
||||
|
||||
## Best Practices
|
||||
|
||||
1. **Unique test data**: Use timestamps to ensure unique test users
|
||||
2. **Wait for navigation**: Always wait for page loads and redirects
|
||||
3. **Clear assertions**: Verify expected state after user actions
|
||||
4. **No database access**: Use UI/API for all data creation
|
||||
5. **Tag organization**: Use tags to filter test suites
|
||||
6. **One assertion per step**: Keep steps focused and readable
|
||||
|
||||
## Troubleshooting
|
||||
|
||||
### Steps showing as undefined
|
||||
Run with `--dry-run` to check step matching:
|
||||
```bash
|
||||
bun cucumber-js --dry-run
|
||||
```
|
||||
|
||||
### Database errors
|
||||
These tests don't interact with the database directly. If you see Prisma errors, check that:
|
||||
- The dev server is running
|
||||
- The database is accessible
|
||||
- Environment variables are set correctly
|
||||
|
||||
### Slow tests
|
||||
Use the `@wip` tag for tests in development and skip them during full runs.
|
||||
|
||||
## CI/CD Integration
|
||||
|
||||
In CI, the tests should:
|
||||
1. Start the dev server (if not already running)
|
||||
2. Run Cucumber tests against `http://localhost:3000`
|
||||
3. Use JUnit format for reporting:
|
||||
```bash
|
||||
bun cucumber-js --format junit --out reports/cucumber.xml
|
||||
```
|
||||
|
||||
## Related
|
||||
|
||||
- [Cucumber.js Documentation](https://github.com/cucumber/cucumber-js)
|
||||
- [Gherkin Syntax Reference](https://cucumber.io/docs/gherkin/)
|
||||
- [Playwright Documentation](https://playwright.dev)
|
||||
@@ -0,0 +1,43 @@
|
||||
/**
|
||||
* Cucumber configuration for EuchreCamp E2E tests
|
||||
*/
|
||||
|
||||
module.exports = {
|
||||
// Paths to feature files
|
||||
paths: ['e2e/cucumber/features/**/*.feature'],
|
||||
|
||||
// Paths to step definitions
|
||||
import: ['e2e/cucumber/step-definitions/**/*.ts'],
|
||||
|
||||
// Paths to support files (hooks, world)
|
||||
require: ['e2e/cucumber/support/**/*.ts'],
|
||||
|
||||
// Use tsx loader for TypeScript with path aliases
|
||||
requireModule: ['tsx'],
|
||||
|
||||
// Format options
|
||||
format: [
|
||||
'progress-bar',
|
||||
'pretty:cucumber-pretty'
|
||||
],
|
||||
|
||||
// Output directory for reports
|
||||
formatOptions: {
|
||||
snippetInterface: 'async-await'
|
||||
},
|
||||
|
||||
// Tags to run (can be overridden via command line)
|
||||
tags: 'not @wip and not @skip',
|
||||
|
||||
// Fail fast on first error
|
||||
failFast: false,
|
||||
|
||||
// Retry failed tests (useful in CI)
|
||||
retry: process.env.CI ? 2 : 0,
|
||||
|
||||
// Dry run (just list scenarios without executing)
|
||||
dryRun: false,
|
||||
|
||||
// Strict mode (fail on undefined steps)
|
||||
strict: true,
|
||||
};
|
||||
@@ -0,0 +1,35 @@
|
||||
Feature: User Authentication
|
||||
As a registered user
|
||||
I want to log in and log out
|
||||
So that I can access my account features
|
||||
|
||||
@happy-path @authentication
|
||||
Scenario: Login with valid credentials
|
||||
Given I am logged in as a player
|
||||
When I go to the home page
|
||||
Then I should see "Sign out"
|
||||
|
||||
@happy-path @authentication
|
||||
Scenario: Logout successfully
|
||||
Given I am logged in as a player
|
||||
When I click the "Sign out" button
|
||||
Then I should be redirected to the login page
|
||||
|
||||
@negative-test @authentication
|
||||
Scenario: Login with invalid credentials
|
||||
Given I am on the login page
|
||||
When I log in with invalid credentials
|
||||
Then I should see an error message
|
||||
And I should remain on the login page
|
||||
|
||||
@happy-path @authentication
|
||||
Scenario: Navigate to login page
|
||||
Given I am on the home page
|
||||
When I click the "Log in" link
|
||||
Then I should be on the login page
|
||||
|
||||
@happy-path @authentication
|
||||
Scenario: Navigate to registration from login
|
||||
Given I am on the login page
|
||||
When I click the "Create an account" link
|
||||
Then I should be on the registration page
|
||||
@@ -0,0 +1,32 @@
|
||||
Feature: Password Reset
|
||||
As a user who forgot my password
|
||||
I want to reset my password
|
||||
So that I can regain access to my account
|
||||
|
||||
@happy-path @authentication @issue-10
|
||||
Scenario: User can access password reset page
|
||||
Given I am on the login page
|
||||
When I click the "Forgot password?" link
|
||||
Then I should be on the password reset page
|
||||
And I should see "Reset Password"
|
||||
|
||||
@happy-path @authentication @issue-10 @wip
|
||||
Scenario: User requests password reset with valid email
|
||||
Given I am on the password reset page
|
||||
When I fill in "email" with "existing-user@example.com"
|
||||
And I click the "Send Reset Link" button
|
||||
Then I should see "Password reset link sent"
|
||||
And I should see "Check your email"
|
||||
|
||||
@negative-test @authentication @issue-10 @wip
|
||||
Scenario: User requests password reset with invalid email
|
||||
Given I am on the password reset page
|
||||
When I fill in "email" with "nonexistent@example.com"
|
||||
And I click the "Send Reset Link" button
|
||||
Then I should see an error message
|
||||
|
||||
@negative-test @authentication @issue-10 @wip
|
||||
Scenario: User submits empty email field
|
||||
Given I am on the password reset page
|
||||
When I click the "Send Reset Link" button
|
||||
Then I should see "email is required" error
|
||||
@@ -0,0 +1,28 @@
|
||||
Feature: Player Schedule
|
||||
As a player
|
||||
I want to see my upcoming matches
|
||||
So that I can prepare for and attend my games
|
||||
|
||||
@happy-path @player-features @issue-9
|
||||
Scenario: Player views empty schedule when no matches
|
||||
Given I am logged in as a player
|
||||
When I go to my schedule page
|
||||
Then I should see "No upcoming matches"
|
||||
|
||||
@happy-path @player-features @issue-9 @wip
|
||||
Scenario: Player views schedule with upcoming matches
|
||||
Given I am logged in as a player
|
||||
And I have upcoming matches in my schedule
|
||||
When I go to my schedule page
|
||||
Then I should see "Upcoming Matches"
|
||||
And I should see the match date
|
||||
And I should see my opponent's name
|
||||
And I should see the tournament name
|
||||
|
||||
@happy-path @player-features @issue-9 @wip
|
||||
Scenario: Player can navigate to match details from schedule
|
||||
Given I am logged in as a player
|
||||
And I have upcoming matches in my schedule
|
||||
When I go to my schedule page
|
||||
And I click on a match
|
||||
Then I should be on the match detail page
|
||||
@@ -0,0 +1,39 @@
|
||||
Feature: Tournament Schedule
|
||||
As a tournament admin
|
||||
I want to view and generate round matchups in a Schedule tab
|
||||
So that I can manage tournament matches
|
||||
|
||||
@happy-path @tournament @issue-7
|
||||
Scenario: Tournament admin views schedule page for tournament
|
||||
Given I am logged in as a tournament admin
|
||||
And a tournament exists with 4 teams
|
||||
When I go to the tournament schedule page
|
||||
Then I should see "Schedule"
|
||||
And I should see the "Generate Schedule" button
|
||||
|
||||
@happy-path @tournament @issue-7 @wip
|
||||
Scenario: Tournament admin generates round-robin schedule
|
||||
Given I am logged in as a tournament admin
|
||||
And a tournament exists with 4 teams
|
||||
When I go to the tournament schedule page
|
||||
And I click the "Generate Schedule" button
|
||||
Then I should see "Schedule generated successfully"
|
||||
And I should see round 1 matchups
|
||||
And I should see round 2 matchups
|
||||
|
||||
@happy-path @tournament @issue-7 @wip
|
||||
Scenario: Tournament admin views schedule with bye rounds
|
||||
Given I am logged in as a tournament admin
|
||||
And a tournament exists with 5 teams
|
||||
When I go to the tournament schedule page
|
||||
And I click the "Generate Schedule" button
|
||||
Then I should see a bye round for one team
|
||||
And each team should play every other team exactly once
|
||||
|
||||
@happy-path @tournament @issue-7 @wip
|
||||
Scenario: Tournament admin clicks on a matchup to enter results
|
||||
Given I am logged in as a tournament admin
|
||||
And a tournament has a generated schedule
|
||||
When I go to the tournament schedule page
|
||||
And I click on a matchup
|
||||
Then I should be on the match result entry page
|
||||
@@ -0,0 +1,59 @@
|
||||
Feature: User Registration
|
||||
As a new user
|
||||
I want to register for an account
|
||||
So that I can participate in Euchre tournaments and track my games
|
||||
|
||||
Background:
|
||||
Given I am on the registration page
|
||||
|
||||
@happy-path @authentication
|
||||
Scenario: Successful registration with valid data
|
||||
When I fill in "name" with "Test User"
|
||||
And I fill in "email" with "test-user@example.com"
|
||||
And I fill in "password" with "TestPassword123!"
|
||||
And I click the "Create Account" button
|
||||
Then I should be redirected to my profile page
|
||||
And my user account should exist
|
||||
|
||||
@happy-path @authentication
|
||||
Scenario: Auto-created player profile is linked to user
|
||||
When I fill in "name" with "Profile Test User"
|
||||
And I fill in "email" with "profile-test@example.com"
|
||||
And I fill in "password" with "ProfilePass123!"
|
||||
And I click the "Create Account" button
|
||||
Then I should be redirected to my profile page
|
||||
And I should see "Welcome, Profile Test User"
|
||||
|
||||
@negative-test @authentication
|
||||
Scenario: Registration with duplicate email fails
|
||||
Given I am logged in as a player
|
||||
When I navigate to the registration page
|
||||
And I fill in "name" with "Duplicate User"
|
||||
And I fill in "email" with the same email
|
||||
And I fill in "password" with "TestPassword123!"
|
||||
And I click the "Create Account" button
|
||||
Then I should see a registration error
|
||||
|
||||
@negative-test @authentication
|
||||
Scenario: Registration with weak password fails
|
||||
When I fill in "name" with "Weak Password User"
|
||||
And I fill in "email" with "weak@example.com"
|
||||
And I fill in "password" with "weak"
|
||||
And I click the "Create Account" button
|
||||
Then I should see "password" validation error
|
||||
And I should remain on the registration page
|
||||
|
||||
@happy-path @authentication
|
||||
Scenario: Registration form validation
|
||||
When I click the "Create Account" button
|
||||
Then I should see "name is required" error
|
||||
And I should see "email is required" error
|
||||
And I should see "password is required" error
|
||||
|
||||
@wip
|
||||
Scenario: Registration with email verification (placeholder)
|
||||
When I fill in "name" with "Email Verify User"
|
||||
And I fill in "email" with "verify@example.com"
|
||||
And I fill in "password" with "TestPassword123!"
|
||||
And I click the "Create Account" button
|
||||
Then I should see "Please check your email to verify your account"
|
||||
@@ -0,0 +1,26 @@
|
||||
Feature: Wordmark Navigation
|
||||
As a user
|
||||
I want the EuchreCamp wordmark to navigate to my appropriate homepage
|
||||
So that I can quickly access my relevant content
|
||||
|
||||
@happy-path @navigation @issue-24
|
||||
Scenario: Unauthenticated user clicks wordmark goes to public homepage
|
||||
Given I am on the home page
|
||||
When I click the "EuchreCamp" wordmark
|
||||
Then I should be on the home page
|
||||
And I should see "Top 10 Players"
|
||||
And I should see "Sign In"
|
||||
|
||||
@happy-path @navigation @authenticated @issue-24
|
||||
Scenario: Authenticated player clicks wordmark goes to their profile
|
||||
Given I am logged in as a player
|
||||
When I click the "EuchreCamp" wordmark
|
||||
Then I should be redirected to my profile page
|
||||
And I should see "Welcome"
|
||||
|
||||
@happy-path @navigation @authenticated @issue-24
|
||||
Scenario: Authenticated admin clicks wordmark goes to admin dashboard
|
||||
Given I am logged in as a club admin
|
||||
When I click the "EuchreCamp" wordmark
|
||||
Then I should be on the admin page
|
||||
And I should see "Admin"
|
||||
@@ -0,0 +1,322 @@
|
||||
/**
|
||||
* Authentication step definitions for EuchreCamp E2E tests
|
||||
*
|
||||
* These steps interact ONLY with the browser UI - no direct database access.
|
||||
* All user creation happens via the registration/login UI.
|
||||
*/
|
||||
|
||||
import { Given, When, Then } from '@cucumber/cucumber';
|
||||
import { expect } from '@playwright/test';
|
||||
import { world } from '../support/world';
|
||||
|
||||
// Generate unique test credentials
|
||||
function generateTestCredentials() {
|
||||
const timestamp = Date.now();
|
||||
return {
|
||||
email: `cucumber-test-${timestamp}@example.com`,
|
||||
password: 'TestPassword123!',
|
||||
name: `Cucumber Test User ${timestamp}`
|
||||
};
|
||||
}
|
||||
|
||||
/**
|
||||
* Precondition: I am logged in as a player
|
||||
* Creates a new user via the registration UI
|
||||
*/
|
||||
Given('I am logged in as a player', async function () {
|
||||
console.log('🌍 Creating and logging in as a player via UI...');
|
||||
|
||||
const credentials = generateTestCredentials();
|
||||
world.user = credentials;
|
||||
|
||||
// Navigate to registration page
|
||||
await world.page.goto(`${world.baseURL}/auth/register`);
|
||||
await world.page.waitForLoadState('networkidle');
|
||||
|
||||
// Fill registration form
|
||||
await world.page.fill('input[name="name"]', credentials.name);
|
||||
await world.page.fill('input[name="email"]', credentials.email);
|
||||
await world.page.fill('input[name="password"]', credentials.password);
|
||||
|
||||
// Submit form
|
||||
await world.page.click('button[type="submit"]');
|
||||
|
||||
// Wait for redirect to profile page
|
||||
try {
|
||||
await world.page.waitForURL(/\/players\/\d+\/profile/, { timeout: 10000 });
|
||||
console.log(`🌍 Player created and logged in: ${credentials.email}`);
|
||||
} catch (e) {
|
||||
// If redirect doesn't happen, check if we're still on the page
|
||||
console.log('🌍 Registration may have failed or redirected elsewhere');
|
||||
console.log('🌍 Current URL:', world.page.url());
|
||||
}
|
||||
});
|
||||
|
||||
/**
|
||||
* Precondition: I am logged in as a tournament admin
|
||||
* Note: In the actual app, admin roles are assigned by club admins or via API.
|
||||
* For acceptance tests, we'll use the default player role and test admin features
|
||||
* as the dev site would handle them.
|
||||
*/
|
||||
Given('I am logged in as a tournament admin', async function () {
|
||||
console.log('🌍 Creating and logging in as a player (tournament admin role is assigned via UI/API)...');
|
||||
// For now, use the same flow as player
|
||||
// In real usage, the admin would either:
|
||||
// 1. Be pre-created on the dev site
|
||||
// 2. Have role assigned via API
|
||||
// 3. Use the admin dashboard to manage users
|
||||
|
||||
const credentials = generateTestCredentials();
|
||||
world.user = credentials;
|
||||
|
||||
await world.page.goto(`${world.baseURL}/auth/register`);
|
||||
await world.page.waitForLoadState('networkidle');
|
||||
|
||||
await world.page.fill('input[name="name"]', credentials.name);
|
||||
await world.page.fill('input[name="email"]', credentials.email);
|
||||
await world.page.fill('input[name="password"]', credentials.password);
|
||||
|
||||
await world.page.click('button[type="submit"]');
|
||||
await world.page.waitForLoadState('networkidle');
|
||||
|
||||
console.log(`🌍 User created: ${credentials.email}`);
|
||||
});
|
||||
|
||||
/**
|
||||
* Precondition: I am logged in as a club admin
|
||||
*/
|
||||
Given('I am logged in as a club admin', async function () {
|
||||
console.log('🌍 Creating and logging in as a club admin...');
|
||||
|
||||
const credentials = generateTestCredentials();
|
||||
world.user = credentials;
|
||||
|
||||
await world.page.goto(`${world.baseURL}/auth/register`);
|
||||
await world.page.waitForLoadState('networkidle');
|
||||
|
||||
await world.page.fill('input[name="name"]', credentials.name);
|
||||
await world.page.fill('input[name="email"]', credentials.email);
|
||||
await world.page.fill('input[name="password"]', credentials.password);
|
||||
|
||||
await world.page.click('button[type="submit"]');
|
||||
await world.page.waitForLoadState('networkidle');
|
||||
|
||||
console.log(`🌍 User created: ${credentials.email}`);
|
||||
});
|
||||
|
||||
/**
|
||||
* Registration steps (UI-only)
|
||||
*/
|
||||
When('I register with valid credentials', async function () {
|
||||
const credentials = generateTestCredentials();
|
||||
world.user = credentials;
|
||||
|
||||
await world.page.goto(`${world.baseURL}/auth/register`);
|
||||
await world.page.waitForLoadState('networkidle');
|
||||
|
||||
await world.page.fill('input[name="name"]', credentials.name);
|
||||
await world.page.fill('input[name="email"]', credentials.email);
|
||||
await world.page.fill('input[name="password"]', credentials.password);
|
||||
|
||||
await world.page.click('button[type="submit"]');
|
||||
|
||||
console.log(`🌍 Registered with credentials: ${credentials.email}`);
|
||||
});
|
||||
|
||||
When('I register with duplicate email', async function () {
|
||||
if (!world.user) {
|
||||
throw new Error('No existing user to duplicate');
|
||||
}
|
||||
|
||||
await world.page.goto(`${world.baseURL}/auth/register`);
|
||||
await world.page.waitForLoadState('networkidle');
|
||||
|
||||
await world.page.fill('input[name="name"]', world.user.name);
|
||||
await world.page.fill('input[name="email"]', world.user.email);
|
||||
await world.page.fill('input[name="password"]', world.user.password);
|
||||
|
||||
await world.page.click('button[type="submit"]');
|
||||
|
||||
console.log(`🌍 Attempted registration with duplicate: ${world.user.email}`);
|
||||
});
|
||||
|
||||
When('I fill in {string} with the same email', async function (fieldName: string) {
|
||||
if (!world.user) {
|
||||
throw new Error('No user credentials available');
|
||||
}
|
||||
|
||||
const selector = `input[name="email"]`;
|
||||
console.log(`🌍 Filling ${fieldName} with same email: ${world.user.email}`);
|
||||
await world.page.fill(selector, world.user.email);
|
||||
});
|
||||
|
||||
When('I register with weak password', async function () {
|
||||
const credentials = generateTestCredentials();
|
||||
credentials.password = 'weak'; // Too short
|
||||
|
||||
await world.page.goto(`${world.baseURL}/auth/register`);
|
||||
await world.page.waitForLoadState('networkidle');
|
||||
|
||||
await world.page.fill('input[name="name"]', credentials.name);
|
||||
await world.page.fill('input[name="email"]', credentials.email);
|
||||
await world.page.fill('input[name="password"]', credentials.password);
|
||||
|
||||
await world.page.click('button[type="submit"]');
|
||||
|
||||
console.log('🌍 Attempted registration with weak password');
|
||||
});
|
||||
|
||||
/**
|
||||
* Login steps (UI-only)
|
||||
*/
|
||||
When('I log in with valid credentials', async function () {
|
||||
if (!world.user) {
|
||||
throw new Error('No user credentials available');
|
||||
}
|
||||
|
||||
await world.page.goto(`${world.baseURL}/auth/login`);
|
||||
await world.page.waitForLoadState('networkidle');
|
||||
|
||||
await world.page.fill('input[name="email"]', world.user.email);
|
||||
await world.page.fill('input[name="password"]', world.user.password);
|
||||
|
||||
await world.page.click('button[type="submit"]');
|
||||
|
||||
console.log(`🌍 Logged in as: ${world.user.email}`);
|
||||
});
|
||||
|
||||
When('I log in with invalid credentials', async function () {
|
||||
await world.page.goto(`${world.baseURL}/auth/login`);
|
||||
await world.page.waitForLoadState('networkidle');
|
||||
|
||||
await world.page.fill('input[name="email"]', 'nonexistent@example.com');
|
||||
await world.page.fill('input[name="password"]', 'wrongpassword');
|
||||
|
||||
await world.page.click('button[type="submit"]');
|
||||
|
||||
console.log('🌍 Attempted login with invalid credentials');
|
||||
});
|
||||
|
||||
/**
|
||||
* Logout steps (UI-only)
|
||||
*/
|
||||
When('I log out', async function () {
|
||||
// Click on logout button
|
||||
await world.page.click('text=Sign out');
|
||||
|
||||
console.log('🌍 Logged out');
|
||||
});
|
||||
|
||||
/**
|
||||
* Verification steps (browser-based)
|
||||
*/
|
||||
Then('I should be logged in', async function () {
|
||||
// Check for logout button or user menu
|
||||
await expect(world.page.locator('text=Sign out')).toBeVisible();
|
||||
console.log('🌍 Verified user is logged in');
|
||||
});
|
||||
|
||||
Then('I should not be logged in', async function () {
|
||||
// Check that we're on login page or don't see logout button
|
||||
const currentUrl = world.page.url();
|
||||
expect(currentUrl).toContain('/auth/login');
|
||||
console.log('🌍 Verified user is not logged in');
|
||||
});
|
||||
|
||||
Then('I should see a registration error', async function () {
|
||||
// Check for error message in page content
|
||||
const content = await world.page.content();
|
||||
expect(content).toContain('error');
|
||||
console.log('🌍 Verified registration error is visible');
|
||||
});
|
||||
|
||||
Then('I should see an error message', async function () {
|
||||
// Check for any error message on the page
|
||||
const content = await world.page.content();
|
||||
expect(content).toMatch(/error|Error/i);
|
||||
console.log('🌍 Verified error message is visible');
|
||||
});
|
||||
|
||||
Then('I should remain on the login page', async function () {
|
||||
const currentUrl = world.page.url();
|
||||
expect(currentUrl).toContain('/auth/login');
|
||||
console.log('🌍 Verified still on login page');
|
||||
});
|
||||
|
||||
Then('I should remain on the registration page', async function () {
|
||||
const currentUrl = world.page.url();
|
||||
expect(currentUrl).toContain('/auth/register');
|
||||
console.log('🌍 Verified still on registration page');
|
||||
});
|
||||
|
||||
/**
|
||||
* Helper: Verify user was created via UI (no database check)
|
||||
*/
|
||||
Then('my user account should exist', async function () {
|
||||
if (!world.user) {
|
||||
throw new Error('No user credentials available');
|
||||
}
|
||||
|
||||
// We can't check the database directly in acceptance tests
|
||||
// Instead, verify we're logged in (user profile page)
|
||||
await expect(world.page.locator('text=Welcome')).toBeVisible();
|
||||
console.log('🌍 Verified user account was created successfully');
|
||||
});
|
||||
|
||||
/**
|
||||
* Schedule steps (player)
|
||||
*/
|
||||
When('I go to my schedule page', async function () {
|
||||
console.log('🌍 Going to schedule page');
|
||||
// Navigate to the schedule page
|
||||
// The URL pattern would be /players/{playerId}/schedule
|
||||
// We'll navigate to /players/schedule which should redirect or work
|
||||
await world.page.goto(`${world.baseURL}/players/schedule`);
|
||||
await world.page.waitForLoadState('networkidle');
|
||||
});
|
||||
|
||||
Given('I have upcoming matches in my schedule', async function () {
|
||||
console.log('🌍 Note: This step requires database setup via API or UI');
|
||||
console.log('🌍 For acceptance tests, this would be set up before running the test');
|
||||
// For true acceptance testing, we would:
|
||||
// 1. Create a tournament
|
||||
// 2. Add the player as a participant
|
||||
// 3. Generate a schedule
|
||||
// 4. The match would then appear in the player's schedule
|
||||
|
||||
// For now, this is a placeholder that indicates data setup is needed
|
||||
// In a real test run, this data would already exist in the dev database
|
||||
});
|
||||
|
||||
/**
|
||||
* Tournament schedule steps
|
||||
*/
|
||||
Given('a tournament exists with {int} teams', async function (teamCount: number) {
|
||||
console.log(`🌍 Note: Tournament with ${teamCount} teams requires data setup`);
|
||||
console.log('🌍 For acceptance tests, this would be created via UI or API');
|
||||
// In a real test run, this would either:
|
||||
// 1. Create a tournament via the UI (slower but more realistic)
|
||||
// 2. Use API to create tournament and add teams (faster)
|
||||
// 3. Pre-existing test data in dev database
|
||||
|
||||
// Store the team count in world context for later steps
|
||||
world.tournamentTeamCount = teamCount;
|
||||
});
|
||||
|
||||
When('I go to the tournament schedule page', async function () {
|
||||
console.log('🌍 Going to tournament schedule page');
|
||||
// Navigate to a tournament schedule page
|
||||
// This would typically be /admin/tournaments/{id}/schedule
|
||||
// For testing, we'll go to the first tournament's schedule
|
||||
await world.page.goto(`${world.baseURL}/admin/tournaments/1/schedule`);
|
||||
await world.page.waitForLoadState('networkidle');
|
||||
});
|
||||
|
||||
Given('a tournament has a generated schedule', async function () {
|
||||
console.log('🌍 Note: Tournament schedule requires generation via API or UI');
|
||||
console.log('🌍 For acceptance tests, this would be created before running the test');
|
||||
// In a real test run, we would:
|
||||
// 1. Create a tournament
|
||||
// 2. Add teams/participants
|
||||
// 3. Generate schedule via API or UI
|
||||
});
|
||||
@@ -0,0 +1,309 @@
|
||||
/**
|
||||
* Common step definitions for EuchreCamp E2E tests
|
||||
*/
|
||||
|
||||
import { Given, When, Then } from '@cucumber/cucumber';
|
||||
import { expect } from '@playwright/test';
|
||||
import { world } from '../support/world';
|
||||
|
||||
// console.log('🌍 Loading common-steps.ts step definitions');
|
||||
|
||||
/**
|
||||
* Navigation steps
|
||||
*/
|
||||
Given('I am on the home page', async function () {
|
||||
console.log('🌍 Navigating to home page');
|
||||
await world.page.goto(world.baseURL);
|
||||
await world.page.waitForLoadState('networkidle');
|
||||
});
|
||||
|
||||
Given('I am on the registration page', async function () {
|
||||
console.log('🌍 Navigating to registration page');
|
||||
await world.page.goto(`${world.baseURL}/auth/register`);
|
||||
await world.page.waitForLoadState('networkidle');
|
||||
});
|
||||
|
||||
Given('I am on the login page', async function () {
|
||||
console.log('🌍 Navigating to login page');
|
||||
await world.page.goto(`${world.baseURL}/auth/login`);
|
||||
await world.page.waitForLoadState('networkidle');
|
||||
});
|
||||
|
||||
Given('I am on the {string} page', async function (pageName: string) {
|
||||
const pageUrls: Record<string, string> = {
|
||||
'home': '/',
|
||||
'home page': '/',
|
||||
'registration': '/auth/register',
|
||||
'registration page': '/auth/register',
|
||||
'login': '/auth/login',
|
||||
'login page': '/auth/login',
|
||||
'rankings': '/rankings',
|
||||
'rankings page': '/rankings',
|
||||
'admin': '/admin',
|
||||
'admin page': '/admin',
|
||||
'admin dashboard': '/admin',
|
||||
'tournaments': '/admin/tournaments',
|
||||
'tournaments page': '/admin/tournaments',
|
||||
};
|
||||
|
||||
const url = pageUrls[pageName.toLowerCase()] || `/${pageName.toLowerCase().replace(' ', '-')}`;
|
||||
|
||||
console.log(`🌍 Navigating to ${pageName}: ${url}`);
|
||||
await world.page.goto(`${world.baseURL}${url}`);
|
||||
|
||||
// Wait for page to load
|
||||
await world.page.waitForLoadState('networkidle');
|
||||
});
|
||||
|
||||
When('I navigate to the {string} page', async function (pageName: string) {
|
||||
// This step is a synonym for "I go to the {string} page"
|
||||
await world.page.goto(`${world.baseURL}/auth/register`);
|
||||
await world.page.waitForLoadState('networkidle');
|
||||
});
|
||||
|
||||
When('I navigate to the registration page', async function () {
|
||||
console.log('🌍 Navigating to registration page');
|
||||
await world.page.goto(`${world.baseURL}/auth/register`);
|
||||
await world.page.waitForLoadState('networkidle');
|
||||
});
|
||||
|
||||
When('I go to the home page', async function () {
|
||||
console.log('🌍 Navigating to home page');
|
||||
await world.page.goto(world.baseURL);
|
||||
await world.page.waitForLoadState('networkidle');
|
||||
});
|
||||
|
||||
When('I go to the {string} page', async function (pageName: string) {
|
||||
const pageUrls: Record<string, string> = {
|
||||
'home': '/',
|
||||
'home page': '/',
|
||||
'registration': '/auth/register',
|
||||
'registration page': '/auth/register',
|
||||
'login': '/auth/login',
|
||||
'login page': '/auth/login',
|
||||
'rankings': '/rankings',
|
||||
'rankings page': '/rankings',
|
||||
'admin': '/admin',
|
||||
'admin page': '/admin',
|
||||
'admin dashboard': '/admin',
|
||||
'tournaments': '/admin/tournaments',
|
||||
'tournaments page': '/admin/tournaments',
|
||||
};
|
||||
|
||||
const url = pageUrls[pageName.toLowerCase()] || `/${pageName.toLowerCase().replace(' ', '-')}`;
|
||||
|
||||
console.log(`🌍 Going to ${pageName}: ${url}`);
|
||||
await world.page.goto(`${world.baseURL}${url}`);
|
||||
await world.page.waitForLoadState('networkidle');
|
||||
});
|
||||
|
||||
When('I go back', async function () {
|
||||
await world.page.goBack();
|
||||
await world.page.waitForLoadState('networkidle');
|
||||
});
|
||||
|
||||
When('I refresh the page', async function () {
|
||||
await world.page.reload();
|
||||
await world.page.waitForLoadState('networkidle');
|
||||
});
|
||||
|
||||
/**
|
||||
* Form interaction steps
|
||||
*/
|
||||
When('I fill in {string} with {string}', async function (fieldName: string, value: string) {
|
||||
// Map field names to selectors
|
||||
const fieldSelectors: Record<string, string> = {
|
||||
'name': 'input[name="name"]',
|
||||
'email': 'input[name="email"]',
|
||||
'password': 'input[name="password"]',
|
||||
'confirm password': 'input[name="confirmPassword"]',
|
||||
'player name': 'input[name="playerName"]',
|
||||
'tournament name': 'input[name="name"]',
|
||||
};
|
||||
|
||||
const selector = fieldSelectors[fieldName.toLowerCase()] || `input[name="${fieldName.toLowerCase()}"]`;
|
||||
|
||||
console.log(`🌍 Filling ${fieldName} with ${value}`);
|
||||
await world.page.fill(selector, value);
|
||||
});
|
||||
|
||||
When('I click the {string} button', async function (buttonText: string) {
|
||||
const selector = `button:has-text("${buttonText}")`;
|
||||
console.log(`🌍 Clicking button: ${buttonText}`);
|
||||
await world.page.click(selector);
|
||||
});
|
||||
|
||||
When('I click the {string} link', async function (linkText: string) {
|
||||
const selector = `a:has-text("${linkText}")`;
|
||||
console.log(`🌍 Clicking link: ${linkText}`);
|
||||
await world.page.click(selector);
|
||||
});
|
||||
|
||||
When('I click the {string} wordmark', async function (wordmarkText: string) {
|
||||
const selector = `a:has-text("${wordmarkText}")`;
|
||||
console.log(`🌍 Clicking wordmark: ${wordmarkText}`);
|
||||
await world.page.click(selector);
|
||||
await world.page.waitForLoadState('networkidle');
|
||||
});
|
||||
|
||||
When('I click the {string} element', async function (elementText: string) {
|
||||
const selector = `text=${elementText}`;
|
||||
console.log(`🌍 Clicking element: ${elementText}`);
|
||||
await world.page.click(selector);
|
||||
});
|
||||
|
||||
/**
|
||||
* Assertion steps
|
||||
*/
|
||||
Then('I should see {string}', async function (text: string) {
|
||||
console.log(`🌍 Checking for text: ${text}`);
|
||||
await expect(world.page.locator(`text=${text}`)).toBeVisible();
|
||||
});
|
||||
|
||||
Then('I should not see {string}', async function (text: string) {
|
||||
console.log(`🌍 Checking text is NOT visible: ${text}`);
|
||||
await expect(world.page.locator(`text=${text}`)).not.toBeVisible();
|
||||
});
|
||||
|
||||
Then('I should be on the {string} page', async function (pageName: string) {
|
||||
const pageUrls: Record<string, string> = {
|
||||
'home': '/',
|
||||
'home page': '/',
|
||||
'registration': '/auth/register',
|
||||
'registration page': '/auth/register',
|
||||
'login': '/auth/login',
|
||||
'login page': '/auth/login',
|
||||
'profile': '/players',
|
||||
'admin': '/admin',
|
||||
'admin page': '/admin',
|
||||
};
|
||||
|
||||
const expectedPath = pageUrls[pageName.toLowerCase()] || `/${pageName.toLowerCase().replace(' ', '-')}`;
|
||||
const currentUrl = world.page.url();
|
||||
|
||||
console.log(`🌍 Checking current URL: ${currentUrl}`);
|
||||
console.log(`🌍 Expected path: ${expectedPath}`);
|
||||
|
||||
expect(currentUrl).toContain(expectedPath);
|
||||
});
|
||||
|
||||
Then('I should be redirected to my profile page', async function () {
|
||||
await world.page.waitForLoadState('networkidle');
|
||||
const currentUrl = world.page.url();
|
||||
|
||||
console.log(`🌍 Checking redirect to profile page. Current URL: ${currentUrl}`);
|
||||
expect(currentUrl).toMatch(/\/players\/\d+\/profile/);
|
||||
});
|
||||
|
||||
Then('I should be on the registration page', async function () {
|
||||
const currentUrl = world.page.url();
|
||||
|
||||
console.log(`🌍 Checking current URL: ${currentUrl}`);
|
||||
expect(currentUrl).toContain('/auth/register');
|
||||
});
|
||||
|
||||
Then('I should be on the login page', async function () {
|
||||
const currentUrl = world.page.url();
|
||||
|
||||
console.log(`🌍 Checking current URL: ${currentUrl}`);
|
||||
expect(currentUrl).toContain('/auth/login');
|
||||
});
|
||||
|
||||
Then('I should be on the admin page', async function () {
|
||||
const currentUrl = world.page.url();
|
||||
|
||||
console.log(`🌍 Checking current URL: ${currentUrl}`);
|
||||
expect(currentUrl).toContain('/admin');
|
||||
});
|
||||
|
||||
Then('I should be on the home page', async function () {
|
||||
const currentUrl = world.page.url();
|
||||
|
||||
console.log(`🌍 Checking current URL: ${currentUrl}`);
|
||||
expect(currentUrl).toContain('/');
|
||||
});
|
||||
|
||||
Then('I should be on the password reset page', async function () {
|
||||
const currentUrl = world.page.url();
|
||||
|
||||
console.log(`🌍 Checking current URL: ${currentUrl}`);
|
||||
expect(currentUrl).toContain('/auth/password-reset');
|
||||
});
|
||||
|
||||
Then('I should see the {string} button', async function (buttonText: string) {
|
||||
const button = world.page.locator(`button:has-text("${buttonText}")`);
|
||||
await expect(button).toBeVisible();
|
||||
console.log(`🌍 Verified button is visible: ${buttonText}`);
|
||||
});
|
||||
|
||||
Then('I should be redirected to {string}', async function (path: string) {
|
||||
await world.page.waitForLoadState('networkidle');
|
||||
const currentUrl = world.page.url();
|
||||
|
||||
console.log(`🌍 Checking redirect to: ${path}`);
|
||||
console.log(`🌍 Current URL: ${currentUrl}`);
|
||||
|
||||
expect(currentUrl).toContain(path);
|
||||
});
|
||||
|
||||
/**
|
||||
* Element visibility steps
|
||||
*/
|
||||
Then('the element {string} should be visible', async function (elementText: string) {
|
||||
const element = world.page.locator(`text=${elementText}`);
|
||||
await expect(element).toBeVisible();
|
||||
});
|
||||
|
||||
Then('the element {string} should not be visible', async function (elementText: string) {
|
||||
const element = world.page.locator(`text=${elementText}`);
|
||||
await expect(element).not.toBeVisible();
|
||||
});
|
||||
|
||||
/**
|
||||
* Wait steps
|
||||
*/
|
||||
When('I wait for {int} seconds', async function (seconds: number) {
|
||||
console.log(`🌍 Waiting ${seconds} seconds`);
|
||||
await world.page.waitForTimeout(seconds * 1000);
|
||||
});
|
||||
|
||||
When('I wait for the page to load', async function () {
|
||||
await world.page.waitForLoadState('networkidle');
|
||||
});
|
||||
|
||||
/**
|
||||
* URL verification steps
|
||||
*/
|
||||
Then('the URL should contain {string}', async function (expectedPath: string) {
|
||||
const currentUrl = world.page.url();
|
||||
console.log(`🌍 Checking URL contains: ${expectedPath}`);
|
||||
expect(currentUrl).toContain(expectedPath);
|
||||
});
|
||||
|
||||
Then('I should be redirected to the login page', async function () {
|
||||
await world.page.waitForLoadState('networkidle');
|
||||
const currentUrl = world.page.url();
|
||||
|
||||
console.log(`🌍 Checking redirect to login page. Current URL: ${currentUrl}`);
|
||||
expect(currentUrl).toContain('/auth/login');
|
||||
});
|
||||
|
||||
Then('I should see a {string} error', async function (errorMessage: string) {
|
||||
const content = await world.page.content();
|
||||
expect(content).toMatch(new RegExp(errorMessage, 'i'));
|
||||
console.log(`🌍 Verified error message: ${errorMessage}`);
|
||||
});
|
||||
|
||||
Then('I should see {string} validation error', async function (field: string) {
|
||||
// Look for validation error near the input field
|
||||
const content = await world.page.content();
|
||||
// Check if the field name is mentioned in an error context
|
||||
expect(content).toMatch(new RegExp(field, 'i'));
|
||||
console.log(`🌍 Verified validation error for: ${field}`);
|
||||
});
|
||||
|
||||
Then('I should see {string} error', async function (errorMessage: string) {
|
||||
const content = await world.page.content();
|
||||
expect(content).toMatch(new RegExp(errorMessage, 'i'));
|
||||
console.log(`🌍 Verified error message: ${errorMessage}`);
|
||||
});
|
||||
@@ -0,0 +1,108 @@
|
||||
/**
|
||||
* Cucumber hooks for EuchreCamp E2E tests
|
||||
*/
|
||||
|
||||
import { Before, After, BeforeAll, AfterAll } from '@cucumber/cucumber';
|
||||
import { chromium, Browser } from '@playwright/test';
|
||||
import { world } from './world';
|
||||
import path from 'path';
|
||||
import fs from 'fs';
|
||||
|
||||
// Global browser instance
|
||||
let browser: Browser;
|
||||
|
||||
// Load environment files
|
||||
const envPath = path.resolve(process.cwd(), '.env');
|
||||
const envDevPath = path.resolve(process.cwd(), '.env.development');
|
||||
|
||||
if (fs.existsSync(envPath)) {
|
||||
require('dotenv').config({ path: envPath });
|
||||
}
|
||||
|
||||
if (fs.existsSync(envDevPath)) {
|
||||
require('dotenv').config({ path: envDevPath, override: true });
|
||||
}
|
||||
|
||||
/**
|
||||
* Before all scenarios: Launch browser
|
||||
*/
|
||||
BeforeAll(async function () {
|
||||
console.log('🌍 Launching browser for Cucumber tests...');
|
||||
browser = await chromium.launch();
|
||||
});
|
||||
|
||||
/**
|
||||
* After all scenarios: Close browser and cleanup
|
||||
*/
|
||||
AfterAll(async function () {
|
||||
console.log('🌍 Closing browser...');
|
||||
await browser.close();
|
||||
|
||||
console.log('🌍 Disconnecting from database...');
|
||||
if (world.prisma) {
|
||||
await world.prisma.$disconnect();
|
||||
}
|
||||
});
|
||||
|
||||
/**
|
||||
* Before each scenario: Create new page context
|
||||
*/
|
||||
Before(async function () {
|
||||
// Create new context and page
|
||||
world.context = await browser.newContext();
|
||||
world.page = await world.context.newPage();
|
||||
|
||||
// Log console messages for debugging
|
||||
world.page.on('console', msg => {
|
||||
if (msg.type() === 'error') {
|
||||
console.log('❌ PAGE ERROR:', msg.text());
|
||||
}
|
||||
});
|
||||
|
||||
// Log network errors
|
||||
world.page.on('requestfailed', request => {
|
||||
console.log('❌ REQUEST FAILED:', request.url());
|
||||
});
|
||||
|
||||
console.log('🌍 Created new page context');
|
||||
});
|
||||
|
||||
/**
|
||||
* After each scenario: Close page
|
||||
*/
|
||||
After(async function () {
|
||||
console.log('🌍 Cleaning up after scenario...');
|
||||
|
||||
// Close page and context
|
||||
if (world.page) {
|
||||
await world.page.close();
|
||||
}
|
||||
if (world.context) {
|
||||
await world.context.close();
|
||||
}
|
||||
});
|
||||
|
||||
/**
|
||||
* Custom hook for navigating to a page
|
||||
*/
|
||||
Before({ tags: '@navigation' }, async function () {
|
||||
console.log('🌍 Navigation hook triggered');
|
||||
});
|
||||
|
||||
/**
|
||||
* Custom hook for authenticated scenarios
|
||||
*/
|
||||
Before({ tags: '@authenticated' }, async function () {
|
||||
console.log('🌍 Authenticated hook triggered');
|
||||
});
|
||||
|
||||
/**
|
||||
* Tag hooks for specific scenarios
|
||||
*/
|
||||
Before({ tags: '@slow' }, async function () {
|
||||
console.log('🌍 Slow test - extending timeout');
|
||||
});
|
||||
|
||||
Before({ tags: '@flaky' }, async function () {
|
||||
console.log('🌍 Flaky test - retry enabled');
|
||||
});
|
||||
@@ -0,0 +1,55 @@
|
||||
/**
|
||||
* World context for Cucumber tests
|
||||
* Provides shared state between step definitions
|
||||
*/
|
||||
|
||||
import { Page, BrowserContext, chromium, Browser } from '@playwright/test';
|
||||
|
||||
export interface WorldState {
|
||||
page: Page;
|
||||
context: BrowserContext;
|
||||
prisma: any; // Lazy-loaded PrismaClient
|
||||
baseURL: string;
|
||||
user?: {
|
||||
email: string;
|
||||
name: string;
|
||||
password: string;
|
||||
};
|
||||
tournament?: any;
|
||||
tournamentTeamCount?: number;
|
||||
player?: any;
|
||||
match?: any;
|
||||
}
|
||||
|
||||
export class World implements WorldState {
|
||||
page!: Page;
|
||||
context!: BrowserContext;
|
||||
prisma: any;
|
||||
baseURL: string;
|
||||
user?: {
|
||||
email: string;
|
||||
name: string;
|
||||
password: string;
|
||||
};
|
||||
tournament?: any;
|
||||
tournamentTeamCount?: number;
|
||||
player?: any;
|
||||
match?: any;
|
||||
|
||||
constructor() {
|
||||
this.baseURL = process.env.BASE_URL || 'http://localhost:3000';
|
||||
this.prisma = null; // Will be lazily loaded when needed
|
||||
}
|
||||
|
||||
async getPrisma() {
|
||||
if (!this.prisma) {
|
||||
// Lazily load PrismaClient when first accessed
|
||||
const { PrismaClient } = await import('@prisma/client');
|
||||
this.prisma = new PrismaClient();
|
||||
}
|
||||
return this.prisma;
|
||||
}
|
||||
}
|
||||
|
||||
// Export singleton instance
|
||||
export const world = new World();
|
||||
+3
-3
@@ -41,7 +41,7 @@ test.describe('Tournament Edit - allowTies functionality', () => {
|
||||
}
|
||||
});
|
||||
|
||||
test('should display allowTies checkbox on edit form', async ({ page }) => {
|
||||
test('should display allowTies checkbox on edit form @chromium-admin', async ({ page }) => {
|
||||
// Navigate to tournament edit page
|
||||
await page.goto(`/admin/tournaments/${tournamentId}/edit`);
|
||||
|
||||
@@ -54,7 +54,7 @@ test.describe('Tournament Edit - allowTies functionality', () => {
|
||||
await expect(allowTiesCheckbox).not.toBeChecked();
|
||||
});
|
||||
|
||||
test('should save allowTies when toggled to true', async ({ page }) => {
|
||||
test('should save allowTies when toggled to true @chromium-admin', async ({ page }) => {
|
||||
// Navigate to tournament edit page
|
||||
await page.goto(`/admin/tournaments/${tournamentId}/edit`);
|
||||
|
||||
@@ -80,7 +80,7 @@ test.describe('Tournament Edit - allowTies functionality', () => {
|
||||
expect(updatedTournament?.allowTies).toBe(true);
|
||||
});
|
||||
|
||||
test('should save allowTies when toggled to false', async ({ page }) => {
|
||||
test('should save allowTies when toggled to false @chromium-admin', async ({ page }) => {
|
||||
// First, set allowTies to true
|
||||
await prisma.event.update({
|
||||
where: { id: tournamentId },
|
||||
@@ -1,4 +1,5 @@
|
||||
[tools]
|
||||
bun = "latest"
|
||||
docker-compose = "latest"
|
||||
just = "latest"
|
||||
node = "latest"
|
||||
|
||||
+36
-24
@@ -1,38 +1,45 @@
|
||||
{
|
||||
"name": "euchre_camp",
|
||||
"version": "0.1.3",
|
||||
"version": "0.1.4",
|
||||
"private": true,
|
||||
"scripts": {
|
||||
"dev": "NEXT_PUBLIC_GIT_COMMIT=$(git rev-parse --short HEAD) next dev",
|
||||
"build": "next build",
|
||||
"start": "next start",
|
||||
"lint": "eslint",
|
||||
"test": "vitest",
|
||||
"test:run": "vitest run",
|
||||
"test:acceptance": "playwright test src/__tests__/e2e/",
|
||||
"test:acceptance:headed": "playwright test src/__tests__/e2e/ --headed",
|
||||
"db:switch": "node scripts/switch-database.js",
|
||||
"db:setup-postgres": "node scripts/setup-postgres.js",
|
||||
"db:setup-dev": "node scripts/setup-postgres.js",
|
||||
"db:setup-dev:clean": "node scripts/setup-postgres.js --drop",
|
||||
"db:reset-dev": "node scripts/reset-dev-db.js",
|
||||
"db:use-dev": "node scripts/use-dev-db.js",
|
||||
"db:cleanup-prod": "node scripts/cleanup-prod-db.js",
|
||||
"db:check-prod": "node scripts/check-test-records.js",
|
||||
"db:seed": "node scripts/seed.js",
|
||||
"lint": "bun run eslint",
|
||||
"test": "bun test 'src/__tests__/unit/**' 'src/__tests__/*.test.tsx' 'src/__tests__/auth-simple.test.ts'",
|
||||
"test:unit": "bun test src/__tests__/unit/",
|
||||
"test:component": "bun test src/__tests__/*.test.tsx",
|
||||
"test:run": "bun test 'src/__tests__/unit/**' 'src/__tests__/*.test.tsx' 'src/__tests__/auth-simple.test.ts'",
|
||||
"test:randomize": "bun test src/__tests__/unit/ --randomize",
|
||||
"test:unit:sequential": "bun test src/__tests__/unit/ --max-concurrency=1",
|
||||
"test:acceptance": "bun x playwright test e2e/",
|
||||
"test:acceptance:headed": "bun x playwright test e2e/ --headed",
|
||||
"test:acceptance:cucumber": "bun cucumber-js --config e2e/cucumber/cucumber.config.ts",
|
||||
"test:acceptance:cucumber:pretty": "bun cucumber-js --config e2e/cucumber/cucumber.config.ts --format pretty:cucumber-pretty",
|
||||
"cucumber": "bun cucumber-js --config e2e/cucumber/cucumber.config.ts",
|
||||
"db:switch": "bun run scripts/switch-database.js",
|
||||
"db:setup-postgres": "bun run scripts/setup-postgres.js",
|
||||
"db:setup-dev": "bun run scripts/setup-postgres.js",
|
||||
"db:setup-dev:clean": "bun run scripts/setup-postgres.js --drop",
|
||||
"db:reset-dev": "bun run scripts/reset-dev-db.js",
|
||||
"db:use-dev": "bun run scripts/use-dev-db.js",
|
||||
"db:cleanup-prod": "bun run scripts/cleanup-prod-db.js",
|
||||
"db:check-prod": "bun run scripts/check-test-records.js",
|
||||
"db:seed": "bun run scripts/seed.js",
|
||||
"docker:up": "docker-compose up -d",
|
||||
"docker:down": "docker-compose down",
|
||||
"docker:logs": "docker-compose logs -f",
|
||||
"docker:build": "docker-compose build",
|
||||
"docker:shell": "docker exec -it euchre-camp-app sh",
|
||||
"version": "node scripts/bump-version.js",
|
||||
"set-version": "node scripts/set-version.js",
|
||||
"version:patch": "node scripts/bump-version.js patch",
|
||||
"version:minor": "node scripts/bump-version.js minor",
|
||||
"version:major": "node scripts/bump-version.js major",
|
||||
"docker:build:push": "node scripts/build-and-push-docker.js",
|
||||
"docker:compose:generate": "node scripts/generate-docker-compose.js",
|
||||
"release": "npm run version:patch && npm run docker:compose:generate && npm run docker:build:push"
|
||||
"version": "bun run scripts/bump-version.js",
|
||||
"set-version": "bun run scripts/set-version.js",
|
||||
"version:patch": "bun run scripts/bump-version.js patch",
|
||||
"version:minor": "bun run scripts/bump-version.js minor",
|
||||
"version:major": "bun run scripts/bump-version.js major",
|
||||
"docker:build:push": "bun run scripts/build-and-push-docker.js",
|
||||
"docker:compose:generate": "bun run scripts/generate-docker-compose.js",
|
||||
"release": "bun run version:patch && bun run docker:compose:generate && bun run docker:build:push"
|
||||
},
|
||||
"dependencies": {
|
||||
"@hookform/resolvers": "^5.2.2",
|
||||
@@ -55,12 +62,15 @@
|
||||
"zod": "^4.3.6"
|
||||
},
|
||||
"devDependencies": {
|
||||
"@cucumber/cucumber": "^12.8.2",
|
||||
"@playwright/test": "^1.58.2",
|
||||
"@tailwindcss/postcss": "^4",
|
||||
"@testing-library/jest-dom": "^6.9.1",
|
||||
"@testing-library/react": "^16.3.2",
|
||||
"@testing-library/user-event": "^14.6.1",
|
||||
"@types/bcrypt": "^6.0.0",
|
||||
"@types/bun": "^1.3.11",
|
||||
"@types/jsdom": "^28.0.1",
|
||||
"@types/node": "^20",
|
||||
"@types/papaparse": "^5.5.2",
|
||||
"@types/pg": "^8.20.0",
|
||||
@@ -68,10 +78,12 @@
|
||||
"@types/react-dom": "^19.2.3",
|
||||
"@vitejs/plugin-react": "^6.0.1",
|
||||
"argon2": "^0.44.0",
|
||||
"eslint": "^10.1.0",
|
||||
"cucumber-pretty": "^6.0.1",
|
||||
"eslint": "^8.57.0",
|
||||
"eslint-config-next": "^16.2.1",
|
||||
"jsdom": "^29.0.1",
|
||||
"tailwindcss": "^4",
|
||||
"tsx": "^4.21.0",
|
||||
"typescript": "^5",
|
||||
"vitest": "^4.1.2"
|
||||
}
|
||||
|
||||
@@ -1,7 +1,7 @@
|
||||
import { defineConfig, devices } from '@playwright/test';
|
||||
|
||||
export default defineConfig({
|
||||
testDir: './src/__tests__/e2e',
|
||||
testDir: './e2e',
|
||||
timeout: 30000,
|
||||
expect: {
|
||||
timeout: 5000
|
||||
@@ -17,7 +17,7 @@ export default defineConfig({
|
||||
// Reporter to use
|
||||
reporter: 'html',
|
||||
// Global setup and teardown
|
||||
globalSetup: require.resolve('./src/__tests__/e2e/global.setup'),
|
||||
globalSetup: require.resolve('./e2e/global.setup'),
|
||||
// Use base URL for relative navigation
|
||||
use: {
|
||||
baseURL: 'http://localhost:3000',
|
||||
|
||||
@@ -12,7 +12,7 @@ const path = require('path');
|
||||
const { execSync } = require('child_process');
|
||||
|
||||
// Configuration
|
||||
const REGISTRY = 'euchre-camp';
|
||||
const REGISTRY = 'docker.notsosm.art';
|
||||
const IMAGE_NAME = 'euchre-camp';
|
||||
|
||||
// Get current version from package.json
|
||||
|
||||
@@ -1,25 +1,25 @@
|
||||
import { describe, it, expect, vi, beforeEach, MockedFunction } from 'vitest'
|
||||
import { describe, it, expect, mock, beforeEach } from 'bun:test'
|
||||
import { render, screen, waitFor } from '@testing-library/react'
|
||||
import userEvent from '@testing-library/user-event'
|
||||
import EditTournamentForm from '@/components/EditTournamentForm'
|
||||
|
||||
// Mock next/navigation
|
||||
vi.mock('next/navigation', () => ({
|
||||
mock.module('next/navigation', () => ({
|
||||
useRouter: () => ({
|
||||
push: vi.fn(),
|
||||
push: mock(() => {}),
|
||||
}),
|
||||
}))
|
||||
|
||||
// Mock next/link
|
||||
vi.mock('next/link', () => ({
|
||||
mock.module('next/link', () => ({
|
||||
default: ({ children, href }: { children: React.ReactNode; href: string }) => (
|
||||
<a href={href}>{children}</a>
|
||||
),
|
||||
}))
|
||||
|
||||
// Mock fetch
|
||||
const mockFetch = vi.fn()
|
||||
global.fetch = mockFetch as MockedFunction<typeof global.fetch>
|
||||
const mockFetch = mock(async () => new Response())
|
||||
global.fetch = mockFetch as any
|
||||
|
||||
const mockTournament = {
|
||||
id: 1,
|
||||
@@ -40,7 +40,8 @@ const mockTournament = {
|
||||
|
||||
describe('EditTournamentForm', () => {
|
||||
beforeEach(() => {
|
||||
vi.clearAllMocks()
|
||||
// Only clear the specific mocks we create, not global module mocks
|
||||
mockFetch.mockClear()
|
||||
})
|
||||
|
||||
it('renders form with initial values', () => {
|
||||
|
||||
@@ -5,31 +5,40 @@
|
||||
* Tests the navigation component for proper display based on session state
|
||||
*/
|
||||
|
||||
import { describe, it, expect, vi, beforeEach, afterEach } from 'vitest'
|
||||
import { describe, it, expect, mock, beforeEach, afterEach } from 'bun:test'
|
||||
import { render, screen, waitFor } from '@testing-library/react'
|
||||
import Navigation from '@/components/Navigation'
|
||||
|
||||
// Mock next/link
|
||||
mock.module('next/link', () => ({
|
||||
default: ({ children, href }: { children: React.ReactNode; href: string }) => (
|
||||
<a href={href}>{children}</a>
|
||||
),
|
||||
}))
|
||||
|
||||
// Mock the SessionProvider
|
||||
vi.mock('@/components/SessionProvider', () => ({
|
||||
useSession: vi.fn(),
|
||||
mock.module('@/components/SessionProvider', () => ({
|
||||
useSession: mock(() => ({ session: null, loading: false, refreshSession: mock(() => {}) })),
|
||||
}))
|
||||
|
||||
// Mock the auth-client
|
||||
vi.mock('@/lib/auth-client', () => ({
|
||||
mock.module('@/lib/auth-client', () => ({
|
||||
authClient: {
|
||||
signOut: vi.fn(),
|
||||
signOut: mock(() => {}),
|
||||
},
|
||||
}))
|
||||
|
||||
// Mock fetch for role API call
|
||||
global.fetch = vi.fn()
|
||||
global.fetch = mock(async () => new Response()) as any
|
||||
|
||||
import { useSession } from '@/components/SessionProvider'
|
||||
import { useSession as useSessionOriginal } from '@/components/SessionProvider'
|
||||
const useSession = useSessionOriginal as any
|
||||
|
||||
describe('Epic 1: Navigation Component', () => {
|
||||
beforeEach(() => {
|
||||
vi.clearAllMocks()
|
||||
vi.mocked(global.fetch).mockImplementation(async (url) => {
|
||||
// Don't clear all mocks as it might affect bun-setup.ts
|
||||
// Set up default fetch mock
|
||||
(global.fetch as any).mockImplementation?.(async (url: any) => {
|
||||
if (url?.toString().includes('/api/users/')) {
|
||||
return new Response(JSON.stringify({ role: 'player' }), {
|
||||
status: 200,
|
||||
@@ -41,14 +50,15 @@ describe('Epic 1: Navigation Component', () => {
|
||||
})
|
||||
|
||||
afterEach(() => {
|
||||
vi.restoreAllMocks()
|
||||
// Don't clear mocks - let them persist for other test files
|
||||
// Navigation relies on module mocks set up at file level
|
||||
})
|
||||
|
||||
it('renders the logo and basic links when not logged in', () => {
|
||||
vi.mocked(useSession).mockReturnValue({
|
||||
(useSession).mockReturnValue({
|
||||
session: null,
|
||||
loading: false,
|
||||
refreshSession: vi.fn(),
|
||||
refreshSession: mock(() => {}),
|
||||
})
|
||||
|
||||
render(<Navigation />)
|
||||
@@ -60,7 +70,7 @@ describe('Epic 1: Navigation Component', () => {
|
||||
})
|
||||
|
||||
it('shows user menu when logged in', async () => {
|
||||
vi.mocked(useSession).mockReturnValue({
|
||||
(useSession).mockReturnValue({
|
||||
session: {
|
||||
user: {
|
||||
id: 'user-123',
|
||||
@@ -71,7 +81,7 @@ describe('Epic 1: Navigation Component', () => {
|
||||
session: { token: 'abc123' },
|
||||
},
|
||||
loading: false,
|
||||
refreshSession: vi.fn(),
|
||||
refreshSession: mock(() => {}),
|
||||
})
|
||||
|
||||
render(<Navigation />)
|
||||
@@ -85,7 +95,7 @@ describe('Epic 1: Navigation Component', () => {
|
||||
})
|
||||
|
||||
it('shows Tournaments link when logged in', async () => {
|
||||
vi.mocked(useSession).mockReturnValue({
|
||||
(useSession).mockReturnValue({
|
||||
session: {
|
||||
user: {
|
||||
id: 'user-123',
|
||||
@@ -96,7 +106,7 @@ describe('Epic 1: Navigation Component', () => {
|
||||
session: { token: 'abc123' },
|
||||
},
|
||||
loading: false,
|
||||
refreshSession: vi.fn(),
|
||||
refreshSession: mock(() => {}),
|
||||
})
|
||||
|
||||
render(<Navigation />)
|
||||
@@ -107,7 +117,7 @@ describe('Epic 1: Navigation Component', () => {
|
||||
})
|
||||
|
||||
it('shows admin link for club_admin role', async () => {
|
||||
vi.mocked(useSession).mockReturnValue({
|
||||
(useSession).mockReturnValue({
|
||||
session: {
|
||||
user: {
|
||||
id: 'admin-123',
|
||||
@@ -118,11 +128,11 @@ describe('Epic 1: Navigation Component', () => {
|
||||
session: { token: 'abc123' },
|
||||
},
|
||||
loading: false,
|
||||
refreshSession: vi.fn(),
|
||||
})
|
||||
refreshSession: mock(() => {}),
|
||||
});
|
||||
|
||||
// Mock fetch to return club_admin role
|
||||
vi.mocked(global.fetch).mockImplementation(async (url) => {
|
||||
(global.fetch as any).mockImplementation(async (url: any) => {
|
||||
if (url?.toString().includes('/api/users/admin-123/role')) {
|
||||
return new Response(JSON.stringify({ role: 'club_admin' }), {
|
||||
status: 200,
|
||||
@@ -141,7 +151,7 @@ describe('Epic 1: Navigation Component', () => {
|
||||
})
|
||||
|
||||
it('hides admin link for non-admin users', async () => {
|
||||
vi.mocked(useSession).mockReturnValue({
|
||||
(useSession).mockReturnValue({
|
||||
session: {
|
||||
user: {
|
||||
id: 'player-123',
|
||||
@@ -152,11 +162,11 @@ describe('Epic 1: Navigation Component', () => {
|
||||
session: { token: 'abc123' },
|
||||
},
|
||||
loading: false,
|
||||
refreshSession: vi.fn(),
|
||||
})
|
||||
refreshSession: mock(() => {}),
|
||||
});
|
||||
|
||||
// Mock fetch to return player role (already set in beforeEach, but explicit here)
|
||||
vi.mocked(global.fetch).mockImplementation(async (url) => {
|
||||
(global.fetch as any).mockImplementation(async (url: any) => {
|
||||
if (url?.toString().includes('/api/users/')) {
|
||||
return {
|
||||
json: () => Promise.resolve({ role: 'player' }),
|
||||
@@ -176,10 +186,10 @@ describe('Epic 1: Navigation Component', () => {
|
||||
})
|
||||
|
||||
it('shows loading state', () => {
|
||||
vi.mocked(useSession).mockReturnValue({
|
||||
(useSession).mockReturnValue({
|
||||
session: null,
|
||||
loading: true,
|
||||
refreshSession: vi.fn(),
|
||||
refreshSession: mock(() => {}),
|
||||
})
|
||||
|
||||
render(<Navigation />)
|
||||
|
||||
@@ -1,23 +1,23 @@
|
||||
import { describe, it, expect, vi, beforeEach, MockedFunction } from 'vitest'
|
||||
import { describe, it, expect, mock, beforeEach } from 'bun:test'
|
||||
import { getSession } from '@/lib/auth-simple'
|
||||
|
||||
// Mock next/headers
|
||||
vi.mock('next/headers', () => ({
|
||||
cookies: vi.fn().mockResolvedValue({
|
||||
get: vi.fn().mockReturnValue({ name: 'better-auth.session_token', value: 'test-token' }),
|
||||
}),
|
||||
headers: vi.fn().mockResolvedValue({
|
||||
get: vi.fn().mockReturnValue(null),
|
||||
}),
|
||||
mock.module('next/headers', () => ({
|
||||
cookies: mock(() => Promise.resolve({
|
||||
get: mock(() => ({ name: 'better-auth.session_token', value: 'test-token' })),
|
||||
})),
|
||||
headers: mock(() => Promise.resolve({
|
||||
get: mock(() => null),
|
||||
})),
|
||||
}))
|
||||
|
||||
// Mock fetch
|
||||
const mockFetch = vi.fn()
|
||||
global.fetch = mockFetch as MockedFunction<typeof global.fetch>
|
||||
const mockFetch = mock(async () => new Response())
|
||||
global.fetch = mockFetch as any
|
||||
|
||||
describe('getSession', () => {
|
||||
beforeEach(() => {
|
||||
vi.clearAllMocks()
|
||||
mock.clearAllMocks()
|
||||
})
|
||||
|
||||
it('returns session data when auth API returns 200', async () => {
|
||||
@@ -26,7 +26,7 @@ describe('getSession', () => {
|
||||
user: { id: 'user-123', email: 'test@example.com' },
|
||||
}
|
||||
|
||||
mockFetch.mockResolvedValue(
|
||||
mockFetch.mockImplementation(async () =>
|
||||
new Response(JSON.stringify(mockSession), { status: 200 })
|
||||
)
|
||||
|
||||
@@ -37,7 +37,7 @@ describe('getSession', () => {
|
||||
})
|
||||
|
||||
it('returns null when auth API returns non-200 status', async () => {
|
||||
mockFetch.mockResolvedValue(
|
||||
mockFetch.mockImplementation(async () =>
|
||||
new Response(null, { status: 401 })
|
||||
)
|
||||
|
||||
|
||||
@@ -0,0 +1,40 @@
|
||||
// Setup file for Bun test runner to provide DOM environment
|
||||
import { JSDOM } from 'jsdom';
|
||||
import { mock } from 'bun:test';
|
||||
|
||||
console.log('Loading bun-setup.ts...');
|
||||
|
||||
// Mock @prisma/client to avoid dependency on generated Prisma client
|
||||
// This allows unit tests to run without requiring `prisma generate`
|
||||
mock.module('@prisma/client', () => {
|
||||
return {
|
||||
PrismaClient: class MockPrismaClient {
|
||||
$connect() {}
|
||||
$disconnect() {}
|
||||
$transaction() {}
|
||||
}
|
||||
};
|
||||
});
|
||||
|
||||
const dom = new JSDOM('<!DOCTYPE html><html><body></body></html>', {
|
||||
url: 'http://localhost',
|
||||
pretendToBeVisual: true,
|
||||
});
|
||||
|
||||
(global as any).window = dom.window;
|
||||
(global as any).document = dom.window.document;
|
||||
(global as any).navigator = dom.window.navigator;
|
||||
|
||||
console.log('bun-setup.ts loaded - document:', typeof (global as any).document);
|
||||
console.log('document.body:', (global as any).document?.body);
|
||||
|
||||
// Import jest-dom matchers after setting up the DOM
|
||||
import '@testing-library/jest-dom';
|
||||
|
||||
// Clear document body after each test
|
||||
import { afterEach } from 'bun:test';
|
||||
afterEach(() => {
|
||||
if (global.document?.body) {
|
||||
global.document.body.innerHTML = '';
|
||||
}
|
||||
});
|
||||
Vendored
+12
@@ -0,0 +1,12 @@
|
||||
/// <reference types="@testing-library/jest-dom" />
|
||||
|
||||
import { type expect } from 'bun:test'
|
||||
import { type TestingLibraryMatchers } from '@testing-library/jest-dom/matchers'
|
||||
|
||||
declare module 'bun:test' {
|
||||
interface Matchers<T = any>
|
||||
extends TestingLibraryMatchers<
|
||||
ReturnType<typeof expect.stringContaining>,
|
||||
T
|
||||
> {}
|
||||
}
|
||||
@@ -4,7 +4,7 @@
|
||||
* Tests the mathematical correctness of Elo calculations and rating updates
|
||||
*/
|
||||
|
||||
import { describe, test, expect } from 'vitest';
|
||||
import { describe, test, expect } from 'bun:test';
|
||||
import { calculateEloChange, calculateExpectedScore, calculateTeamElo } from '@/lib/elo-utils';
|
||||
|
||||
describe('Elo Rating System', () => {
|
||||
|
||||
@@ -4,7 +4,7 @@
|
||||
* Tests for ID parsing and validation in API routes and pages
|
||||
*/
|
||||
|
||||
import { describe, test, expect } from 'vitest';
|
||||
import { describe, test, expect } from 'bun:test';
|
||||
|
||||
describe('ID Validation', () => {
|
||||
describe('parseInt with validation', () => {
|
||||
|
||||
@@ -4,12 +4,33 @@
|
||||
* Tests the permission system for tournament management
|
||||
*/
|
||||
|
||||
import { describe, test, expect, vi } from 'vitest';
|
||||
import { describe, test, expect, mock, beforeEach } from 'bun:test';
|
||||
import { hasRole, canManageTournament, canCreateTournaments } from '@/lib/permissions';
|
||||
import { getSession } from '@/lib/auth-simple';
|
||||
import { prisma } from '@/lib/prisma';
|
||||
import type { User } from '@prisma/client';
|
||||
|
||||
// Create mock functions at module level
|
||||
const getSessionMock = mock(() => {});
|
||||
const userFindUniqueMock = mock(() => {});
|
||||
const eventFindUniqueMock = mock(() => {});
|
||||
|
||||
// Mock the getSession and prisma functions
|
||||
mock.module('@/lib/auth-simple', () => ({
|
||||
getSession: getSessionMock,
|
||||
}));
|
||||
|
||||
mock.module('@/lib/prisma', () => ({
|
||||
prisma: {
|
||||
user: {
|
||||
findUnique: userFindUniqueMock,
|
||||
},
|
||||
event: {
|
||||
findUnique: eventFindUniqueMock,
|
||||
},
|
||||
},
|
||||
}));
|
||||
|
||||
// Helper to create mock user
|
||||
const createMockUser = (id: string, email: string, role: string): User => ({
|
||||
id,
|
||||
@@ -23,30 +44,21 @@ const createMockUser = (id: string, email: string, role: string): User => ({
|
||||
updatedAt: new Date(),
|
||||
});
|
||||
|
||||
// Mock the getSession and prisma functions
|
||||
vi.mock('@/lib/auth-simple', () => ({
|
||||
getSession: vi.fn(),
|
||||
}));
|
||||
|
||||
vi.mock('@/lib/prisma', () => ({
|
||||
prisma: {
|
||||
user: {
|
||||
findUnique: vi.fn(),
|
||||
},
|
||||
event: {
|
||||
findUnique: vi.fn(),
|
||||
},
|
||||
},
|
||||
}));
|
||||
|
||||
describe('Permissions', () => {
|
||||
beforeEach(() => {
|
||||
// Reset mock implementations to default (no-op) before each test
|
||||
getSessionMock.mockImplementation(() => undefined);
|
||||
userFindUniqueMock.mockImplementation(() => undefined);
|
||||
eventFindUniqueMock.mockImplementation(() => undefined);
|
||||
});
|
||||
|
||||
describe('hasRole', () => {
|
||||
test('should allow club_admin to access tournament_admin resources', async () => {
|
||||
vi.mocked(getSession).mockResolvedValue({
|
||||
getSessionMock.mockImplementation(async () => ({
|
||||
user: { id: '1', email: 'test@example.com' },
|
||||
session: { token: 'test', expiresAt: new Date() }
|
||||
});
|
||||
vi.mocked(prisma.user.findUnique).mockResolvedValue(
|
||||
}));
|
||||
userFindUniqueMock.mockImplementation(async () =>
|
||||
createMockUser('1', 'test@example.com', 'club_admin')
|
||||
);
|
||||
|
||||
@@ -55,11 +67,11 @@ describe('Permissions', () => {
|
||||
});
|
||||
|
||||
test('should deny player from accessing tournament_admin resources', async () => {
|
||||
vi.mocked(getSession).mockResolvedValue({
|
||||
getSessionMock.mockImplementation(async () => ({
|
||||
user: { id: '1', email: 'test@example.com' },
|
||||
session: { token: 'test', expiresAt: new Date() }
|
||||
});
|
||||
vi.mocked(prisma.user.findUnique).mockResolvedValue(
|
||||
}));
|
||||
userFindUniqueMock.mockImplementation(async () =>
|
||||
createMockUser('1', 'test@example.com', 'player')
|
||||
);
|
||||
|
||||
@@ -68,7 +80,7 @@ describe('Permissions', () => {
|
||||
});
|
||||
|
||||
test('should deny unauthenticated user', async () => {
|
||||
vi.mocked(getSession).mockResolvedValue(null);
|
||||
getSessionMock.mockImplementation(async () => null);
|
||||
|
||||
const result = await hasRole('club_admin');
|
||||
expect(result.allowed).toBe(false);
|
||||
@@ -78,11 +90,11 @@ describe('Permissions', () => {
|
||||
|
||||
describe('canManageTournament', () => {
|
||||
test('should allow club_admin to manage any tournament', async () => {
|
||||
vi.mocked(getSession).mockResolvedValue({
|
||||
getSessionMock.mockImplementation(async () => ({
|
||||
user: { id: 'admin-1', email: 'admin@example.com' },
|
||||
session: { token: 'test', expiresAt: new Date() }
|
||||
});
|
||||
vi.mocked(prisma.user.findUnique).mockResolvedValue(
|
||||
}));
|
||||
userFindUniqueMock.mockImplementation(async () =>
|
||||
createMockUser('admin-1', 'admin@example.com', 'club_admin')
|
||||
);
|
||||
|
||||
@@ -91,11 +103,11 @@ describe('Permissions', () => {
|
||||
});
|
||||
|
||||
test('should deny player from managing tournaments', async () => {
|
||||
vi.mocked(getSession).mockResolvedValue({
|
||||
getSessionMock.mockImplementation(async () => ({
|
||||
user: { id: 'player-1', email: 'player@example.com' },
|
||||
session: { token: 'test', expiresAt: new Date() }
|
||||
});
|
||||
vi.mocked(prisma.user.findUnique).mockResolvedValue(
|
||||
}));
|
||||
userFindUniqueMock.mockImplementation(async () =>
|
||||
createMockUser('player-1', 'player@example.com', 'player')
|
||||
);
|
||||
|
||||
@@ -107,11 +119,11 @@ describe('Permissions', () => {
|
||||
|
||||
describe('canCreateTournaments', () => {
|
||||
test('should allow tournament_admin to create tournaments', async () => {
|
||||
vi.mocked(getSession).mockResolvedValue({
|
||||
getSessionMock.mockImplementation(async () => ({
|
||||
user: { id: 'admin-1', email: 'admin@example.com' },
|
||||
session: { token: 'test', expiresAt: new Date() }
|
||||
});
|
||||
vi.mocked(prisma.user.findUnique).mockResolvedValue(
|
||||
}));
|
||||
userFindUniqueMock.mockImplementation(async () =>
|
||||
createMockUser('admin-1', 'admin@example.com', 'tournament_admin')
|
||||
);
|
||||
|
||||
@@ -120,11 +132,11 @@ describe('Permissions', () => {
|
||||
});
|
||||
|
||||
test('should allow club_admin to create tournaments', async () => {
|
||||
vi.mocked(getSession).mockResolvedValue({
|
||||
getSessionMock.mockImplementation(async () => ({
|
||||
user: { id: 'admin-1', email: 'admin@example.com' },
|
||||
session: { token: 'test', expiresAt: new Date() }
|
||||
});
|
||||
vi.mocked(prisma.user.findUnique).mockResolvedValue(
|
||||
}));
|
||||
userFindUniqueMock.mockImplementation(async () =>
|
||||
createMockUser('admin-1', 'admin@example.com', 'club_admin')
|
||||
);
|
||||
|
||||
@@ -133,11 +145,11 @@ describe('Permissions', () => {
|
||||
});
|
||||
|
||||
test('should deny player from creating tournaments', async () => {
|
||||
vi.mocked(getSession).mockResolvedValue({
|
||||
getSessionMock.mockImplementation(async () => ({
|
||||
user: { id: 'player-1', email: 'player@example.com' },
|
||||
session: { token: 'test', expiresAt: new Date() }
|
||||
});
|
||||
vi.mocked(prisma.user.findUnique).mockResolvedValue(
|
||||
}));
|
||||
userFindUniqueMock.mockImplementation(async () =>
|
||||
createMockUser('player-1', 'player@example.com', 'player')
|
||||
);
|
||||
|
||||
|
||||
@@ -4,15 +4,19 @@
|
||||
* Tests the player deduplication logic for CSV uploads
|
||||
*/
|
||||
|
||||
import { describe, test, expect, vi, beforeEach } from 'vitest';
|
||||
import { describe, test, expect, mock, beforeEach,} from 'bun:test';
|
||||
import { prisma } from '@/lib/prisma';
|
||||
|
||||
// Create mock functions at module level
|
||||
const playerFindFirstMock = mock(() => {});
|
||||
const playerCreateMock = mock(() => {});
|
||||
|
||||
// Mock the prisma module
|
||||
vi.mock('@/lib/prisma', () => ({
|
||||
mock.module('@/lib/prisma', () => ({
|
||||
prisma: {
|
||||
player: {
|
||||
findFirst: vi.fn(),
|
||||
create: vi.fn(),
|
||||
findFirst: playerFindFirstMock,
|
||||
create: playerCreateMock,
|
||||
},
|
||||
},
|
||||
}));
|
||||
@@ -46,7 +50,9 @@ async function findOrCreatePlayer(name: string) {
|
||||
|
||||
describe('Player Deduplication', () => {
|
||||
beforeEach(() => {
|
||||
vi.clearAllMocks();
|
||||
// Clear all mock history before each test
|
||||
playerFindFirstMock.mockClear();
|
||||
playerCreateMock.mockClear();
|
||||
});
|
||||
|
||||
describe('findOrCreatePlayer', () => {
|
||||
@@ -64,7 +70,7 @@ describe('Player Deduplication', () => {
|
||||
rating: 0,
|
||||
};
|
||||
|
||||
vi.mocked(prisma.player.findFirst).mockResolvedValue(mockPlayer);
|
||||
playerFindFirstMock.mockImplementation(async () => mockPlayer);
|
||||
|
||||
const result = await findOrCreatePlayer('Emily');
|
||||
|
||||
@@ -89,7 +95,7 @@ describe('Player Deduplication', () => {
|
||||
rating: 0,
|
||||
};
|
||||
|
||||
vi.mocked(prisma.player.findFirst).mockResolvedValue(mockPlayer);
|
||||
playerFindFirstMock.mockImplementation(async () => mockPlayer);
|
||||
|
||||
const result = await findOrCreatePlayer('EMILY');
|
||||
|
||||
@@ -114,7 +120,7 @@ describe('Player Deduplication', () => {
|
||||
rating: 0,
|
||||
};
|
||||
|
||||
vi.mocked(prisma.player.findFirst).mockResolvedValue(mockPlayer);
|
||||
playerFindFirstMock.mockImplementation(async () => mockPlayer);
|
||||
|
||||
const result = await findOrCreatePlayer(' Emily ');
|
||||
|
||||
@@ -126,7 +132,7 @@ describe('Player Deduplication', () => {
|
||||
});
|
||||
|
||||
test('should create new player if not found', async () => {
|
||||
vi.mocked(prisma.player.findFirst).mockResolvedValue(null);
|
||||
playerFindFirstMock.mockImplementation(async () => null);
|
||||
|
||||
const newPlayer = {
|
||||
id: 100,
|
||||
@@ -141,7 +147,7 @@ describe('Player Deduplication', () => {
|
||||
updatedAt: new Date(),
|
||||
};
|
||||
|
||||
vi.mocked(prisma.player.create).mockResolvedValue(newPlayer);
|
||||
playerCreateMock.mockImplementation(async () => newPlayer);
|
||||
|
||||
const result = await findOrCreatePlayer('NewPlayer');
|
||||
|
||||
@@ -162,7 +168,7 @@ describe('Player Deduplication', () => {
|
||||
});
|
||||
|
||||
test('should handle names with special characters', async () => {
|
||||
vi.mocked(prisma.player.findFirst).mockResolvedValue(null);
|
||||
playerFindFirstMock.mockImplementation(async () => null);
|
||||
|
||||
const newPlayer = {
|
||||
id: 100,
|
||||
@@ -177,7 +183,7 @@ describe('Player Deduplication', () => {
|
||||
updatedAt: new Date(),
|
||||
};
|
||||
|
||||
vi.mocked(prisma.player.create).mockResolvedValue(newPlayer);
|
||||
playerCreateMock.mockImplementation(async () => newPlayer);
|
||||
|
||||
const result = await findOrCreatePlayer('Test-Player_123');
|
||||
|
||||
@@ -195,7 +201,7 @@ describe('Player Deduplication', () => {
|
||||
});
|
||||
|
||||
test('should handle names with spaces', async () => {
|
||||
vi.mocked(prisma.player.findFirst).mockResolvedValue(null);
|
||||
playerFindFirstMock.mockImplementation(async () => null);
|
||||
|
||||
const newPlayer = {
|
||||
id: 100,
|
||||
@@ -210,7 +216,7 @@ describe('Player Deduplication', () => {
|
||||
updatedAt: new Date(),
|
||||
};
|
||||
|
||||
vi.mocked(prisma.player.create).mockResolvedValue(newPlayer);
|
||||
playerCreateMock.mockImplementation(async () => newPlayer);
|
||||
|
||||
const result = await findOrCreatePlayer('Dave B');
|
||||
|
||||
@@ -242,7 +248,7 @@ describe('Player Deduplication', () => {
|
||||
rating: 0,
|
||||
};
|
||||
|
||||
vi.mocked(prisma.player.findFirst).mockResolvedValue(mockPlayer);
|
||||
playerFindFirstMock.mockImplementation(async () => mockPlayer);
|
||||
|
||||
const result1 = await findOrCreatePlayer('EMILY');
|
||||
const result2 = await findOrCreatePlayer('Emily');
|
||||
@@ -255,7 +261,7 @@ describe('Player Deduplication', () => {
|
||||
});
|
||||
|
||||
test('should handle empty or whitespace-only names', async () => {
|
||||
vi.mocked(prisma.player.findFirst).mockResolvedValue(null);
|
||||
playerFindFirstMock.mockImplementation(async () => null);
|
||||
|
||||
const newPlayer = {
|
||||
id: 100,
|
||||
@@ -270,7 +276,7 @@ describe('Player Deduplication', () => {
|
||||
updatedAt: new Date(),
|
||||
};
|
||||
|
||||
vi.mocked(prisma.player.create).mockResolvedValue(newPlayer);
|
||||
playerCreateMock.mockImplementation(async () => newPlayer);
|
||||
|
||||
const result = await findOrCreatePlayer(' ');
|
||||
|
||||
@@ -320,7 +326,7 @@ describe('Player Deduplication', () => {
|
||||
rating: 0,
|
||||
};
|
||||
|
||||
vi.mocked(prisma.player.findFirst)
|
||||
playerFindFirstMock
|
||||
.mockResolvedValueOnce(existingPlayer) // First call for "Emily"
|
||||
.mockResolvedValueOnce(existingPlayer) // Second call for "EMILY"
|
||||
.mockResolvedValueOnce(existingPlayer); // Third call for " Emily "
|
||||
|
||||
@@ -7,24 +7,30 @@
|
||||
* - Partnership performance
|
||||
*/
|
||||
|
||||
import { describe, test, expect, vi, beforeEach } from 'vitest';
|
||||
import { describe, test, expect, mock, beforeEach } from 'bun:test';
|
||||
import { prisma } from '@/lib/prisma';
|
||||
import type { Player, Event, Match, PartnershipStat } from '@prisma/client';
|
||||
|
||||
// Create mock functions at module level
|
||||
const playerFindUniqueMock = mock(() => {});
|
||||
const eventFindManyMock = mock(() => {});
|
||||
const matchFindManyMock = mock(() => {});
|
||||
const partnershipStatFindManyMock = mock(() => {});
|
||||
|
||||
// Mock the prisma module
|
||||
vi.mock('@/lib/prisma', () => ({
|
||||
mock.module('@/lib/prisma', () => ({
|
||||
prisma: {
|
||||
player: {
|
||||
findUnique: vi.fn(),
|
||||
findUnique: playerFindUniqueMock,
|
||||
},
|
||||
event: {
|
||||
findMany: vi.fn(),
|
||||
findMany: eventFindManyMock,
|
||||
},
|
||||
match: {
|
||||
findMany: vi.fn(),
|
||||
findMany: matchFindManyMock,
|
||||
},
|
||||
partnershipStat: {
|
||||
findMany: vi.fn(),
|
||||
findMany: partnershipStatFindManyMock,
|
||||
},
|
||||
},
|
||||
}));
|
||||
@@ -111,7 +117,11 @@ const createMockPartnershipStat = (
|
||||
|
||||
describe('Player Profile Enhancements', () => {
|
||||
beforeEach(() => {
|
||||
vi.clearAllMocks();
|
||||
// Reset mock implementations to default (no-op) before each test
|
||||
playerFindUniqueMock.mockImplementation(() => undefined);
|
||||
eventFindManyMock.mockImplementation(() => undefined);
|
||||
matchFindManyMock.mockImplementation(() => undefined);
|
||||
partnershipStatFindManyMock.mockImplementation(() => undefined);
|
||||
});
|
||||
|
||||
describe('Tournaments Participated', () => {
|
||||
@@ -122,8 +132,8 @@ describe('Player Profile Enhancements', () => {
|
||||
createMockTournament(2, 'Tournament B'),
|
||||
];
|
||||
|
||||
vi.mocked(prisma.player.findUnique).mockResolvedValue(mockPlayer);
|
||||
vi.mocked(prisma.event.findMany).mockResolvedValue(mockTournaments);
|
||||
playerFindUniqueMock.mockImplementation(async () => mockPlayer);
|
||||
eventFindManyMock.mockImplementation(async () => mockTournaments);
|
||||
|
||||
// Simulate the query that would be run
|
||||
const tournaments = await prisma.event.findMany({
|
||||
@@ -145,8 +155,8 @@ describe('Player Profile Enhancements', () => {
|
||||
test('should return empty array if player has no tournaments', async () => {
|
||||
const mockPlayer = createMockPlayer(1, 'Test Player');
|
||||
|
||||
vi.mocked(prisma.player.findUnique).mockResolvedValue(mockPlayer);
|
||||
vi.mocked(prisma.event.findMany).mockResolvedValue([]);
|
||||
playerFindUniqueMock.mockImplementation(async () => mockPlayer);
|
||||
eventFindManyMock.mockImplementation(async () => []);
|
||||
|
||||
const tournaments = await prisma.event.findMany({
|
||||
where: {
|
||||
@@ -173,8 +183,8 @@ describe('Player Profile Enhancements', () => {
|
||||
createMockMatch(2, 1, 3, 5, 6, 4, 4),
|
||||
];
|
||||
|
||||
vi.mocked(prisma.player.findUnique).mockResolvedValue(mockPlayer);
|
||||
vi.mocked(prisma.match.findMany).mockResolvedValue(mockMatches);
|
||||
playerFindUniqueMock.mockImplementation(async () => mockPlayer);
|
||||
matchFindManyMock.mockImplementation(async () => mockMatches);
|
||||
|
||||
// Simulate the query that would be run
|
||||
const matches = await prisma.match.findMany({
|
||||
@@ -204,8 +214,8 @@ describe('Player Profile Enhancements', () => {
|
||||
test('should return empty array if player has no matches', async () => {
|
||||
const mockPlayer = createMockPlayer(1, 'Test Player');
|
||||
|
||||
vi.mocked(prisma.player.findUnique).mockResolvedValue(mockPlayer);
|
||||
vi.mocked(prisma.match.findMany).mockResolvedValue([]);
|
||||
playerFindUniqueMock.mockImplementation(async () => mockPlayer);
|
||||
matchFindManyMock.mockImplementation(async () => []);
|
||||
|
||||
const matches = await prisma.match.findMany({
|
||||
where: {
|
||||
@@ -240,8 +250,8 @@ describe('Player Profile Enhancements', () => {
|
||||
createMockPartnershipStat(2, 1, 3, 5, 2, 3),
|
||||
];
|
||||
|
||||
vi.mocked(prisma.player.findUnique).mockResolvedValue(mockPlayer);
|
||||
vi.mocked(prisma.partnershipStat.findMany).mockResolvedValue(mockPartnershipStats);
|
||||
playerFindUniqueMock.mockImplementation(async () => mockPlayer);
|
||||
partnershipStatFindManyMock.mockImplementation(async () => mockPartnershipStats);
|
||||
|
||||
// Simulate the query that would be run
|
||||
const partnershipStats = await prisma.partnershipStat.findMany({
|
||||
@@ -265,8 +275,8 @@ describe('Player Profile Enhancements', () => {
|
||||
test('should return empty array if player has no partnership data', async () => {
|
||||
const mockPlayer = createMockPlayer(1, 'Test Player');
|
||||
|
||||
vi.mocked(prisma.player.findUnique).mockResolvedValue(mockPlayer);
|
||||
vi.mocked(prisma.partnershipStat.findMany).mockResolvedValue([]);
|
||||
playerFindUniqueMock.mockImplementation(async () => mockPlayer);
|
||||
partnershipStatFindManyMock.mockImplementation(async () => []);
|
||||
|
||||
const partnershipStats = await prisma.partnershipStat.findMany({
|
||||
where: {
|
||||
|
||||
@@ -5,33 +5,33 @@
|
||||
*/
|
||||
|
||||
/* eslint-disable @typescript-eslint/no-explicit-any */
|
||||
import { describe, test, expect, beforeEach, vi } from 'vitest';
|
||||
import { describe, test, expect, beforeEach, mock } from 'bun:test';
|
||||
import { recalculateAllElo } from '@/lib/elo-utils';
|
||||
|
||||
// Mock Prisma client
|
||||
const mockPrisma = {
|
||||
player: {
|
||||
updateMany: vi.fn().mockResolvedValue({ count: 0 }),
|
||||
update: vi.fn().mockResolvedValue({}),
|
||||
updateMany: mock(() => {}).mockResolvedValue({ count: 0 }),
|
||||
update: mock(() => {}).mockResolvedValue({}),
|
||||
},
|
||||
eloSnapshot: {
|
||||
deleteMany: vi.fn().mockResolvedValue({ count: 0 }),
|
||||
create: vi.fn().mockResolvedValue({}),
|
||||
deleteMany: mock(() => {}).mockResolvedValue({ count: 0 }),
|
||||
create: mock(() => {}).mockResolvedValue({}),
|
||||
},
|
||||
partnershipStat: {
|
||||
deleteMany: vi.fn().mockResolvedValue({ count: 0 }),
|
||||
findFirst: vi.fn().mockResolvedValue(null), // No existing stats initially
|
||||
update: vi.fn().mockResolvedValue({}),
|
||||
create: vi.fn().mockResolvedValue({}),
|
||||
deleteMany: mock(() => {}).mockResolvedValue({ count: 0 }),
|
||||
findFirst: mock(() => {}).mockResolvedValue(null), // No existing stats initially
|
||||
update: mock(() => {}).mockResolvedValue({}),
|
||||
create: mock(() => {}).mockResolvedValue({}),
|
||||
},
|
||||
match: {
|
||||
findMany: vi.fn().mockResolvedValue([]),
|
||||
findMany: mock(() => {}).mockResolvedValue([]),
|
||||
},
|
||||
};
|
||||
|
||||
describe('recalculateAllElo', () => {
|
||||
beforeEach(() => {
|
||||
vi.clearAllMocks();
|
||||
mock.clearAllMocks();
|
||||
});
|
||||
|
||||
test('should reset all player stats to zero', async () => {
|
||||
|
||||
@@ -5,25 +5,31 @@
|
||||
* Regression tests for the issue where tournament_admin users were redirected to login
|
||||
*/
|
||||
|
||||
import { describe, test, expect, vi, beforeEach } from 'vitest';
|
||||
import { describe, test, expect, mock, beforeEach } from 'bun:test';
|
||||
import { canManageTournament, ownsTournament, getManageableTournaments } from '@/lib/permissions';
|
||||
import { getSession } from '@/lib/auth-simple';
|
||||
import { prisma } from '@/lib/prisma';
|
||||
import type { User, Event } from '@prisma/client';
|
||||
|
||||
// Create mock functions first
|
||||
const getSessionMock = mock(() => {});
|
||||
const userFindUniqueMock = mock(() => {});
|
||||
const eventFindUniqueMock = mock(() => {});
|
||||
const eventFindManyMock = mock(() => {});
|
||||
|
||||
// Mock the getSession and prisma functions
|
||||
vi.mock('@/lib/auth-simple', () => ({
|
||||
getSession: vi.fn(),
|
||||
mock.module('@/lib/auth-simple', () => ({
|
||||
getSession: getSessionMock,
|
||||
}));
|
||||
|
||||
vi.mock('@/lib/prisma', () => ({
|
||||
mock.module('@/lib/prisma', () => ({
|
||||
prisma: {
|
||||
user: {
|
||||
findUnique: vi.fn(),
|
||||
findUnique: userFindUniqueMock,
|
||||
},
|
||||
event: {
|
||||
findUnique: vi.fn(),
|
||||
findMany: vi.fn(),
|
||||
findUnique: eventFindUniqueMock,
|
||||
findMany: eventFindManyMock,
|
||||
},
|
||||
},
|
||||
}));
|
||||
@@ -61,16 +67,21 @@ const createMockTournament = (id: number, ownerId: string | null): Event => ({
|
||||
|
||||
describe('Tournament Permissions', () => {
|
||||
beforeEach(() => {
|
||||
vi.clearAllMocks();
|
||||
// Reset mock implementations to default (no-op) before each test
|
||||
// This prevents pollution from previous tests
|
||||
getSessionMock.mockImplementation(() => undefined);
|
||||
userFindUniqueMock.mockImplementation(() => undefined);
|
||||
eventFindUniqueMock.mockImplementation(() => undefined);
|
||||
eventFindManyMock.mockImplementation(() => undefined);
|
||||
});
|
||||
|
||||
describe('canManageTournament', () => {
|
||||
test('should allow club_admin to manage any tournament', async () => {
|
||||
vi.mocked(getSession).mockResolvedValue({
|
||||
getSessionMock.mockImplementation(async () => ({
|
||||
user: { id: 'admin-1', email: 'admin@example.com' },
|
||||
session: { token: 'test', expiresAt: new Date() }
|
||||
});
|
||||
vi.mocked(prisma.user.findUnique).mockResolvedValue(
|
||||
}));
|
||||
userFindUniqueMock.mockImplementation(async () =>
|
||||
createMockUser('admin-1', 'admin@example.com', 'club_admin')
|
||||
);
|
||||
|
||||
@@ -79,14 +90,14 @@ describe('Tournament Permissions', () => {
|
||||
});
|
||||
|
||||
test('should allow tournament_admin to manage their own tournament', async () => {
|
||||
vi.mocked(getSession).mockResolvedValue({
|
||||
getSessionMock.mockImplementation(async () => ({
|
||||
user: { id: 'tour-admin-1', email: 'tour@example.com' },
|
||||
session: { token: 'test', expiresAt: new Date() }
|
||||
});
|
||||
vi.mocked(prisma.user.findUnique).mockResolvedValue(
|
||||
}));
|
||||
userFindUniqueMock.mockImplementation(async () =>
|
||||
createMockUser('tour-admin-1', 'tour@example.com', 'tournament_admin')
|
||||
);
|
||||
vi.mocked(prisma.event.findUnique).mockResolvedValue(
|
||||
eventFindUniqueMock.mockImplementation(async () =>
|
||||
createMockTournament(1, 'tour-admin-1')
|
||||
);
|
||||
|
||||
@@ -95,14 +106,14 @@ describe('Tournament Permissions', () => {
|
||||
});
|
||||
|
||||
test('should deny tournament_admin from managing other users tournaments', async () => {
|
||||
vi.mocked(getSession).mockResolvedValue({
|
||||
getSessionMock.mockImplementation(async () => ({
|
||||
user: { id: 'tour-admin-1', email: 'tour@example.com' },
|
||||
session: { token: 'test', expiresAt: new Date() }
|
||||
});
|
||||
vi.mocked(prisma.user.findUnique).mockResolvedValue(
|
||||
}));
|
||||
userFindUniqueMock.mockImplementation(async () =>
|
||||
createMockUser('tour-admin-1', 'tour@example.com', 'tournament_admin')
|
||||
);
|
||||
vi.mocked(prisma.event.findUnique).mockResolvedValue(
|
||||
eventFindUniqueMock.mockImplementation(async () =>
|
||||
createMockTournament(1, 'other-user-1')
|
||||
);
|
||||
|
||||
@@ -112,11 +123,11 @@ describe('Tournament Permissions', () => {
|
||||
});
|
||||
|
||||
test('should deny player from managing tournaments', async () => {
|
||||
vi.mocked(getSession).mockResolvedValue({
|
||||
getSessionMock.mockImplementation(async () => ({
|
||||
user: { id: 'player-1', email: 'player@example.com' },
|
||||
session: { token: 'test', expiresAt: new Date() }
|
||||
});
|
||||
vi.mocked(prisma.user.findUnique).mockResolvedValue(
|
||||
}));
|
||||
userFindUniqueMock.mockImplementation(async () =>
|
||||
createMockUser('player-1', 'player@example.com', 'player')
|
||||
);
|
||||
|
||||
@@ -126,7 +137,7 @@ describe('Tournament Permissions', () => {
|
||||
});
|
||||
|
||||
test('should deny unauthenticated user', async () => {
|
||||
vi.mocked(getSession).mockResolvedValue(null);
|
||||
getSessionMock.mockImplementation(async () => null);
|
||||
|
||||
const result = await canManageTournament(999);
|
||||
expect(result.allowed).toBe(false);
|
||||
@@ -136,11 +147,11 @@ describe('Tournament Permissions', () => {
|
||||
|
||||
describe('ownsTournament', () => {
|
||||
test('should return true if user owns tournament', async () => {
|
||||
vi.mocked(getSession).mockResolvedValue({
|
||||
getSessionMock.mockImplementation(async () => ({
|
||||
user: { id: 'owner-1', email: 'owner@example.com' },
|
||||
session: { token: 'test', expiresAt: new Date() }
|
||||
});
|
||||
vi.mocked(prisma.event.findUnique).mockResolvedValue(
|
||||
}));
|
||||
eventFindUniqueMock.mockImplementation(async () =>
|
||||
createMockTournament(1, 'owner-1')
|
||||
);
|
||||
|
||||
@@ -149,11 +160,11 @@ describe('Tournament Permissions', () => {
|
||||
});
|
||||
|
||||
test('should return false if user does not own tournament', async () => {
|
||||
vi.mocked(getSession).mockResolvedValue({
|
||||
getSessionMock.mockImplementation(async () => ({
|
||||
user: { id: 'non-owner-1', email: 'nonowner@example.com' },
|
||||
session: { token: 'test', expiresAt: new Date() }
|
||||
});
|
||||
vi.mocked(prisma.event.findUnique).mockResolvedValue(
|
||||
}));
|
||||
eventFindUniqueMock.mockImplementation(async () =>
|
||||
createMockTournament(1, 'owner-1')
|
||||
);
|
||||
|
||||
@@ -165,11 +176,11 @@ describe('Tournament Permissions', () => {
|
||||
|
||||
describe('getManageableTournaments', () => {
|
||||
test('should return all tournaments for club_admin', async () => {
|
||||
vi.mocked(getSession).mockResolvedValue({
|
||||
getSessionMock.mockImplementation(async () => ({
|
||||
user: { id: 'admin-1', email: 'admin@example.com' },
|
||||
session: { token: 'test', expiresAt: new Date() }
|
||||
});
|
||||
vi.mocked(prisma.user.findUnique).mockResolvedValue(
|
||||
}));
|
||||
userFindUniqueMock.mockImplementation(async () =>
|
||||
createMockUser('admin-1', 'admin@example.com', 'club_admin')
|
||||
);
|
||||
|
||||
@@ -178,11 +189,11 @@ describe('Tournament Permissions', () => {
|
||||
createMockTournament(2, 'user-2'),
|
||||
createMockTournament(3, 'user-3'),
|
||||
];
|
||||
vi.mocked(prisma.event.findMany).mockResolvedValue(mockTournaments);
|
||||
eventFindManyMock.mockImplementation(async () => mockTournaments);
|
||||
|
||||
const result = await getManageableTournaments();
|
||||
expect(result).toEqual(mockTournaments);
|
||||
expect(prisma.event.findMany).toHaveBeenCalledWith({
|
||||
expect(eventFindManyMock).toHaveBeenCalledWith({
|
||||
where: { eventType: 'tournament' },
|
||||
include: { participants: true },
|
||||
orderBy: { createdAt: 'desc' }
|
||||
@@ -190,11 +201,11 @@ describe('Tournament Permissions', () => {
|
||||
});
|
||||
|
||||
test('should return only owned tournaments for tournament_admin', async () => {
|
||||
vi.mocked(getSession).mockResolvedValue({
|
||||
getSessionMock.mockImplementation(async () => ({
|
||||
user: { id: 'tour-admin-1', email: 'tour@example.com' },
|
||||
session: { token: 'test', expiresAt: new Date() }
|
||||
});
|
||||
vi.mocked(prisma.user.findUnique).mockResolvedValue(
|
||||
}));
|
||||
userFindUniqueMock.mockImplementation(async () =>
|
||||
createMockUser('tour-admin-1', 'tour@example.com', 'tournament_admin')
|
||||
);
|
||||
|
||||
@@ -202,11 +213,11 @@ describe('Tournament Permissions', () => {
|
||||
createMockTournament(1, 'tour-admin-1'),
|
||||
createMockTournament(2, 'tour-admin-1'),
|
||||
];
|
||||
vi.mocked(prisma.event.findMany).mockResolvedValue(mockTournaments);
|
||||
eventFindManyMock.mockImplementation(async () => mockTournaments);
|
||||
|
||||
const result = await getManageableTournaments();
|
||||
expect(result).toEqual(mockTournaments);
|
||||
expect(prisma.event.findMany).toHaveBeenCalledWith({
|
||||
expect(eventFindManyMock).toHaveBeenCalledWith({
|
||||
where: {
|
||||
eventType: 'tournament',
|
||||
ownerId: 'tour-admin-1'
|
||||
@@ -217,11 +228,11 @@ describe('Tournament Permissions', () => {
|
||||
});
|
||||
|
||||
test('should return only non-draft tournaments for players', async () => {
|
||||
vi.mocked(getSession).mockResolvedValue({
|
||||
getSessionMock.mockImplementation(async () => ({
|
||||
user: { id: 'player-1', email: 'player@example.com' },
|
||||
session: { token: 'test', expiresAt: new Date() }
|
||||
});
|
||||
vi.mocked(prisma.user.findUnique).mockResolvedValue(
|
||||
}));
|
||||
userFindUniqueMock.mockImplementation(async () =>
|
||||
createMockUser('player-1', 'player@example.com', 'player')
|
||||
);
|
||||
|
||||
@@ -229,11 +240,11 @@ describe('Tournament Permissions', () => {
|
||||
createMockTournament(1, 'user-1'),
|
||||
createMockTournament(2, 'user-2'),
|
||||
];
|
||||
vi.mocked(prisma.event.findMany).mockResolvedValue(mockTournaments);
|
||||
eventFindManyMock.mockImplementation(async () => mockTournaments);
|
||||
|
||||
const result = await getManageableTournaments();
|
||||
expect(result).toEqual(mockTournaments);
|
||||
expect(prisma.event.findMany).toHaveBeenCalledWith({
|
||||
expect(eventFindManyMock).toHaveBeenCalledWith({
|
||||
where: {
|
||||
eventType: 'tournament',
|
||||
status: { not: 'draft' }
|
||||
@@ -249,14 +260,14 @@ describe('Tournament Permissions', () => {
|
||||
// This simulates the scenario where a tournament_admin user
|
||||
// clicks "Edit" on a tournament they own
|
||||
|
||||
vi.mocked(getSession).mockResolvedValue({
|
||||
getSessionMock.mockImplementation(async () => ({
|
||||
user: { id: 'tour-admin-1', email: 'tour@example.com' },
|
||||
session: { token: 'test', expiresAt: new Date() }
|
||||
});
|
||||
vi.mocked(prisma.user.findUnique).mockResolvedValue(
|
||||
}));
|
||||
userFindUniqueMock.mockImplementation(async () =>
|
||||
createMockUser('tour-admin-1', 'tour@example.com', 'tournament_admin')
|
||||
);
|
||||
vi.mocked(prisma.event.findUnique).mockResolvedValue(
|
||||
eventFindUniqueMock.mockImplementation(async () =>
|
||||
createMockTournament(1, 'tour-admin-1')
|
||||
);
|
||||
|
||||
@@ -271,11 +282,11 @@ describe('Tournament Permissions', () => {
|
||||
test('club_admin should still be able to manage any tournament', async () => {
|
||||
// This ensures we didn't break the existing club_admin functionality
|
||||
|
||||
vi.mocked(getSession).mockResolvedValue({
|
||||
getSessionMock.mockImplementation(async () => ({
|
||||
user: { id: 'club-admin-1', email: 'club@example.com' },
|
||||
session: { token: 'test', expiresAt: new Date() }
|
||||
});
|
||||
vi.mocked(prisma.user.findUnique).mockResolvedValue(
|
||||
}));
|
||||
userFindUniqueMock.mockImplementation(async () =>
|
||||
createMockUser('club-admin-1', 'club@example.com', 'club_admin')
|
||||
);
|
||||
|
||||
@@ -286,13 +297,16 @@ describe('Tournament Permissions', () => {
|
||||
test('players should still be denied from managing tournaments', async () => {
|
||||
// This ensures we didn't accidentally grant players access
|
||||
|
||||
vi.mocked(getSession).mockResolvedValue({
|
||||
getSessionMock.mockImplementation(async () => ({
|
||||
user: { id: 'player-1', email: 'player@example.com' },
|
||||
session: { token: 'test', expiresAt: new Date() }
|
||||
});
|
||||
vi.mocked(prisma.user.findUnique).mockResolvedValue(
|
||||
}));
|
||||
userFindUniqueMock.mockImplementation(async () =>
|
||||
createMockUser('player-1', 'player@example.com', 'player')
|
||||
);
|
||||
eventFindUniqueMock.mockImplementation(async () =>
|
||||
createMockTournament(1, 'other-user-1')
|
||||
);
|
||||
|
||||
const result = await canManageTournament(1);
|
||||
expect(result.allowed).toBe(false);
|
||||
|
||||
@@ -3,23 +3,33 @@
|
||||
* Tests the allowTies field is properly saved when updating tournaments
|
||||
*/
|
||||
|
||||
import { describe, it, expect, vi, beforeEach } from 'vitest';
|
||||
import { describe, it, expect, mock, beforeEach,} from 'bun:test';
|
||||
import { prisma } from '@/lib/prisma';
|
||||
|
||||
// Create mock functions at module level
|
||||
const eventFindUniqueMock = mock(() => {});
|
||||
const eventUpdateMock = mock(() => {});
|
||||
const canManageTournamentMock = mock(() => {});
|
||||
const canDeleteTournamentMock = mock(() => {});
|
||||
|
||||
// Store default implementations
|
||||
const defaultCanManageTournament = canManageTournamentMock.mockResolvedValue({ allowed: true });
|
||||
const defaultCanDeleteTournament = canDeleteTournamentMock.mockResolvedValue({ allowed: true });
|
||||
|
||||
// Mock the prisma client
|
||||
vi.mock('@/lib/prisma', () => ({
|
||||
mock.module('@/lib/prisma', () => ({
|
||||
prisma: {
|
||||
event: {
|
||||
findUnique: vi.fn(),
|
||||
update: vi.fn(),
|
||||
findUnique: eventFindUniqueMock,
|
||||
update: eventUpdateMock,
|
||||
},
|
||||
},
|
||||
}));
|
||||
|
||||
// Mock the permissions module
|
||||
vi.mock('@/lib/permissions', () => ({
|
||||
canManageTournament: vi.fn().mockResolvedValue({ allowed: true }),
|
||||
canDeleteTournament: vi.fn().mockResolvedValue({ allowed: true }),
|
||||
mock.module('@/lib/permissions', () => ({
|
||||
canManageTournament: defaultCanManageTournament,
|
||||
canDeleteTournament: defaultCanDeleteTournament,
|
||||
}));
|
||||
|
||||
// Import the route handler after mocking
|
||||
@@ -27,12 +37,16 @@ import { PUT } from '@/app/api/tournaments/[id]/route';
|
||||
|
||||
describe('Tournament Update API', () => {
|
||||
beforeEach(() => {
|
||||
vi.clearAllMocks();
|
||||
// Clear all mock history before each test
|
||||
eventFindUniqueMock.mockClear();
|
||||
eventUpdateMock.mockClear();
|
||||
canManageTournamentMock.mockClear();
|
||||
canDeleteTournamentMock.mockClear();
|
||||
});
|
||||
|
||||
it('should update allowTies field when provided', async () => {
|
||||
// Mock existing tournament
|
||||
vi.mocked(prisma.event.findUnique).mockResolvedValue({
|
||||
eventFindUniqueMock.mockImplementation(async () => ({
|
||||
id: 1,
|
||||
name: 'Test Tournament',
|
||||
allowTies: false,
|
||||
@@ -46,10 +60,10 @@ describe('Tournament Update API', () => {
|
||||
ownerId: null,
|
||||
createdAt: new Date(),
|
||||
updatedAt: new Date(),
|
||||
} as any);
|
||||
} as any));
|
||||
|
||||
// Mock successful update
|
||||
vi.mocked(prisma.event.update).mockResolvedValue({
|
||||
eventUpdateMock.mockImplementation(async () => ({
|
||||
id: 1,
|
||||
name: 'Test Tournament',
|
||||
allowTies: true,
|
||||
@@ -63,7 +77,7 @@ describe('Tournament Update API', () => {
|
||||
ownerId: null,
|
||||
createdAt: new Date(),
|
||||
updatedAt: new Date(),
|
||||
} as any);
|
||||
} as any));
|
||||
|
||||
const request = new Request('http://localhost/api/tournaments/1', {
|
||||
method: 'PUT',
|
||||
@@ -78,7 +92,7 @@ describe('Tournament Update API', () => {
|
||||
const response = await PUT(request, { params });
|
||||
|
||||
expect(response.status).toBe(200);
|
||||
expect(vi.mocked(prisma.event.update)).toHaveBeenCalledWith(
|
||||
expect(prisma.event.update).toHaveBeenCalledWith(
|
||||
expect.objectContaining({
|
||||
data: expect.objectContaining({
|
||||
allowTies: true,
|
||||
@@ -89,7 +103,7 @@ describe('Tournament Update API', () => {
|
||||
|
||||
it('should default allowTies to false when not provided', async () => {
|
||||
// Mock existing tournament
|
||||
vi.mocked(prisma.event.findUnique).mockResolvedValue({
|
||||
eventFindUniqueMock.mockImplementation(async () => ({
|
||||
id: 1,
|
||||
name: 'Test Tournament',
|
||||
allowTies: true,
|
||||
@@ -103,10 +117,10 @@ describe('Tournament Update API', () => {
|
||||
ownerId: null,
|
||||
createdAt: new Date(),
|
||||
updatedAt: new Date(),
|
||||
} as any);
|
||||
} as any));
|
||||
|
||||
// Mock successful update
|
||||
vi.mocked(prisma.event.update).mockResolvedValue({
|
||||
eventUpdateMock.mockImplementation(async () => ({
|
||||
id: 1,
|
||||
name: 'Test Tournament',
|
||||
allowTies: false,
|
||||
@@ -120,7 +134,7 @@ describe('Tournament Update API', () => {
|
||||
ownerId: null,
|
||||
createdAt: new Date(),
|
||||
updatedAt: new Date(),
|
||||
} as any);
|
||||
} as any));
|
||||
|
||||
const request = new Request('http://localhost/api/tournaments/1', {
|
||||
method: 'PUT',
|
||||
@@ -135,7 +149,7 @@ describe('Tournament Update API', () => {
|
||||
const response = await PUT(request, { params });
|
||||
|
||||
expect(response.status).toBe(200);
|
||||
expect(vi.mocked(prisma.event.update)).toHaveBeenCalledWith(
|
||||
expect(prisma.event.update).toHaveBeenCalledWith(
|
||||
expect.objectContaining({
|
||||
data: expect.objectContaining({
|
||||
allowTies: false, // Should default to false
|
||||
@@ -146,7 +160,7 @@ describe('Tournament Update API', () => {
|
||||
|
||||
it('should preserve allowTies value when updating other fields', async () => {
|
||||
// Mock existing tournament with allowTies = true
|
||||
vi.mocked(prisma.event.findUnique).mockResolvedValue({
|
||||
eventFindUniqueMock.mockImplementation(async () => ({
|
||||
id: 1,
|
||||
name: 'Test Tournament',
|
||||
allowTies: true,
|
||||
@@ -160,10 +174,10 @@ describe('Tournament Update API', () => {
|
||||
ownerId: null,
|
||||
createdAt: new Date(),
|
||||
updatedAt: new Date(),
|
||||
} as any);
|
||||
} as any));
|
||||
|
||||
// Mock successful update
|
||||
vi.mocked(prisma.event.update).mockResolvedValue({
|
||||
eventUpdateMock.mockImplementation(async () => ({
|
||||
id: 1,
|
||||
name: 'Updated Tournament Name',
|
||||
allowTies: true,
|
||||
@@ -177,7 +191,7 @@ describe('Tournament Update API', () => {
|
||||
ownerId: null,
|
||||
createdAt: new Date(),
|
||||
updatedAt: new Date(),
|
||||
} as any);
|
||||
} as any));
|
||||
|
||||
const request = new Request('http://localhost/api/tournaments/1', {
|
||||
method: 'PUT',
|
||||
@@ -192,7 +206,7 @@ describe('Tournament Update API', () => {
|
||||
const response = await PUT(request, { params });
|
||||
|
||||
expect(response.status).toBe(200);
|
||||
const updateCall = vi.mocked(prisma.event.update).mock.calls[0][0];
|
||||
const updateCall = eventUpdateMock.mock.calls[0][0];
|
||||
expect(updateCall.data.allowTies).toBe(true);
|
||||
expect(updateCall.data.name).toBe('Updated Tournament Name');
|
||||
expect(updateCall.data.targetScore).toBe(10);
|
||||
|
||||
@@ -4,25 +4,31 @@
|
||||
* Tests for user name editing and profile management
|
||||
*/
|
||||
|
||||
import { describe, test, expect, vi, beforeEach } from 'vitest';
|
||||
import { describe, test, expect, mock, beforeEach,} from 'bun:test';
|
||||
import { hasRole } from '@/lib/permissions';
|
||||
import { getSession } from '@/lib/auth-simple';
|
||||
import { prisma } from '@/lib/prisma';
|
||||
import type { User, Player } from '@prisma/client';
|
||||
|
||||
// Create mock functions at module level
|
||||
const getSessionMock = mock(() => {});
|
||||
const userFindUniqueMock = mock(() => {});
|
||||
const userUpdateMock = mock(() => {});
|
||||
const playerFindUniqueMock = mock(() => {});
|
||||
|
||||
// Mock the getSession and prisma functions
|
||||
vi.mock('@/lib/auth-simple', () => ({
|
||||
getSession: vi.fn(),
|
||||
mock.module('@/lib/auth-simple', () => ({
|
||||
getSession: getSessionMock,
|
||||
}));
|
||||
|
||||
vi.mock('@/lib/prisma', () => ({
|
||||
mock.module('@/lib/prisma', () => ({
|
||||
prisma: {
|
||||
user: {
|
||||
findUnique: vi.fn(),
|
||||
update: vi.fn(),
|
||||
findUnique: userFindUniqueMock,
|
||||
update: userUpdateMock,
|
||||
},
|
||||
player: {
|
||||
findUnique: vi.fn(),
|
||||
findUnique: playerFindUniqueMock,
|
||||
},
|
||||
},
|
||||
}));
|
||||
@@ -56,16 +62,20 @@ const createMockPlayer = (id: number, name: string): Player => ({
|
||||
|
||||
describe('User Management', () => {
|
||||
beforeEach(() => {
|
||||
vi.clearAllMocks();
|
||||
// Reset mock implementations to default (no-op) before each test
|
||||
getSessionMock.mockImplementation(() => undefined);
|
||||
userFindUniqueMock.mockImplementation(() => undefined);
|
||||
userUpdateMock.mockImplementation(() => undefined);
|
||||
playerFindUniqueMock.mockImplementation(() => undefined);
|
||||
});
|
||||
|
||||
describe('User Name Editing', () => {
|
||||
test('club_admin should be able to edit any user name', async () => {
|
||||
vi.mocked(getSession).mockResolvedValue({
|
||||
getSessionMock.mockImplementation(async () => ({
|
||||
user: { id: 'admin-1', email: 'admin@example.com' },
|
||||
session: { token: 'test', expiresAt: new Date() }
|
||||
});
|
||||
vi.mocked(prisma.user.findUnique).mockResolvedValue(
|
||||
}));
|
||||
userFindUniqueMock.mockImplementation(async () =>
|
||||
createMockUser('admin-1', 'admin@example.com', 'club_admin')
|
||||
);
|
||||
|
||||
@@ -74,11 +84,11 @@ describe('User Management', () => {
|
||||
});
|
||||
|
||||
test('tournament_admin should NOT be able to edit user names', async () => {
|
||||
vi.mocked(getSession).mockResolvedValue({
|
||||
user: { id: 'tour-admin-1', email: 'tour@example.com' },
|
||||
getSessionMock.mockImplementation(async () => ({
|
||||
user: { id: 'admin-1', email: 'admin@example.com' },
|
||||
session: { token: 'test', expiresAt: new Date() }
|
||||
});
|
||||
vi.mocked(prisma.user.findUnique).mockResolvedValue(
|
||||
}));
|
||||
userFindUniqueMock.mockImplementation(async () =>
|
||||
createMockUser('tour-admin-1', 'tour@example.com', 'tournament_admin')
|
||||
);
|
||||
|
||||
@@ -87,11 +97,11 @@ describe('User Management', () => {
|
||||
});
|
||||
|
||||
test('player should NOT be able to edit user names', async () => {
|
||||
vi.mocked(getSession).mockResolvedValue({
|
||||
user: { id: 'player-1', email: 'player@example.com' },
|
||||
getSessionMock.mockImplementation(async () => ({
|
||||
user: { id: 'admin-1', email: 'admin@example.com' },
|
||||
session: { token: 'test', expiresAt: new Date() }
|
||||
});
|
||||
vi.mocked(prisma.user.findUnique).mockResolvedValue(
|
||||
}));
|
||||
userFindUniqueMock.mockImplementation(async () =>
|
||||
createMockUser('player-1', 'player@example.com', 'player')
|
||||
);
|
||||
|
||||
@@ -100,7 +110,7 @@ describe('User Management', () => {
|
||||
});
|
||||
|
||||
test('unauthenticated user should NOT be able to edit user names', async () => {
|
||||
vi.mocked(getSession).mockResolvedValue(null);
|
||||
getSessionMock.mockImplementation(async () => null);
|
||||
|
||||
const result = await hasRole('club_admin');
|
||||
expect(result.allowed).toBe(false);
|
||||
@@ -111,11 +121,11 @@ describe('User Management', () => {
|
||||
test('user should be able to view their own profile', async () => {
|
||||
const mockUser = createMockUser('user-1', 'user@example.com', 'player');
|
||||
|
||||
vi.mocked(getSession).mockResolvedValue({
|
||||
user: { id: 'user-1', email: 'user@example.com' },
|
||||
getSessionMock.mockImplementation(async () => ({
|
||||
user: { id: 'admin-1', email: 'admin@example.com' },
|
||||
session: { token: 'test', expiresAt: new Date() }
|
||||
});
|
||||
vi.mocked(prisma.user.findUnique).mockResolvedValue(mockUser);
|
||||
}));
|
||||
userFindUniqueMock.mockImplementation(async () => mockUser);
|
||||
|
||||
// In the actual implementation, this would check if session.user.id === params.id
|
||||
const canViewOwnProfile = true; // This logic is in the API route
|
||||
@@ -126,11 +136,11 @@ describe('User Management', () => {
|
||||
const mockAdmin = createMockUser('admin-1', 'admin@example.com', 'club_admin');
|
||||
const mockUser = createMockUser('user-1', 'user@example.com', 'player');
|
||||
|
||||
vi.mocked(getSession).mockResolvedValue({
|
||||
getSessionMock.mockImplementation(async () => ({
|
||||
user: { id: 'admin-1', email: 'admin@example.com' },
|
||||
session: { token: 'test', expiresAt: new Date() }
|
||||
});
|
||||
vi.mocked(prisma.user.findUnique)
|
||||
}));
|
||||
(userFindUniqueMock)
|
||||
.mockResolvedValueOnce(mockAdmin) // For the requesting user
|
||||
.mockResolvedValueOnce(mockUser); // For the target user
|
||||
|
||||
@@ -142,11 +152,11 @@ describe('User Management', () => {
|
||||
test('non-admin should NOT be able to view other user profiles', async () => {
|
||||
const mockUser = createMockUser('user-1', 'user@example.com', 'player');
|
||||
|
||||
vi.mocked(getSession).mockResolvedValue({
|
||||
user: { id: 'user-1', email: 'user@example.com' },
|
||||
getSessionMock.mockImplementation(async () => ({
|
||||
user: { id: 'admin-1', email: 'admin@example.com' },
|
||||
session: { token: 'test', expiresAt: new Date() }
|
||||
});
|
||||
vi.mocked(prisma.user.findUnique).mockResolvedValue(mockUser);
|
||||
}));
|
||||
userFindUniqueMock.mockImplementation(async () => mockUser);
|
||||
|
||||
// In the actual implementation, this would check if session.user.id === params.id
|
||||
const canViewOtherProfile = false; // This logic is in the API route
|
||||
@@ -159,11 +169,11 @@ describe('User Management', () => {
|
||||
const mockUser = createMockUser('user-1', 'user@example.com', 'club_admin');
|
||||
const mockPlayer = createMockPlayer(1, 'Old Name');
|
||||
|
||||
vi.mocked(getSession).mockResolvedValue({
|
||||
user: { id: 'user-1', email: 'user@example.com' },
|
||||
getSessionMock.mockImplementation(async () => ({
|
||||
user: { id: 'admin-1', email: 'admin@example.com' },
|
||||
session: { token: 'test', expiresAt: new Date() }
|
||||
});
|
||||
vi.mocked(prisma.user.findUnique).mockResolvedValue(mockUser);
|
||||
}));
|
||||
userFindUniqueMock.mockImplementation(async () => mockUser);
|
||||
|
||||
const updatedUser = {
|
||||
...mockUser,
|
||||
@@ -171,7 +181,7 @@ describe('User Management', () => {
|
||||
player: { ...mockPlayer, name: 'New Name', normalizedName: 'new name' }
|
||||
};
|
||||
|
||||
vi.mocked(prisma.user.update).mockResolvedValue(updatedUser);
|
||||
userUpdateMock.mockImplementation(async () => updatedUser);
|
||||
|
||||
// The API route should update both user.name and player.name
|
||||
expect(updatedUser.name).toBe('New Name');
|
||||
|
||||
@@ -14,6 +14,8 @@ export const auth = betterAuth({
|
||||
enabled: true,
|
||||
autoSignIn: true, // Automatically sign in after registration
|
||||
requireEmailVerification: false, // Don't require email verification for tests
|
||||
minPasswordLength: 8, // Set minimum password length
|
||||
maxPasswordLength: 128, // Set maximum password length
|
||||
},
|
||||
secret: process.env.BETTER_AUTH_SECRET || process.env.NEXTAUTH_SECRET,
|
||||
baseURL: process.env.BETTER_AUTH_URL || process.env.NEXTAUTH_URL || "http://localhost:3000",
|
||||
|
||||
+11
-4
@@ -1,8 +1,7 @@
|
||||
import { PrismaClient } from '@prisma/client'
|
||||
import dotenv from 'dotenv'
|
||||
|
||||
// Load .env file if it exists
|
||||
dotenv.config()
|
||||
require('dotenv').config()
|
||||
|
||||
const globalForPrisma = globalThis as unknown as {
|
||||
prisma: PrismaClient | undefined
|
||||
@@ -10,16 +9,24 @@ const globalForPrisma = globalThis as unknown as {
|
||||
|
||||
// Detect database provider from environment (default to sqlite for local development)
|
||||
const databaseProvider = process.env.DATABASE_PROVIDER || 'sqlite'
|
||||
const databaseUrl = process.env.DATABASE_URL
|
||||
|
||||
// Create PrismaClient with appropriate adapter
|
||||
const createPrismaClient = () => {
|
||||
let client: PrismaClient
|
||||
|
||||
if (databaseProvider === 'postgresql') {
|
||||
// Validate DATABASE_URL is present for PostgreSQL
|
||||
if (!databaseUrl) {
|
||||
throw new Error(
|
||||
'DATABASE_URL environment variable is required when DATABASE_PROVIDER is set to postgresql. ' +
|
||||
'Current DATABASE_PROVIDER: ' + databaseProvider
|
||||
)
|
||||
}
|
||||
|
||||
// Use PrismaPg adapter for PostgreSQL
|
||||
const { PrismaPg } = require('@prisma/adapter-pg')
|
||||
const pg = require('pg')
|
||||
const adapter = new PrismaPg({ connectionString: process.env.DATABASE_URL })
|
||||
const adapter = new PrismaPg({ connectionString: databaseUrl })
|
||||
client = new PrismaClient({ adapter })
|
||||
} else {
|
||||
// No adapter needed for SQLite
|
||||
|
||||
Reference in New Issue
Block a user