david
bb6be245b7
## Summary This PR implements the tournament schedule tab functionality and fixes all remaining E2E test failures. ### Changes Included 1. **Tournament Schedule Feature** - Added tournament schedule page at `/admin/tournaments/[id]/schedule` - Implemented "Generate Schedule" button functionality - Added schedule generation logic for round-robin tournaments 2. **E2E Test Fixes** - Fixed database connection issues in production builds - Improved test reliability with better error handling and debugging - Updated test infrastructure to use environment variables instead of hardcoded values 3. **CI/CD Updates** - Added E2E test job to PR workflow - Configured tests to run against development database - Moved database password to Gitea secrets 4. **Code Quality** - Removed hardcoded passwords from codebase - Improved Prisma client configuration - Enhanced authentication and navigation components ### Test Results All 16 E2E test scenarios are now passing: - Authentication tests: ✅ - Registration tests: ✅ - Tournament schedule tests: ✅ - Player schedule tests: ✅ - Admin navigation tests: ✅ ### Database Configuration - Tests run against `euchre_camp_dev` database - Production builds use environment variables for database configuration - Database password stored in Gitea secrets as `DB_PASSWORD` ### CI Pipeline The PR workflow now includes: 1. Unit tests 2. E2E tests (using production build) 3. Version bump analysis E2E tests must pass before PR can be merged. Reviewed-on: #27 Co-authored-by: David Gwilliam <dhgwilliam@gmail.com> Co-committed-by: David Gwilliam <dhgwilliam@gmail.com>
118 lines
3.3 KiB
YAML
118 lines
3.3 KiB
YAML
name: Pull Request
|
|
|
|
on:
|
|
pull_request:
|
|
branches:
|
|
- main
|
|
|
|
jobs:
|
|
unit-tests:
|
|
runs-on: ubuntu-latest
|
|
container:
|
|
image: docker.notsosm.art/euchre-camp/ci-base:latest
|
|
options: --user root
|
|
|
|
steps:
|
|
- name: Checkout code
|
|
uses: actions/checkout@v4
|
|
|
|
- name: Install dependencies
|
|
run: bun install
|
|
|
|
- name: Generate Prisma client
|
|
run: bun x prisma generate
|
|
env:
|
|
DATABASE_URL: postgresql://user:pass@localhost:5432/dummy
|
|
|
|
- name: Run unit tests
|
|
run: bun test src/__tests__/unit/ src/__tests__/*.test.tsx src/__tests__/auth-simple.test.ts
|
|
|
|
e2e-tests:
|
|
runs-on: ubuntu-latest
|
|
needs: unit-tests
|
|
container:
|
|
image: docker.notsosm.art/euchre-camp/ci-base:latest
|
|
options: --user root
|
|
|
|
steps:
|
|
- name: Checkout code
|
|
uses: actions/checkout@v4
|
|
|
|
- name: Install dependencies
|
|
run: bun install
|
|
|
|
- name: Generate Prisma client
|
|
run: bun x prisma generate
|
|
env:
|
|
DATABASE_URL: postgresql://user:pass@localhost:5432/dummy
|
|
|
|
- name: Run E2E tests
|
|
run: npm run test:acceptance:cucumber:prod
|
|
env:
|
|
DATABASE_URL: postgresql://euchre_camp:${{ secrets.DB_PASSWORD }}@dhg.lol:5432/euchre_camp_dev
|
|
DATABASE_PROVIDER: postgresql
|
|
|
|
analyze-bump-type:
|
|
runs-on: ubuntu-latest
|
|
needs: e2e-tests
|
|
|
|
steps:
|
|
- name: Checkout code
|
|
uses: actions/checkout@v4
|
|
with:
|
|
fetch-depth: 0
|
|
|
|
- name: Analyze commits for bump type
|
|
id: bump_type
|
|
run: |
|
|
# Get the merge base and PR commits
|
|
git fetch origin main:main 2>/dev/null || true
|
|
MERGE_BASE=$(git merge-base HEAD main 2>/dev/null || echo "HEAD~1")
|
|
|
|
# Analyze commits in this PR
|
|
COMMITS=$(git log --oneline ${MERGE_BASE}..HEAD 2>/dev/null || git log --oneline -10)
|
|
|
|
echo "Commits in this PR:"
|
|
echo "$COMMITS"
|
|
echo ""
|
|
|
|
# Determine bump type
|
|
if echo "$COMMITS" | grep -qE "(BREAKING CHANGE|!:)"; then
|
|
BUMP="major"
|
|
REASON="Breaking change detected in commit messages"
|
|
elif echo "$COMMITS" | grep -qE "^feat"; then
|
|
BUMP="minor"
|
|
REASON="Feature commits detected"
|
|
else
|
|
BUMP="patch"
|
|
REASON="Defaulting to patch (fixes or other changes)"
|
|
fi
|
|
|
|
echo "Suggested bump type: $BUMP"
|
|
echo "Reason: $REASON"
|
|
echo "bump=$BUMP" >> $GITHUB_OUTPUT
|
|
echo "reason=$REASON" >> $GITHUB_OUTPUT
|
|
|
|
- name: Comment bump type on PR
|
|
uses: actions/github-script@v7
|
|
with:
|
|
script: |
|
|
const bump = '${{ steps.bump_type.outputs.bump }}';
|
|
const reason = '${{ steps.bump_type.outputs.reason }}';
|
|
|
|
const comment = `
|
|
## 🏷️ Version Bump Analysis
|
|
|
|
**Suggested bump type:** \`${bump.toUpperCase()}\`
|
|
**Reason:** ${reason}
|
|
|
|
This PR will bump the version to the next ${bump} version when merged.
|
|
`;
|
|
|
|
github.rest.issues.createComment({
|
|
issue_number: context.issue.number,
|
|
owner: context.repo.owner,
|
|
repo: context.repo.repo,
|
|
body: comment
|
|
});
|