Skip to content

fix: plugin template test cases.#2317

Merged
crivetimihai merged 2 commits intomainfrom
fix/prompt_template_tests
Jan 24, 2026
Merged

fix: plugin template test cases.#2317
crivetimihai merged 2 commits intomainfrom
fix/prompt_template_tests

Conversation

@terylt
Copy link
Copy Markdown
Collaborator

@terylt terylt commented Jan 22, 2026

This PR fixes a small bug in the test cases of a plugin template.

@araujof araujof marked this pull request as ready for review January 22, 2026 16:09
@araujof araujof requested a review from crivetimihai as a code owner January 22, 2026 16:09
@araujof araujof self-requested a review January 22, 2026 16:09
Copy link
Copy Markdown
Member

@araujof araujof left a comment

Choose a reason for hiding this comment

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

LGTM

make test from the external plugin template is now ported to the latest changes in the plugin framework, and works in a freshly bootstrapped plugin.

@araujof araujof added bug Something isn't working plugins labels Jan 22, 2026
@araujof araujof added this to the Release 1.0.0-RC1 milestone Jan 22, 2026
@crivetimihai crivetimihai self-assigned this Jan 24, 2026
Teryl Taylor and others added 2 commits January 24, 2026 19:10
Signed-off-by: Teryl Taylor <terylt@ibm.com>
Signed-off-by: Frederico Araujo <frederico.araujo@ibm.com>
@crivetimihai crivetimihai force-pushed the fix/prompt_template_tests branch from dc5725f to 7531456 Compare January 24, 2026 19:13
@crivetimihai crivetimihai merged commit f3207f1 into main Jan 24, 2026
38 checks passed
@crivetimihai crivetimihai deleted the fix/prompt_template_tests branch January 24, 2026 19:19
kcostell06 pushed a commit to kcostell06/mcp-context-forge that referenced this pull request Feb 24, 2026
* fix: plugin template test cases.

Signed-off-by: Teryl Taylor <terylt@ibm.com>

* fix: context passing in unit test

Signed-off-by: Frederico Araujo <frederico.araujo@ibm.com>

---------

Signed-off-by: Teryl Taylor <terylt@ibm.com>
Signed-off-by: Frederico Araujo <frederico.araujo@ibm.com>
Co-authored-by: Teryl Taylor <terylt@ibm.com>
Co-authored-by: Frederico Araujo <frederico.araujo@ibm.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Something isn't working plugins

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants