Skip to content

Enable integration tests#3734

Closed
bgavrilMS wants to merge 6 commits intomasterfrom
bogavril/tests
Closed

Enable integration tests#3734
bgavrilMS wants to merge 6 commits intomasterfrom
bogavril/tests

Conversation

@bgavrilMS
Copy link
Copy Markdown
Member

{PR title}

  • You've read the Contributor Guide and Code of Conduct.
  • You've included unit or integration tests for your change, where applicable.
  • You've included inline docs for your change, where applicable.
  • There's an open issue for the PR that you are making. If you'd like to propose a new feature or change, please open an issue to discuss the change or find an existing issue.

Summary of the changes (Less than 80 chars)

Description

{Detail}

Fixes #{bug number} (in this specific format)

@bgavrilMS bgavrilMS requested a review from a team as a code owner February 24, 2026 14:33
Comment thread build/template-run-unit-tests.yaml Outdated
tests\E2E Tests\WebAppUiTests\bin\**\WebAppUiTests.dll
tests\E2E Tests\TokenAcquirerTests\bin\**\TokenAcquirerTests.dll
tests\E2E Tests\NET 7 tests\IntegrationTests\bin\**\IntegrationTests.dll
tests\E2E Tests\AgentApplicationTests\bin\**\AgentApplicationTests.dll
Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Is it worth it to wildcard to run anything in the E2E Test folder?

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

Let's try

@bgavrilMS bgavrilMS marked this pull request as draft March 6, 2026 14:26
@gladjohn
Copy link
Copy Markdown
Contributor

can we merge this @bgavrilMS

@bgavrilMS
Copy link
Copy Markdown
Member Author

The ADO builds have their own logic, closing this.

@bgavrilMS bgavrilMS closed this Apr 21, 2026
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.

4 participants