Skip to content

Update in-codebase documentation for E2E tests#6355

Merged
florelis merged 6 commits into
microsoft:masterfrom
Trenly:E2EDocs
Jul 14, 2026
Merged

Update in-codebase documentation for E2E tests#6355
florelis merged 6 commits into
microsoft:masterfrom
Trenly:E2EDocs

Conversation

@Trenly

@Trenly Trenly commented Jul 7, 2026

Copy link
Copy Markdown
Contributor

📖 Description

Adds documentation for some E2E test settings that was missing

🔗 References

🔍 Validation

Copilot source code review

✅ Checklist

📋 Issue Type

  • Bug fix
  • Feature
  • Task
Microsoft Reviewers: Open in CodeFlow

@Trenly Trenly requested a review from a team as a code owner July 7, 2026 17:04
@florelis florelis requested a review from Copilot July 14, 2026 21:55
florelis
florelis previously approved these changes Jul 14, 2026

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Pull request overview

Updates the in-repo E2E test documentation and defaults so developers (and automation) have a clearer, more complete set of TestRunParameters for AppInstallerCLIE2ETests.

Changes:

  • Expand Test.runsettings to include additional E2E parameters (e.g., local server cert, in-proc testbed, forced experimental features).
  • Update the E2E README to document the added parameters and show them in the example configuration.
  • Add a Copilot instruction file to keep TestSetup.cs, README.md, and Test.runsettings parameter docs in sync, and ignore developer-local .runsettings files by default.

Reviewed changes

Copilot reviewed 3 out of 4 changed files in this pull request and generated 1 comment.

File Description
src/AppInstallerCLIE2ETests/Test.runsettings Adds/renames documented E2E run parameters and updates default parameter entries.
src/AppInstallerCLIE2ETests/README.md Documents additional E2E runsettings parameters and updates the example snippet.
.gitignore Ignores developer-specific *.runsettings while keeping the tracked example runsettings file.
.github/instructions/e2e-test-setup-parameters.instructions.md Adds guidance to update parameter docs in README and Test.runsettings when modifying TestSetup.cs.

Comment thread src/AppInstallerCLIE2ETests/Test.runsettings
Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com>
@florelis florelis merged commit 47c8143 into microsoft:master Jul 14, 2026
3 checks passed
@Trenly Trenly deleted the E2EDocs branch July 14, 2026 22:32
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.

3 participants