Skip to content

feat(runtime): add support for executing test specs#1389

Open
johnyob wants to merge 1 commit intoajob410@jstz-test-extfrom
ajob410@jstz-test-execution
Open

feat(runtime): add support for executing test specs#1389
johnyob wants to merge 1 commit intoajob410@jstz-test-extfrom
ajob410@jstz-test-execution

Conversation

@johnyob
Copy link
Collaborator

@johnyob johnyob commented Nov 19, 2025

Context

This PR adds support for executing test specifications. The test executor runs the tests and associated lifecycle hooks.

Description

Dependencies: #1378

All logic for this PR is in jstz_test/executor.rs, the bulk of it being in test_specifier.

Manually testing the PR

cargo test --package jstz_runtime --lib --features v2_runtime -- ext::jstz_test::executor

@johnyob johnyob force-pushed the ajob410@jstz-test-execution branch from 58ecac8 to 20a5497 Compare November 19, 2025 16:30
@codecov
Copy link

codecov bot commented Nov 19, 2025

Codecov Report

❌ Patch coverage is 92.64706% with 35 lines in your changes missing coverage. Please review.
✅ Project coverage is 64.38%. Comparing base (a2c945d) to head (20a5497).

Files with missing lines Patch % Lines
crates/jstz_runtime/src/ext/jstz_test/executor.rs 92.64% 16 Missing and 19 partials ⚠️
Files with missing lines Coverage Δ
crates/jstz_runtime/src/ext/jstz_test/mod.rs 44.00% <ø> (+9.09%) ⬆️
crates/jstz_runtime/src/ext/jstz_test/executor.rs 92.64% <92.64%> (ø)

... and 2 files with indirect coverage changes


Continue to review full report in Codecov by Sentry.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update a2c945d...20a5497. Read the comment docs.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

.borrow_mut::<TestContainer>(),
);

println!("Descriptions: {:?}", container.descriptions);
Copy link
Collaborator

Choose a reason for hiding this comment

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

Remove?

}
}

pub async fn test_specifier(
Copy link
Collaborator

Choose a reason for hiding this comment

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

Doc comment overview of how this works would be helpful for future maintainers

@zcabter zcabter self-requested a review December 2, 2025 15:35
@zcabter zcabter assigned johnyob and unassigned zcabter Dec 2, 2025
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.

2 participants