-
Notifications
You must be signed in to change notification settings - Fork 200
Fix E2ETests namespace, simplify CI yml #3291
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
base: main
Are you sure you want to change the base?
Conversation
There was a problem hiding this 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 pull request refactors E2E test namespaces to match the containing assembly name and simplifies CI configuration by consolidating build and test steps.
Changes:
- Updated all E2E test namespaces from
Microsoft.Azure.Functions.Tests.E2ETeststoMicrosoft.Azure.Functions.Worker.E2ETeststo align with assembly name - Simplified CI YAML files to build only test projects instead of full solution, eliminating redundant builds
- Consolidated Windows E2E test execution by removing separate AspNetCore test step (tests now self-skip on netfx via
IgnoreOnNetFxTestRunTheoryattribute)
Reviewed changes
Copilot reviewed 18 out of 18 changed files in this pull request and generated 1 comment.
Show a summary per file
| File | Description |
|---|---|
| test/E2ETests/E2ETests/TimerEndToEndTests.cs | Updated namespace from Tests.E2ETests to Worker.E2ETests |
| test/E2ETests/E2ETests/Tables/TablesEndToEndTests.cs | Updated namespace to include Worker segment |
| test/E2ETests/E2ETests/Storage/QueueEndToEndTests.cs | Updated namespace to include Worker segment |
| test/E2ETests/E2ETests/Storage/BlobEndToEndTests.cs | Updated namespace to include Worker segment |
| test/E2ETests/E2ETests/JobObjectRegistry.cs | Updated namespace and removed unused using System directive |
| test/E2ETests/E2ETests/HttpEndToEndTests.cs | Updated namespace to include Worker segment |
| test/E2ETests/E2ETests/Helpers/TableHelpers.cs | Updated using statement to reference new namespace |
| test/E2ETests/E2ETests/Helpers/StorageHelpers.cs | Updated namespace to include Worker segment |
| test/E2ETests/E2ETests/Helpers/HttpHelpers.cs | Updated namespace to include Worker segment |
| test/E2ETests/E2ETests/Helpers/FactAttributes/IgnoreOnNetFxTestRunTheory.cs | Updated namespace and added trailing newline |
| test/E2ETests/E2ETests/Helpers/CosmosDBHelpers.cs | Updated namespace to include Worker segment |
| test/E2ETests/E2ETests/Fixtures/FunctionAppFixture.cs | Updated namespace to include Worker segment |
| test/E2ETests/E2ETests/Fixtures/FixtureHelpers.cs | Updated namespace to include Worker segment |
| test/E2ETests/E2ETests/Cosmos/CosmosDBEndToEndTests.cs | Updated namespace to include Worker segment |
| test/E2ETests/E2ETests/Constants.cs | Updated namespace to include Worker segment |
| test/E2ETests/E2ETests/AspNetCore/CancellationEndToEndTests.cs | Updated using statement to reference new namespace |
| eng/ci/templates/jobs/run-integration-tests-windows.yml | Added test_projects variable, changed build to target only test projects, consolidated E2E test execution into single step |
| eng/ci/templates/jobs/run-integration-tests-linux.yml | Added test_projects variable, changed build to target only test projects |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
Issue describing the changes in this PR
resolves #issue_for_this_pr
Pull request checklist
release_notes.mdAdditional information
This PR refactors E2E tests: