Skip to content

feat: Matrix integration tests - #98

Open
BenjaminMichaelis wants to merge 6 commits into
mainfrom
bmichaelis/MatrixIntegrationTests
Open

feat: Matrix integration tests#98
BenjaminMichaelis wants to merge 6 commits into
mainfrom
bmichaelis/MatrixIntegrationTests

Conversation

@BenjaminMichaelis

Copy link
Copy Markdown
Member

No description provided.

CopilotAI review requested due to automatic review settings February 21, 2026 18:28
@BenjaminMichaelisBenjaminMichaelis self-assigned this Feb 21, 2026

CopilotAI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

This pull request introduces cross-platform and cross-browser matrix testing for integration tests. The key objectives are to enable comprehensive testing across Windows, Linux, and macOS operating systems using Chromium, Firefox, and WebKit browsers, and to properly separate integration tests from unit tests in the CI/CD pipeline.

Changes:

  • Updated npm registry configuration across multiple packages to use the public npmjs.org registry
  • Added matrix testing strategy to GitHub Actions workflow for integration tests across 3 operating systems and 3 browsers (9 total combinations)
  • Enhanced PlaywrightSession to support browser selection via parameter or environment variable
  • Added [Trait("TestType", "Integration")] to integration test classes to enable test filtering
  • Removed Linux-specific test skipping logic from IntegrationTestFactAttribute
  • Updated deployment configuration with new Azure Container Registry secret names

Reviewed changes

Copilot reviewed 8 out of 8 changed files in this pull request and generated 7 comments.

Show a summary per file
FileDescription
src/microsoft-trydotnet/.npmrcChanged npm registry from Azure DevOps to public npmjs.org
src/microsoft-trydotnet-styles/.npmrcChanged npm registry from Azure DevOps to public npmjs.org
src/microsoft-trydotnet-editor/.npmrcChanged npm registry from Azure DevOps to public npmjs.org
src/microsoft-learn-mock/.npmrcChanged npm registry from Azure DevOps to public npmjs.org
src/Microsoft.TryDotNet.IntegrationTests/WasmRunnerTests.csAdded Integration trait for test filtering
src/Microsoft.TryDotNet.IntegrationTests/TryDotNetJsIntegrationTests.csAdded Integration trait for test filtering
src/Microsoft.TryDotNet.IntegrationTests/PlaywrightSession.csAdded multi-browser support with environment variable and parameter-based browser selection
src/Microsoft.TryDotNet.IntegrationTests/Microsoft.TryDotNet.IntegrationTests.csprojAdded global using for Xunit and removed trailing whitespace
src/Microsoft.TryDotNet.IntegrationTests/IntegrationTestFactAttribute.csRemoved Linux-specific test skipping to enable cross-platform testing
src/Microsoft.TryDotNet.IntegrationTests/EditorTests.csAdded Integration trait for test filtering
.github/workflows/Build-Test-And-Deploy.yamlAdded matrix testing strategy, updated test filters, changed ACR secret names
.github/actions/setup-dotnet/action.ymlAdded configurable build configuration parameter (Release/Debug)

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread.github/workflows/Build-Test-And-Deploy.yaml Outdated
Comment thread.github/workflows/Build-Test-And-Deploy.yaml Outdated
Comment threadsrc/Microsoft.TryDotNet.IntegrationTests/PlaywrightSession.cs Outdated
Comment thread.github/workflows/Build-Test-And-Deploy.yaml Outdated
Comment thread.github/workflows/Build-Test-And-Deploy.yaml
Comment thread.github/workflows/Build-Test-And-Deploy.yaml Outdated
Comment threadsrc/Microsoft.TryDotNet.IntegrationTests/PlaywrightSession.cs Outdated
Sign up for freeto 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

@BenjaminMichaelis