Skip to content

Conversation

@yrobla
Copy link
Contributor

@yrobla yrobla commented Dec 9, 2025

No description provided.

@yrobla yrobla requested a review from Copilot December 9, 2025 09:27
@github-actions github-actions bot added the size/M Medium PR: 300-599 lines changed label Dec 9, 2025
@codecov
Copy link

codecov bot commented Dec 9, 2025

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 56.02%. Comparing base (f7e69fc) to head (4a30f81).
⚠️ Report is 1 commits behind head on main.

Additional details and impacted files
@@           Coverage Diff           @@
##             main    #2956   +/-   ##
=======================================
  Coverage   56.02%   56.02%           
=======================================
  Files         328      328           
  Lines       32342    32342           
=======================================
  Hits        18120    18120           
  Misses      12676    12676           
  Partials     1546     1546           

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

Copy link
Contributor

Copilot AI left a 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 PR adds an E2E test for composite workflow functionality using referenced VirtualMCPCompositeToolDefinition resources rather than inline definitions. This test validates that composite tools can be defined separately as reusable resources and then referenced by VirtualMCPServer instances using CompositeToolRefs.

Key changes:

  • Adds a new test file that validates composite tool definition by reference
  • Tests creation, validation, and execution of referenced composite workflows
  • Verifies proper status conditions and resource configuration

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@yrobla yrobla force-pushed the feat/test_composite_tools branch from 1f5a7c0 to d7d7c01 Compare December 9, 2025 09:44
@github-actions github-actions bot added size/M Medium PR: 300-599 lines changed and removed size/M Medium PR: 300-599 lines changed labels Dec 9, 2025
@yrobla yrobla force-pushed the feat/test_composite_tools branch from d7d7c01 to 4ac5022 Compare December 9, 2025 09:53
@github-actions github-actions bot added size/M Medium PR: 300-599 lines changed and removed size/M Medium PR: 300-599 lines changed labels Dec 9, 2025
@yrobla yrobla force-pushed the feat/test_composite_tools branch from 4ac5022 to 7970c58 Compare December 9, 2025 10:16
@yrobla yrobla requested a review from Copilot December 9, 2025 10:16
@github-actions github-actions bot added size/M Medium PR: 300-599 lines changed and removed size/M Medium PR: 300-599 lines changed labels Dec 9, 2025
@yrobla yrobla requested review from JAORMX and jhrozek December 9, 2025 10:22
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Copilot reviewed 1 out of 1 changed files in this pull request and generated 4 comments.


💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@yrobla yrobla force-pushed the feat/test_composite_tools branch from 7970c58 to 625e33d Compare December 9, 2025 10:32
@github-actions github-actions bot added size/M Medium PR: 300-599 lines changed and removed size/M Medium PR: 300-599 lines changed labels Dec 9, 2025
@github-actions github-actions bot added size/M Medium PR: 300-599 lines changed and removed size/M Medium PR: 300-599 lines changed labels Dec 9, 2025
@github-actions github-actions bot added size/M Medium PR: 300-599 lines changed and removed size/M Medium PR: 300-599 lines changed labels Dec 9, 2025
@github-actions github-actions bot added size/M Medium PR: 300-599 lines changed and removed size/M Medium PR: 300-599 lines changed labels Dec 9, 2025
@github-actions github-actions bot added size/M Medium PR: 300-599 lines changed and removed size/M Medium PR: 300-599 lines changed labels Dec 10, 2025
@yrobla yrobla merged commit 584cfe6 into main Dec 10, 2025
32 checks passed
@yrobla yrobla deleted the feat/test_composite_tools branch December 10, 2025 07:32
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

size/M Medium PR: 300-599 lines changed

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants