-
Notifications
You must be signed in to change notification settings - Fork 0
Open
Milestone
Description
Summary
Add CLI integration test for auto-parent behavior when updating existing prompts.
Context
PR #723 implemented auto-parent as default behavior for prompt push updates. While SDK tests cover the get_commit() method and CLI tests cover single push operations, there's no integration test for the update scenario.
Requirements
Add test to cli/tests/prompt_structured_test.rs that:
- Pushes a structured prompt to a new repository (first commit)
- Pushes again to the same repository with updated content
- Verifies the second push succeeds without 409 conflict
- Verifies auto-parent logic fetched and used the first commit's hash
Test Structure
#[test]
fn test_cli_push_prompt_update_with_auto_parent() {
// Setup: unique repo name, test credentials
// Step 1: Push initial prompt
// Step 2: Push update to same repo
// Step 3: Assert both succeeded
// Cleanup: delete test repo
}References
- Raised in PR ✨ feat(prompt): add parent commit options to resolve 409 conflicts #723 review comment: ✨ feat(prompt): add parent commit options to resolve 409 conflicts #723 (comment)
- Related to issue prompt push fails with 409 conflict when updating existing prompt - missing parent commit option #719 (409 conflict resolution)
Reactions are currently unavailable
Metadata
Metadata
Assignees
Labels
No labels