Merge branch 'bugfix/9-player-schedule-tests': Player schedule clickable matches

e2e/cucumber/features/player-schedule.feature

@@ -9,7 +9,7 @@ Feature: Player Schedule
     When I go to my schedule page
     Then I should see "No upcoming matches"
 
-  @happy-path @player-features @issue-9 @wip
+  @happy-path @player-features @issue-9
   Scenario: Player views schedule with upcoming matches
     Given I am logged in as a player
     And I have upcoming matches in my schedule

e2e/cucumber/step-definitions/auth-steps.ts

@@ -337,16 +337,67 @@ When('I go to my schedule page', async function () {
 });
 
 Given('I have upcoming matches in my schedule', async function () {
-  console.log('🌍 Note: This step requires database setup via API or UI');
+  console.log('🌍 Setting up upcoming matches in schedule');
-  console.log('🌍 For acceptance tests, this would be set up before running the test');
+  const prisma = await world.getPrisma();
-  // For true acceptance testing, we would:
+  const timestamp = Date.now();
-  // 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
+  // Get the current player
-  // In a real test run, this data would already exist in the dev database
+  if (!world.playerId) {
+    throw new Error('No player ID found. Make sure user is logged in as a player first.');
+  }
+  
+  const currentPlayerId = parseInt(world.playerId, 10);
+  
+  // Create 3 other players for the match
+  const opponent1 = await prisma.player.create({
+    data: {
+      name: `Opponent ${timestamp} 1`,
+      normalizedName: `opponent ${timestamp} 1`,
+      currentElo: 1000,
+    },
+  });
+  
+  const opponent2 = await prisma.player.create({
+    data: {
+      name: `Opponent ${timestamp} 2`,
+      normalizedName: `opponent ${timestamp} 2`,
+      currentElo: 1000,
+    },
+  });
+  
+  const partner1 = await prisma.player.create({
+    data: {
+      name: `Partner ${timestamp}`,
+      normalizedName: `partner ${timestamp}`,
+      currentElo: 1000,
+    },
+  });
+  
+  // Create a tournament
+  const tournament = await prisma.event.create({
+    data: {
+      name: `Test Schedule Tournament ${timestamp}`,
+      eventDate: new Date(Date.now() + 86400000), // Tomorrow
+      status: 'planned',
+    },
+  });
+  
+  // Create a match with the current player as player1P1 (played tomorrow)
+  await prisma.match.create({
+    data: {
+      eventId: tournament.id,
+      player1P1Id: currentPlayerId,
+      player1P2Id: partner1.id,
+      player2P1Id: opponent1.id,
+      player2P2Id: opponent2.id,
+      team1Score: 10,
+      team2Score: 5,
+      status: 'completed',
+      playedAt: new Date(Date.now() + 86400000), // Tomorrow
+    },
+  });
+  
+  console.log(`🌍 Created tournament "${tournament.name}" with 1 match for player ${currentPlayerId}`);
 });
 
 /**

e2e/cucumber/step-definitions/common-steps.ts

@@ -595,3 +595,50 @@ Then('I should see the rankings table', async function () {
   await expect(world.page.locator('table')).toBeVisible();
   console.log('🌍 Verified rankings table is visible');
 });
+
+// Player Schedule Steps
+Then('I should see the match date', async function () {
+  // Check for a date-like pattern on the page
+  const content = await world.page.content();
+  const hasDate = content.match(/\d{1,2}\/\d{1,2}\/\d{4}/) || content.match(/\w+ \d{1,2}, \d{4}/);
+  expect(hasDate).toBeTruthy();
+  console.log('🌍 Verified match date is visible');
+});
+
+Then('I should see my opponent\'s name', async function () {
+  // Check for opponent text on the page
+  const content = await world.page.content();
+  const hasOpponent = content.includes('Opponent');
+  expect(hasOpponent).toBe(true);
+  console.log('🌍 Verified opponent name is visible');
+});
+
+Then('I should see my partner\'s name', async function () {
+  // Check for partner text on the page
+  const content = await world.page.content();
+  const hasPartner = content.includes('Partner');
+  expect(hasPartner).toBe(true);
+  console.log('🌍 Verified partner name is visible');
+});
+
+Then('I should see the tournament name', async function () {
+  // Check for tournament name on the page
+  const content = await world.page.content();
+  const hasTournament = content.includes('Test Schedule Tournament');
+  expect(hasTournament).toBe(true);
+  console.log('🌍 Verified tournament name is visible');
+});
+
+When('I click on a match', async function () {
+  // Click on the first match link
+  const matchLink = world.page.locator('a[href*="/matches/"]').first();
+  await matchLink.click();
+  await world.page.waitForLoadState('domcontentloaded');
+  console.log('🌍 Clicked on match');
+});
+
+Then('I should be on the match detail page', async function () {
+  const currentUrl = world.page.url();
+  console.log(`🌍 Checking current URL: ${currentUrl}`);
+  expect(currentUrl).toMatch(/\/matches\/\d+/);
+});

src/app/players/[id]/schedule/page.tsx

@@ -128,9 +128,10 @@ export default async function PlayerSchedulePage({ params }: PageProps) {
                   ].filter(Boolean).join(" + ")
 
                   return (
-                    <div
+                    <Link
+                      href={`/matches/${match.id}`}
                       key={match.id}
-                      className="border border-gray-200 rounded-lg p-4 hover:bg-gray-50"
+                      className="block border border-gray-200 rounded-lg p-4 hover:bg-gray-50 cursor-pointer"
                     >
                       <div className="flex justify-between items-center">
                         <div className="flex-1">
@@ -147,7 +148,7 @@ export default async function PlayerSchedulePage({ params }: PageProps) {
                           </span>
                         </div>
                       </div>
-                    </div>
+                    </Link>
                   )
                 })}
               </div>