Repository navigation
test(embed): isolate runtime integration scenarios - #620
Merged
Merged
Conversation
… test Replace the bare unwrap on runtime construction with unwrap_or_else that panics with the underlying error, so failures in the attached tools test report the actual cause instead of an opaque panic. Auto-committed-on: cuttlefish Co-authored-by: Medulla <medulla@tinyhumans.ai>
The attached tools test now prints the runtime build error to stderr before panicking, so the underlying failure is visible in test output instead of being buried in the panic message. Auto-committed-on: cuttlefish Co-authored-by: Medulla <medulla@tinyhumans.ai>
…se files The four large integration test modules for attached tools, cron agents, host-only tools, and structured turns were broken up into individual per-scenario test files with shared support modules, so each case can be read and run in isolation. Auto-committed-on: cuttlefish Co-authored-by: Medulla <medulla@tinyhumans.ai>
Wire the shared common module into the attached tools, host-only tools, and structured turns integration tests so they can reuse the common test helpers. Also adjust the cron agents support module to match the new shared setup. Auto-committed-on: cuttlefish Co-authored-by: Medulla <medulla@tinyhumans.ai>
Each test target includes the shared scenario modules but only runs one case, so the unused helpers in the other modules triggered dead code warnings. Adding a crate-level allow silences them without changing what the tests exercise. Auto-committed-on: cuttlefish Co-authored-by: Medulla <medulla@tinyhumans.ai>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
Split the multi-scenario
openhuman-embedintegration tests into one test binary per runtime scenario. Each process now owns its process-global runtime slot, so independent tests no longer collide with one another; shared scenario implementations live undertests/support/.Validation
cargo test -p openhuman-embed --tests --no-fail-fast— all targets passed serially.cargo clippy -p openhuman-embed --all-targets -- -D warnings— passed.cargo fmt --all -- --checkand the repository pre-push format/lint/Clippy hooks — passed.pnpm rust:layout,pnpm saas:ambient, andpnpm agent:runtime-boundary— passed.Part of tinyhumansai#8092.