Skip to content

Capability Interface Test Framework #1083

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

Merged
merged 4 commits into from
May 13, 2025

Conversation

EasterTheBunny
Copy link
Collaborator

@EasterTheBunny EasterTheBunny commented Mar 25, 2025

This commit introduces a testing framework for asserting essential tests that all capabilities should conform to. As a starting point, both trigger and executable capabilities must conform to the BaseCapability interface and produce properly structured information. Additionally, executable capabilities should return metering information in a CapabilityResponse.

Tests for the BaseCapability cannot be disabled but those for ExecutableCapability can be per capability implementation as each implemtation will be different.

CRE-366

Requires

Supports

Base automatically changed from CRE-293/add-cost-field-capability-response to main March 26, 2025 06:35
@EasterTheBunny EasterTheBunny force-pushed the CRE-366/basic-capability-interface-tests branch from e5f144b to b99b8b5 Compare March 26, 2025 15:00
This commit introduces a testing framework for asserting essential tests that all capabilities should conform to. As a
starting point, both trigger and executable capabilities must conform to the `BaseCapability` interface and produce
properly structured information. Additionally, executable capabilities should return metering information in
a `CapabilityResponse`.

Tests for the `BaseCapability` cannot be disabled but those for `ExecutableCapability` can be per capability
implementation as each implemtation will be different.
@EasterTheBunny EasterTheBunny marked this pull request as ready for review May 6, 2025 18:23
@EasterTheBunny EasterTheBunny requested a review from a team as a code owner May 6, 2025 18:23
@EasterTheBunny EasterTheBunny merged commit 72874b2 into main May 13, 2025
12 of 15 checks passed
@EasterTheBunny EasterTheBunny deleted the CRE-366/basic-capability-interface-tests branch May 13, 2025 17:22
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