Removing cross SDK tests from repo#368
Merged
rodrigobr-msft merged 3 commits intomainfrom Apr 15, 2026
Merged
Conversation
Contributor
There was a problem hiding this comment.
Pull request overview
Note
Copilot was unable to run its full agentic suite in this review.
Removes the cross-SDK test harness, sample agents, and related configuration under dev/testing/cross-sdk-tests, likely to decouple or relocate cross-SDK integration testing outside this repo.
Changes:
- Deleted cross-SDK pytest test suites (core channels + telemetry) and shared scenario utilities.
- Removed bundled “quickstart” and “core-agent” sample implementations/scripts across Python/.NET/JS.
- Removed local test runner configuration artifacts (e.g.,
pytest.ini, env templates, subdirectory.gitignore) and added a minimalProgram.csplaceholder.
Reviewed changes
Copilot reviewed 45 out of 57 changed files in this pull request and generated 1 comment.
Show a summary per file
| File | Description |
|---|---|
| dev/testing/cross-sdk-tests/tests/telemetry/test_basic_telemetry.py | Removes placeholder telemetry tests and scenario registration. |
| dev/testing/cross-sdk-tests/tests/core/test_webchat.py | Removes WebChat channel integration tests for the basic agent. |
| dev/testing/cross-sdk-tests/tests/core/test_msteams.py | Removes MSTeams channel integration tests for the basic agent. |
| dev/testing/cross-sdk-tests/tests/core/test_directline.py | Removes DirectLine channel integration tests for the basic agent. |
| dev/testing/cross-sdk-tests/tests/core/test_basic_agent_base.py | Removes shared base test class and external scenario config. |
| dev/testing/cross-sdk-tests/tests/basic/test_quickstart.py | Removes cross-SDK “quickstart” integration tests. |
| dev/testing/cross-sdk-tests/tests/_common/utils.py | Removes scenario creation helpers used by tests. |
| dev/testing/cross-sdk-tests/tests/_common/types.py | Removes SDKVersion enum used by scenarios. |
| dev/testing/cross-sdk-tests/tests/_common/source_scenario.py | Removes script-based scenario runner (PowerShell-based). |
| dev/testing/cross-sdk-tests/tests/_common/constants.py | Removes constants for agent paths and local endpoints. |
| dev/testing/cross-sdk-tests/tests/_common/init.py | Removes exports for shared test utilities. |
| dev/testing/cross-sdk-tests/pytest.ini | Removes pytest configuration for the cross-SDK test package. |
| dev/testing/cross-sdk-tests/env.TEMPLATE | Removes environment template for cross-SDK test runs. |
| dev/testing/cross-sdk-tests/agents/quickstart/python/src/start_server.py | Removes Python quickstart agent server bootstrapping. |
| dev/testing/cross-sdk-tests/agents/quickstart/python/src/main.py | Removes Python quickstart agent entrypoint. |
| dev/testing/cross-sdk-tests/agents/quickstart/python/src/agent.py | Removes Python quickstart agent logic and handlers. |
| dev/testing/cross-sdk-tests/agents/quickstart/python/requirements.txt | Removes Python quickstart dependencies list. |
| dev/testing/cross-sdk-tests/agents/quickstart/python/_run_agent.ps1 | Removes Python quickstart run script. |
| dev/testing/cross-sdk-tests/agents/quickstart/net/_run_agent.ps1 | Removes .NET quickstart run script. |
| dev/testing/cross-sdk-tests/agents/quickstart/net/Quickstart.csproj | Removes .NET quickstart project definition. |
| dev/testing/cross-sdk-tests/agents/quickstart/net/Program.cs | Removes .NET quickstart hosting program. |
| dev/testing/cross-sdk-tests/agents/quickstart/net/MyAgent.cs | Removes .NET quickstart agent implementation. |
| dev/testing/cross-sdk-tests/agents/quickstart/net/AspNetExtensions.cs | Removes .NET auth/token validation extensions used by quickstart. |
| dev/testing/cross-sdk-tests/agents/quickstart/js/tsconfig.json | Removes JS/TS quickstart build configuration. |
| dev/testing/cross-sdk-tests/agents/quickstart/js/src/index.ts | Removes JS quickstart agent server entrypoint. |
| dev/testing/cross-sdk-tests/agents/quickstart/js/package.json | Removes JS quickstart package manifest/scripts. |
| dev/testing/cross-sdk-tests/agents/quickstart/js/env.TEMPLATE | Removes JS quickstart env template. |
| dev/testing/cross-sdk-tests/agents/quickstart/js/_run_agent.ps1 | Removes JS quickstart run script. |
| dev/testing/cross-sdk-tests/agents/quickstart/README.md | Removes quickstart agent documentation. |
| dev/testing/cross-sdk-tests/agents/core-agent/python/src/weather/plugins/weather_forecast_plugin.py | Removes core-agent weather forecast plugin. |
| dev/testing/cross-sdk-tests/agents/core-agent/python/src/weather/plugins/weather_forecast.py | Removes weather forecast model. |
| dev/testing/cross-sdk-tests/agents/core-agent/python/src/weather/plugins/date_time_plugin.py | Removes date/time plugin. |
| dev/testing/cross-sdk-tests/agents/core-agent/python/src/weather/plugins/adaptive_card_plugin.py | Removes adaptive card generation plugin. |
| dev/testing/cross-sdk-tests/agents/core-agent/python/src/weather/plugins/init.py | Removes plugin package exports. |
| dev/testing/cross-sdk-tests/agents/core-agent/python/src/weather/agents/weather_forecast_agent.py | Removes LLM-based weather forecast agent implementation. |
| dev/testing/cross-sdk-tests/agents/core-agent/python/src/config.py | Removes core-agent python configuration. |
| dev/testing/cross-sdk-tests/agents/core-agent/python/src/app.py | Removes core-agent python aiohttp host app. |
| dev/testing/cross-sdk-tests/agents/core-agent/python/src/agent.py | Removes core-agent python routing/handlers implementation. |
| dev/testing/cross-sdk-tests/agents/core-agent/python/requirements.txt | Removes core-agent python dependency list. |
| dev/testing/cross-sdk-tests/agents/core-agent/python/pre_requirements.txt | Removes preinstall dependency list for core-agent. |
| dev/testing/cross-sdk-tests/agents/core-agent/python/env.TEMPLATE | Removes core-agent environment template. |
| dev/testing/cross-sdk-tests/agents/core-agent/python/README.md | Removes core-agent python documentation. |
| dev/testing/cross-sdk-tests/agents/core-agent/README.md | Removes core-agent high-level documentation. |
| dev/testing/cross-sdk-tests/Program.cs | Adds a placeholder file intended to address a CodeQL issue. |
| dev/testing/cross-sdk-tests/.gitignore | Removes a directory-scoped ignore file that previously prevented generated artifacts from being committed. |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
MattB-msft
approved these changes
Apr 15, 2026
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.
No description provided.