Skip to content

Update pack tests to use new test framework #4445

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

Open
wants to merge 8 commits into
base: main
Choose a base branch
from
Open

Conversation

Copilot
Copy link
Contributor

@Copilot Copilot AI commented May 27, 2025

  • Analyze current PackFunctionTests.cs structure and understand requirements
  • Study new test framework patterns from existing tests
  • Understand the differences between old CliTester.Run() approach and new command-based approach
  • Create FuncPackCommand class in TestFramework
  • Create Commands/FuncPack directory structure in E2E tests
  • Migrate pack_python_from_cache test to new framework
  • Mark old test as skipped and deprecated with reference to new location
  • Test and validate the migration works correctly

Fixes #4335.


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

@aishwaryabh aishwaryabh changed the title [WIP] Update pack tests to use new test framework Update pack tests to use new test framework Jun 9, 2025
@aishwaryabh aishwaryabh marked this pull request as ready for review June 9, 2025 19:56
@aishwaryabh aishwaryabh requested a review from a team as a code owner June 9, 2025 19:56
@aishwaryabh
Copy link
Contributor

FYI i pulled the changes from @gavin-aguiar's PR to ensure that the test passes. His changes should be merged in first before this PR gets merged in to fix the functionality of the command

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.

Update pack tests to use new test framework
3 participants