Skip to content

Enable polyglot feature flag in E2E CLI tests#13981

Closed
Copilot wants to merge 2 commits intosebros/polyfrom
copilot/sub-pr-13978
Closed

Enable polyglot feature flag in E2E CLI tests#13981
Copilot wants to merge 2 commits intosebros/polyfrom
copilot/sub-pr-13978

Conversation

Copy link
Contributor

Copilot AI commented Jan 17, 2026

Description

Polyglot E2E tests (Python, TypeScript/React) require the features:polyglotSupportEnabled flag to access polyglot templates and commands. Without this flag, CLI commands like aspire init -l python fail as the feature is gated.

Changes:

  • Extended SourceAspireCliEnvironment() with optional enablePolyglotSupport parameter
  • Sets features__polyglotSupportEnabled=true environment variable when enabled
  • Supports both PowerShell (Windows) and Bash (Linux/macOS) syntax
  • Enabled flag in polyglot test classes:
    • PolyglotPythonTests - Python AppHost with Redis
    • PythonReactTemplateTests - FastAPI/React template
    • JsReactTemplateTests - ASP.NET Core/React (TypeScript) template

Checklist

  • Is this feature complete?
    • Yes. Ready to ship.
    • No. Follow-up changes expected.
  • Are you including unit tests for the changes and scenario tests if relevant?
    • Yes
    • No
  • Did you add public API?
    • Yes
    • No
  • Does the change make any security assumptions or guarantees?
    • Yes
    • No
  • Does the change require an update in our Aspire docs?
    • Yes
    • No

💡 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.

Co-authored-by: mitchdenny <513398+mitchdenny@users.noreply.github.com>
Copilot AI changed the title [WIP] Add common languages SDKs for polyglot apphost support Enable polyglot feature flag in E2E CLI tests Jan 17, 2026
Copilot AI requested a review from mitchdenny January 17, 2026 05:29
@sebastienros
Copy link
Member

@mitchdenny this used Sonnet 4.5 and my local attempts using Opus ended up with the simplest solution which seems correct ... currently testing.

@sebastienros sebastienros deleted the copilot/sub-pr-13978 branch January 17, 2026 06:38
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.

3 participants