Skip to content

Conversation

@surgupta-msft
Copy link
Contributor

Issue describing the changes in this PR

resolves #issue_for_this_pr

Pull request checklist

IMPORTANT: Currently, changes must be backported to the in-proc branch to be included in Core Tools and non-Flex deployments.

  • Backporting to the in-proc branch is not required
    • Otherwise: Link to backporting PR
  • My changes do not require documentation changes
    • Otherwise: Documentation issue linked to PR
  • My changes should not be added to the release notes for the next release
    • Otherwise: I've added my notes to release_notes.md
  • My changes do not need to be backported to a previous version
    • Otherwise: Backport tracked by issue/PR #issue_or_pr
  • My changes do not require diagnostic events changes
    • Otherwise: I have added/updated all related diagnostic events and their documentation (Documentation issue linked to PR)
  • I have added all required tests (Unit tests, E2E tests)

Additional information

Additional PR information

@surgupta-msft surgupta-msft requested a review from a team as a code owner October 22, 2025 15:25
Copilot AI review requested due to automatic review settings October 22, 2025 15:25
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull Request Overview

This PR fixes test failures by clearing the environment cache before test execution. The changes ensure that tests using environment variables start with a clean state, preventing interference from previously cached values that could cause intermittent test failures.

Key Changes:

  • Added EnvironmentExtensions.ClearCache() calls at the beginning of two test methods that depend on environment variable settings

@jviau
Copy link
Contributor

jviau commented Oct 22, 2025

Consider implementing IDisposable on this test class to run EnvironmentExtensions.ClearCache(); after every test

@surgupta-msft
Copy link
Contributor Author

/azp run

@azure-pipelines
Copy link

Azure Pipelines successfully started running 2 pipeline(s).

@surgupta-msft surgupta-msft merged commit 19c738d into dev Oct 23, 2025
9 checks passed
@surgupta-msft surgupta-msft deleted the surgupta/tests-fix branch October 23, 2025 19:11
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