Skip to content

Conversation

Copy link
Contributor

Copilot AI commented Dec 6, 2025

Changes proposed in this request

Agentic integration tests were failing due to stale identity coordinates after tenant deauth. Updated test constants to new coordinates following the same pattern as microsoft-identity-web#3640:

  • Agent application ClientId: d15884b6-a447-4dd5-a5a5-a668c49f6300d05619c9-dbf2-4e60-95fd-cc75dd0db451
  • Agent identity: d84da24a-2ea2-42b8-b5ab-8637ec208024edbfbbe7-d240-40dd-aee2-435201dbaa9c
  • User UPN: [email protected][email protected]

Testing

Existing tests validate the functionality. Changes are configuration-only.

Performance impact

None.

Documentation

  • All relevant documentation is updated.

Warning

Firewall rules blocked me from connecting to one or more addresses (expand for details)

I tried to connect to the following addresses, but was blocked by firewall rules:

  • https://api.github.com/repos/AzureAD/microsoft-identity-web/commits/2f87a894f7f581facf3ccdae8b2b51082b49608a
    • Triggering command: /usr/bin/curl curl -s REDACTED (http block)

If you need me to access, download, or install something from one of these locations, you can either:

Original prompt

This section details on the original issue you should resolve

<issue_title>[Internal] Fix Agentic tests</issue_title>
<issue_description>Agentic tests are now failing because of tenant deauth. See AzureAD/microsoft-identity-web@2f87a89 for a similar fix in Identity.Web</issue_description>

<agent_instructions>Use AzureAD/microsoft-identity-web@2f87a89 for details on a new setup that will fix these tests</agent_instructions>

Comments on the Issue (you are @copilot in this section)


💡 You can make Copilot smarter by setting up custom instructions, customizing its development environment and configuring Model Context Protocol (MCP) servers. Learn more Copilot coding agent tips in the docs.

Copilot AI changed the title [WIP] Fix Agentic tests due to tenant deauth Update agent identity coordinates in Agentic tests after tenant deauth Dec 6, 2025
Copilot AI requested a review from bgavrilMS December 6, 2025 12:19
@bgavrilMS bgavrilMS marked this pull request as ready for review December 7, 2025 14:33
@bgavrilMS bgavrilMS requested a review from a team as a code owner December 7, 2025 14:33
@bgavrilMS bgavrilMS force-pushed the copilot/fix-agentic-test-failures branch from c6a706a to e491d6e Compare December 7, 2025 15:10
public class Agentic
{
const string ClientId = "d15884b6-a447-4dd5-a5a5-a668c49f6300"; // agent app
const string ClientId = "d05619c9-dbf2-4e60-95fd-cc75dd0db451"; // agent app
Copy link
Contributor

Choose a reason for hiding this comment

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

Did you try locally, @bgavrilMS ?
This tenant was de-authed the night after my idweb PR

Copy link
Member

Choose a reason for hiding this comment

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

I did :(. I ended up adding [Ignore] to the tests.

@bgavrilMS bgavrilMS closed this Dec 7, 2025
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.

[Internal] Fix Agentic tests

3 participants