Skip to content

Conversation

pkarpala-microsoft
Copy link
Contributor

@pkarpala-microsoft pkarpala-microsoft commented Jun 6, 2025

This pull request introduces several changes aimed at improving test configurations, enhancing error handling, and updating dependencies. The main updates include adding support for mocked data in tests, refining error messages, and upgrading the Nuxt.js framework version. Below is a categorized summary of the most important changes:

Test Configuration Enhancements:

  • Updated Playwright test workflows in .github/workflows/playwright.yml to include environment variables for mocked data and GitHub organization settings (NUXT_PUBLIC_IS_DATA_MOCKED and NUXT_PUBLIC_GITHUB_ORG). [1] [2]
  • Modified e2e test files (e2e-tests/copilot.ent.spec.ts, e2e-tests/copilot.org.spec.ts, e2e-tests/copilot.team.spec.ts, e2e-tests/metrics.spec.ts) to append ?mock=true to URLs, enabling mock data usage during tests. [1] [2] [3] [4]

Error Handling Improvements:

  • Added a new error case for HTTP 422 responses in app/components/MainComponent.vue to provide better feedback when the Copilot Metrics API is not enabled.

Dependency Updates:

  • Upgraded the Nuxt.js framework version from ^3.15.3 to ^3.17.5 in package.json.

Code Simplifications:

  • Simplified header construction in server/modules/authentication.ts by ensuring the Authorization header is always set, even when mocking data. [1] [2]

Dockerfile Enhancements:

  • Added a fallback mechanism in Dockerfile to use NUXT_GITHUB_TOKEN for the session password if SESSION_SECRET is not set.

@karpikpl karpikpl changed the title nuxt upgrade nuxt upgrade to 3.17.5 Jun 6, 2025
@karpikpl karpikpl merged commit 255ecde into github-copilot-resources:main Jun 6, 2025
3 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants