Create comprehensive manual acceptance test plan for prep-nevr branch #99

Copilot · 2025-09-02T23:01:26Z

This PR creates a detailed manual acceptance test plan for the prep-nevr branch to ensure all features and changes meet requirements before merging into main.

Documentation Added

1. Manual Acceptance Test Plan (`docs/MANUAL_ACCEPTANCE_TEST_PLAN.md`)

A comprehensive 26KB document providing:

Objectives and Scope: Clear definition of manual testing goals covering Discord integration, match data journaling, backend refactoring, EVR game server features, and link tickets functionality
Environment Setup: Step-by-step instructions for setting up PostgreSQL, Redis, MongoDB, and Discord test environments
Test Data Requirements: Specifications for test users, match configurations, and Discord server setup
8 Detailed Test Cases: Complete test procedures covering:
- System startup and health verification
- Discord linked roles integration
- Match data journal and EventDispatcher functionality
- Backend module refactoring and match presence handling
- EVR game server registration and matchmaking
- Link tickets functionality
- Console and API integration
- End-to-end user journey validation
Acceptance Criteria: Functional, performance, quality, and integration requirements
Timeline and Responsibilities: 10-day execution plan with milestones
Risk Assessment: Identified risks with mitigation strategies

2. Quick Reference Checklist (`docs/ACCEPTANCE_TESTING_CHECKLIST.md`)

A condensed 5.6KB checklist for rapid validation including:

Pre-testing setup verification
Core feature testing checkboxes
Performance and error handling validation
Final sign-off criteria
Quick command reference for common operations

Features Covered

The test plan validates all major prep-nevr branch enhancements:

Discord Integration: OAuth flow, role synchronization, user authentication
Match Data Persistence: EventDispatcher integration with MongoDB, Redis queue processing
Backend Refactoring: Match presence handling, service modularity improvements
EVR Game Server Support: Registration, matchmaking algorithms, lobby management
Link Tickets: User account linking and validation system
Infrastructure: Docker Compose orchestration, database migrations

Testing Approach

The plan emphasizes manual, human-driven validation to complement automated testing, ensuring:

Real-world user scenarios are properly validated
Integration points work correctly across all services
Discord API interactions function as expected
Performance meets acceptance criteria under realistic conditions
Error handling and edge cases are properly addressed

This comprehensive testing framework provides everything needed to thoroughly validate the prep-nevr branch before production deployment.

Fixes #98.

💬 Share your feedback on Copilot coding agent for the chance to win a $200 gift card! Click here to start the survey.

…mments

… for improved dependency management

* feat: Add file-level options and swagger definitions to proto files * Update evr.proto Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com> Signed-off-by: Andrew Bates <a@sprock.io> * Update evr.proto Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com> Signed-off-by: Andrew Bates <a@sprock.io> * Update evr_http.proto Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com> Signed-off-by: Andrew Bates <a@sprock.io> --------- Signed-off-by: Andrew Bates <a@sprock.io> Co-authored-by: google-labs-jules[bot] <161369871+google-labs-jules[bot]@users.noreply.github.com> Co-authored-by: Andrew Bates <a@sprock.io> Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>

#25) * Initial plan * Add comprehensive characterization tests for LoginHistory struct Co-authored-by: thesprockee <962164+thesprockee@users.noreply.github.com> * Implement focused improvements to LoginHistory struct methods Co-authored-by: thesprockee <962164+thesprockee@users.noreply.github.com> * Apply suggestion from @Copilot Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com> Signed-off-by: Andrew Bates <a@sprock.io> * Update protobuf field names --------- Signed-off-by: Andrew Bates <a@sprock.io> Co-authored-by: copilot-swe-agent[bot] <198982749+Copilot@users.noreply.github.com> Co-authored-by: thesprockee <962164+thesprockee@users.noreply.github.com> Co-authored-by: Andrew Bates <a@sprock.io> Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>

…oles command (#50) * Initial plan * Implement interactive role configuration interface for /set-roles command Co-authored-by: thesprockee <962164+thesprockee@users.noreply.github.com> * Add unit tests for role configuration functionality Co-authored-by: thesprockee <962164+thesprockee@users.noreply.github.com> * Apply suggestion from @Copilot Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com> Signed-off-by: Andrew Bates <a@sprock.io> * Apply suggestion from @Copilot Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com> Signed-off-by: Andrew Bates <a@sprock.io> * Apply suggestion from @Copilot Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com> Signed-off-by: Andrew Bates <a@sprock.io> --------- Signed-off-by: Andrew Bates <a@sprock.io> Co-authored-by: copilot-swe-agent[bot] <198982749+Copilot@users.noreply.github.com> Co-authored-by: Andrew Bates <a@sprock.io> Co-authored-by: thesprockee <962164+thesprockee@users.noreply.github.com> Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>

…hoTools Nakama (#65) * Initial plan * Complete README.md with comprehensive EchoTools Nakama deployment guide Co-authored-by: thesprockee <962164+thesprockee@users.noreply.github.com> * Update README.md Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com> Signed-off-by: Andrew Bates <a@sprock.io> * Update README.md Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com> Signed-off-by: Andrew Bates <a@sprock.io> * Update README.md Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com> Signed-off-by: Andrew Bates <a@sprock.io> * Update README.md Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com> Signed-off-by: Andrew Bates <a@sprock.io> --------- Signed-off-by: Andrew Bates <a@sprock.io> Co-authored-by: copilot-swe-agent[bot] <198982749+Copilot@users.noreply.github.com> Co-authored-by: thesprockee <962164+thesprockee@users.noreply.github.com> Co-authored-by: Andrew Bates <a@sprock.io> Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>

…treamModeLobbySessionTelemetry API (#67) * Initial plan * Implement Redis Streams Event Journaling, MongoDB Summarization, and StreamModeLobbySessionTelemetry API Co-authored-by: thesprockee <962164+thesprockee@users.noreply.github.com> * Add telemetry integration utilities and documentation Co-authored-by: thesprockee <962164+thesprockee@users.noreply.github.com> * Update server/evr_telemetry_manager.go Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com> Signed-off-by: Andrew Bates <a@sprock.io> * Update server/evr_telemetry_integration.go Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com> Signed-off-by: Andrew Bates <a@sprock.io> * Update server/evr_telemetry_test.go Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com> Signed-off-by: Andrew Bates <a@sprock.io> --------- Signed-off-by: Andrew Bates <a@sprock.io> Co-authored-by: copilot-swe-agent[bot] <198982749+Copilot@users.noreply.github.com> Co-authored-by: thesprockee <962164+thesprockee@users.noreply.github.com> Co-authored-by: Andrew Bates <a@sprock.io> Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>

… match data in MongoDB (#42) * Initial plan * Implement EventMatchDataJournal integration with Redis queue and MongoDB persistence Co-authored-by: thesprockee <962164+thesprockee@users.noreply.github.com> * Apply gofmt formatting to new files * Add demo script and finalize MatchDataJournal integration Co-authored-by: thesprockee <962164+thesprockee@users.noreply.github.com> * Update GitHub Actions workflow to reduce unnecessary builds * Enforcement Suspension System Refactor (#47) * Remove obsolete ServiceConfig * Refactor enforcement records and suspension handling - Introduced AllModes and PrivateModes constants for better mode management. - Updated GuildEnforcementRecord to use Expiry instead of SuspensionExpiry. - Enhanced CheckEnforcementSuspensions to return active suspensions by game mode. - Adjusted lobby authorization to handle game mode-specific suspensions. - Removed LimitedAccess role checks and related logic from GuildGroup. - Updated tests to verify correct behavior of group ID and user ID assignments in enforcement records. * Refactor GitHub Actions workflows to streamline build process (#48) * Refactor GitHub Actions workflows to streamline build process * Enforcement Suspension System Refactor (#47) * Remove obsolete ServiceConfig * Refactor enforcement records and suspension handling - Introduced AllModes and PrivateModes constants for better mode management. - Updated GuildEnforcementRecord to use Expiry instead of SuspensionExpiry. - Enhanced CheckEnforcementSuspensions to return active suspensions by game mode. - Adjusted lobby authorization to handle game mode-specific suspensions. - Removed LimitedAccess role checks and related logic from GuildGroup. - Updated tests to verify correct behavior of group ID and user ID assignments in enforcement records. * Update .github/workflows/build.yaml Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com> Signed-off-by: Andrew Bates <a@sprock.io> * Update build workflow: add Go module download step and change build command to 'make nakama' * Update Docker and Makefile configurations for improved build process and caching * Refactor build workflow: remove Go module download step and integrate 'go mod vendor' before building binary * Disable pull request trigger for tests workflow and update event configuration * Enforcement Suspension System Refactor (#47) * Remove obsolete ServiceConfig * Refactor enforcement records and suspension handling - Introduced AllModes and PrivateModes constants for better mode management. - Updated GuildEnforcementRecord to use Expiry instead of SuspensionExpiry. - Enhanced CheckEnforcementSuspensions to return active suspensions by game mode. - Adjusted lobby authorization to handle game mode-specific suspensions. - Removed LimitedAccess role checks and related logic from GuildGroup. - Updated tests to verify correct behavior of group ID and user ID assignments in enforcement records. --------- Signed-off-by: Andrew Bates <a@sprock.io> Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com> * Refactor storage system to use simplified StorableAdapter pattern (#46) * Initial plan * Create StorableAdapter pattern and convert LatencyHistory Co-authored-by: thesprockee <962164+thesprockee@users.noreply.github.com> * Convert GuildGroupState and LoginHistory to StorableAdapter pattern Co-authored-by: thesprockee <962164+thesprockee@users.noreply.github.com> * Convert DisplayNameHistory and EarlyQuitConfig to StorableAdapter pattern Co-authored-by: thesprockee <962164+thesprockee@users.noreply.github.com> * Convert remaining types to StorableAdapter pattern Co-authored-by: thesprockee <962164+thesprockee@users.noreply.github.com> * Complete Storable interface refactoring with legacy compatibility Co-authored-by: thesprockee <962164+thesprockee@users.noreply.github.com> * Replace Storable with initial proposed implementation * Remove deprecated legacy storage interfaces and functions in favor of StorableAdapter * Refactor latency history handling to remove StorableAdapter usage * Refactor storage index registration to use StorableIndexer pattern * Refactor most structs to use simplified StorableAdapter pattern Co-authored-by: thesprockee <962164+thesprockee@users.noreply.github.com> * Enforcement Suspension System Refactor (#47) * Remove obsolete ServiceConfig * Refactor enforcement records and suspension handling - Introduced AllModes and PrivateModes constants for better mode management. - Updated GuildEnforcementRecord to use Expiry instead of SuspensionExpiry. - Enhanced CheckEnforcementSuspensions to return active suspensions by game mode. - Adjusted lobby authorization to handle game mode-specific suspensions. - Removed LimitedAccess role checks and related logic from GuildGroup. - Updated tests to verify correct behavior of group ID and user ID assignments in enforcement records. * Complete StorableAdapter refactoring to simplified pattern Co-authored-by: thesprockee <962164+thesprockee@users.noreply.github.com> * Enforcement Suspension System Refactor (#47) * Remove obsolete ServiceConfig * Refactor enforcement records and suspension handling - Introduced AllModes and PrivateModes constants for better mode management. - Updated GuildEnforcementRecord to use Expiry instead of SuspensionExpiry. - Enhanced CheckEnforcementSuspensions to return active suspensions by game mode. - Adjusted lobby authorization to handle game mode-specific suspensions. - Removed LimitedAccess role checks and related logic from GuildGroup. - Updated tests to verify correct behavior of group ID and user ID assignments in enforcement records. --------- Co-authored-by: copilot-swe-agent[bot] <198982749+Copilot@users.noreply.github.com> Co-authored-by: thesprockee <962164+thesprockee@users.noreply.github.com> Co-authored-by: Andrew Bates <a@sprock.io> * Refactor profile and display name handling (#49) * WIP: Refactor profile * Refactor profile handling to improve display name management and streamline related structures * Fix display name history caching by limiting to 15 recent names per group and filtering by age * Add kick-player full audit log message * Update interaction signature formatting and log message structure for improved clarity * Refactor storage index registration to include DisplayNameHistory and DeveloperApplications * Update server/evr_account_settings.go Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com> Signed-off-by: Andrew Bates <a@sprock.io> * Update server/evr_account_displayname_history.go Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com> Signed-off-by: Andrew Bates <a@sprock.io> * Refactor display name assignment in DisplayNameHistory and clean up unused code in ProfileOptions and LatencyHistory --------- Signed-off-by: Andrew Bates <a@sprock.io> Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com> * Potential crash fix for evr_discord_appbot (#54) * Add check for `GamePauseSettings` being nil. --------- Signed-off-by: Andrew Bates <a@sprock.io> * Update .gitignore to include 'nakama-debug' and remove unnecessary comments * Optimize Dockerfile by adding 'go mod tidy' and 'go mod vendor' steps for improved dependency management * Refactor EVRProfile structure and remove ProfileOptions * Update Go version to 1.25 * Remove unused pprof server setup from EvrPipeline * Update server/evr_runtime_events.go Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com> Signed-off-by: Andrew Bates <a@sprock.io> * Update server/evr_runtime_events.go Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com> Signed-off-by: Andrew Bates <a@sprock.io> * Update server/evr_runtime_event_matchdatajournal.go Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com> Signed-off-by: Andrew Bates <a@sprock.io> * Update server/evr_match_data_indexing.go Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com> Signed-off-by: Andrew Bates <a@sprock.io> * Update server/evr_runtime_event_matchdatajournal.go Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com> Signed-off-by: Andrew Bates <a@sprock.io> --------- Signed-off-by: Andrew Bates <a@sprock.io> Co-authored-by: copilot-swe-agent[bot] <198982749+Copilot@users.noreply.github.com> Co-authored-by: thesprockee <962164+thesprockee@users.noreply.github.com> Co-authored-by: Andrew Bates <a@sprock.io> Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com> Co-authored-by: Avery ❤️ <avery442@proton.me>

…ng (#62) * Initial plan * Implement Discord sessions channel with automated lobby session tracking Co-authored-by: thesprockee <962164+thesprockee@users.noreply.github.com> * Complete Discord sessions channel implementation with cleanup and service guild support Co-authored-by: thesprockee <962164+thesprockee@users.noreply.github.com> * Apply suggestion from @Copilot Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com> Signed-off-by: Andrew Bates <a@sprock.io> * Update server/evr_discord_sessions_channel.go Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com> Signed-off-by: Andrew Bates <a@sprock.io> * Update server/evr_discord_sessions_channel.go Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com> Signed-off-by: Andrew Bates <a@sprock.io> --------- Signed-off-by: Andrew Bates <a@sprock.io> Co-authored-by: copilot-swe-agent[bot] <198982749+Copilot@users.noreply.github.com> Co-authored-by: thesprockee <962164+thesprockee@users.noreply.github.com> Co-authored-by: Andrew Bates <a@sprock.io> Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>

* Initial plan * Implement Discord Linked Roles with custom requirement options Co-authored-by: thesprockee <962164+thesprockee@users.noreply.github.com> * Complete Discord Linked Roles implementation with CORS support and validation guide Co-authored-by: thesprockee <962164+thesprockee@users.noreply.github.com> * Address review feedback: move guide to docs, fix User-Agent, remove gitignore entry Co-authored-by: thesprockee <962164+thesprockee@users.noreply.github.com> * Replace redundant findUserIDByDiscordID with existing GetUserIDByDiscordID function Co-authored-by: thesprockee <962164+thesprockee@users.noreply.github.com> --------- Co-authored-by: copilot-swe-agent[bot] <198982749+Copilot@users.noreply.github.com> Co-authored-by: thesprockee <962164+thesprockee@users.noreply.github.com> Co-authored-by: Andrew Bates <a@sprock.io>

* Initial plan * Add comprehensive Discord bot guild setup documentation Co-authored-by: thesprockee <962164+thesprockee@users.noreply.github.com> * Apply suggestion from @Copilot Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com> Signed-off-by: Andrew Bates <a@sprock.io> * Apply suggestion from @Copilot Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com> Signed-off-by: Andrew Bates <a@sprock.io> * Apply suggestion from @Copilot Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com> Signed-off-by: Andrew Bates <a@sprock.io> * Apply suggestion from @Copilot Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com> Signed-off-by: Andrew Bates <a@sprock.io> --------- Signed-off-by: Andrew Bates <a@sprock.io> Co-authored-by: copilot-swe-agent[bot] <198982749+Copilot@users.noreply.github.com> Co-authored-by: thesprockee <962164+thesprockee@users.noreply.github.com> Co-authored-by: Andrew Bates <a@sprock.io> Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>

… similar calculation corruption (#69) * Initial plan * Initial investigation of profile statistics issue Co-authored-by: thesprockee <962164+thesprockee@users.noreply.github.com> * Fix profile statistics showing large negative numbers Co-authored-by: thesprockee <962164+thesprockee@users.noreply.github.com> * Add comprehensive validation to prevent corrupted statistics in all calculations Co-authored-by: thesprockee <962164+thesprockee@users.noreply.github.com> --------- Co-authored-by: copilot-swe-agent[bot] <198982749+Copilot@users.noreply.github.com> Co-authored-by: thesprockee <962164+thesprockee@users.noreply.github.com>

Bumps the go_modules group with 1 update in the / directory: [golang.org/x/net](https://github.com/golang/net). Updates `golang.org/x/net` from 0.37.0 to 0.38.0 - [Commits](golang/net@v0.37.0...v0.38.0) --- updated-dependencies: - dependency-name: golang.org/x/net dependency-version: 0.38.0 dependency-type: indirect dependency-group: go_modules ... Signed-off-by: dependabot[bot] <support@github.com> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>

…AI assistants (#95) * Initial plan * Add comprehensive PR and ticket management workflow instructions Co-authored-by: thesprockee <962164+thesprockee@users.noreply.github.com> --------- Co-authored-by: copilot-swe-agent[bot] <198982749+Copilot@users.noreply.github.com> Co-authored-by: thesprockee <962164+thesprockee@users.noreply.github.com>

Co-authored-by: thesprockee <962164+thesprockee@users.noreply.github.com>

…tion Co-authored-by: thesprockee <962164+thesprockee@users.noreply.github.com>

thesprockee and others added 18 commits August 15, 2025 20:22

Update .gitignore to include 'nakama-debug' and remove unnecessary co…

58fd299

…mments

Optimize Dockerfile by adding 'go mod tidy' and 'go mod vendor' steps…

ecec5ed

… for improved dependency management

Refactor EVRProfile structure and remove ProfileOptions

ac3c1a6

Update Go version to 1.25

1424e3e

Remove unused pprof server setup from EvrPipeline

da3a4fd

Initial plan

1293e5e

Copilot AI assigned Copilot and thesprockee Sep 2, 2025

Copilot started work on behalf of thesprockee September 2, 2025 23:01 View session

Copilot AI and others added 2 commits September 2, 2025 23:09

Create comprehensive manual acceptance test plan for prep-nevr branch

5e75b1d

Co-authored-by: thesprockee <962164+thesprockee@users.noreply.github.com>

Add quick reference acceptance testing checklist for prep-nevr valida…

4bc494c

…tion Co-authored-by: thesprockee <962164+thesprockee@users.noreply.github.com>

Copilot AI changed the title ~~[WIP] Create a manual acceptance test plan for the prep-nevr branch~~ Create comprehensive manual acceptance test plan for prep-nevr branch Sep 2, 2025

Copilot AI requested a review from thesprockee September 2, 2025 23:11

Copilot finished work on behalf of thesprockee September 2, 2025 23:11

thesprockee force-pushed the main branch 3 times, most recently from 3046b52 to 4a7d4cc Compare September 28, 2025 14:43

Copilot AI mentioned this pull request Nov 6, 2025

Analyze open PRs and provide integration plan for v202 merge #123

Merged

thesprockee force-pushed the main branch from 0a2b882 to 33f8eab Compare December 16, 2025 14:05

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Uh oh!

Create comprehensive manual acceptance test plan for prep-nevr branch #99

Create comprehensive manual acceptance test plan for prep-nevr branch #99

Copilot AI commented Sep 2, 2025 •

edited

Loading

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

2 participants

Create comprehensive manual acceptance test plan for prep-nevr branch #99

Are you sure you want to change the base?

Create comprehensive manual acceptance test plan for prep-nevr branch #99

Conversation

Copilot AI commented Sep 2, 2025 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Documentation Added

1. Manual Acceptance Test Plan (docs/MANUAL_ACCEPTANCE_TEST_PLAN.md)

2. Quick Reference Checklist (docs/ACCEPTANCE_TESTING_CHECKLIST.md)

Features Covered

Testing Approach

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

2 participants

Copilot AI commented Sep 2, 2025 •

edited

Loading

1. Manual Acceptance Test Plan (`docs/MANUAL_ACCEPTANCE_TEST_PLAN.md`)

2. Quick Reference Checklist (`docs/ACCEPTANCE_TESTING_CHECKLIST.md`)