Merged
Conversation
…PI application - Attach session IDs to spans for better traceability in Application Insights.
…gic in user_clarification endpoint Co-authored-by: Abdul-Microsoft <192570837+Abdul-Microsoft@users.noreply.github.com>
fix: resolve memory_store unbound error and RAI condition logic in user_clarification endpoint
ci: Down merge from main to dev-v4
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
- Updated get_azure_credential and get_azure_credential_async methods to use exclude_environment_credential=True for dev environment. - Refactored MCPEnabledBase to acquire credentials using centralized config method. - Added unit tests for async credential retrieval in both dev and production environments.
fix: Improve the code quality
feat: improve Application Insights logging and telemetry handling
…oft/Multi-Agent-Custom-Automation-Engine-Solution-Accelerator into bugfix/codeQL-suggestions # Please enter a commit message to explain why this merge is necessary, # especially if it merges an updated upstream into a topic branch. # # Lines starting with '#' will be ignored, and an empty message aborts # the commit.
fix: implemented codeQL and copilot suggestions
…arity and error prevention
fix: Removed the diagnostic settings from AppInsights to avoid duplicate logs
test: enhance mock specifications in various test files
Roopan-Microsoft
approved these changes
Mar 16, 2026
Avijit-Microsoft
approved these changes
Mar 16, 2026
|
🎉 This PR is included in version 4.1.0 🎉 The release is available on GitHub release Your semantic-release bot 📦🚀 |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Purpose
This pull request introduces improvements to test configuration and isolation for the backend test suite, refactors environment and module mocking, and clarifies test execution requirements. The main focus is to ensure tests run reliably both individually and as part of the full suite, with proper handling of dependencies and environment setup.
Test environment and module mocking improvements:
src/tests/backend/conftest.pythat sets up required environment variables and mocks for external modules (agent_framework,azure.monitor.opentelemetry, andazure.ai.projects.models) to ensure tests are isolated and dependencies are handled consistently.conftest.pyat the project root to addsrcandsrc/backendtosys.path, improving import reliability for backend and common modules, and removed previous path setup formagentic_agents.Clarification and reliability of test execution:
src/tests/backend/test_app.pyto rely onconftest.pyfor environment setup and module mocking, and added logic to skip tests if thev4module has been mocked by other test files, preventing import errors and clarifying when tests should be run individually. [1] [2]test_app.py, simplifying the test file and ensuring it works seamlessly during full suite runs.Test coverage and workflow enhancements:
.github/workflows/test.ymlto setPYTHONPATH, run tests with coverage in the correct order, and provide clearer error reporting and threshold checks for coverage results.Test correctness improvements:
create_RAI_agentintest_utils_af.pyto verify that the team configuration copy is used and updated, ensuring correct behavior and test coverage. [1] [2] [3]Cleanup:
__init__.pyfiles from test directories to reduce clutter. [1] [2]Does this introduce a breaking change?
How to Test
What to Check
Verify that the following are valid
Other Information