feat(queue): add e2e tests for fast-v2v and restyle-v2v#83
Merged
AdirAmsalem merged 6 commits intomainfrom Feb 15, 2026
Merged
Conversation
… helpers - Add missing e2e tests for lucy-fast-v2v and lucy-restyle-v2v - Unify expectResult helper to handle both Blob and QueueJobResult - Add job_id to QueueJobResult type for better failure diagnostics - Fix .gitignore path for e2e-output directory - Remove accidentally committed e2e output files
commit: |
AdirAmsalem
changed the title
…eference image scenarios - Updated the test for lucy-restyle-v2v to include a prompt in the description. - Added a new test case for lucy-restyle-v2v using a reference image, improving coverage for video restyling functionality. - Adjusted expected result identifiers to reflect the new test scenarios.
Add e2e tests for all 4 realtime models (mirage, mirage_v2, lucy_v2v_720p_rt, lucy_2_rt) running in headless Chromium via @vitest/browser and Playwright. - Synthetic canvas stream avoids needing a real camera - Tests connect, verify session, send prompt, and disconnect - API key injected via Vite define from DECART_API_KEY env var - New test:e2e:realtime script in package.json
![cursor[bot]](https://avatars.githubusercontent.com/in/1210556?s=60&v=4){kind=small}
- Removed fallback for DECART_API_KEY in vitest config to ensure it is always defined. - Enhanced vitest configuration for e2e tests by specifying test files directly. - Refactored e2e-realtime tests to streamline the test flow and improve readability, including reducing timeout and retry logic. - Updated test descriptions and assertions for clarity and consistency.
- Updated vitest configuration to exclude the `e2e-realtime.test.ts` file from test runs. - Enhanced cleanup in the `e2e-realtime.test.ts` by ensuring all media tracks are stopped after tests to prevent resource leaks.
- Updated the cleanup logic in `e2e-realtime.test.ts` to use a for-of loop for stopping media tracks, improving readability and consistency in resource management.
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
2 participants
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
lucy-fast-v2vandlucy-restyle-v2vvideo modelsexpectProcessResultandexpectQueueResultinto a singleexpectResulthelper that handles bothBlobandQueueJobResultjob_idtoQueueJobResulttype so failed jobs surface the job ID in errors.gitignorepath and remove accidentally committed e2e output filesNote
Medium Risk
Public SDK typing/output for
queue.submitAndPollchanges to always includejob_id, which may require downstream type updates. New Playwright/Vitest browser dependencies and concurrent/retry behavior may affect CI stability and runtime.Overview
Adds new E2E coverage in the SDK: video queue tests now include
lucy-fast-v2vandlucy-restyle-v2vvariants, and a separate Playwright-powered browser E2E suite is introduced for realtime models (wired viatest:e2e:realtimeand a new Vitest config).Queue polling now returns
job_idin both success and failureQueueJobResultvariants, and the existing E2E tests are refactored to use a singleexpectResulthelper that can validate either directBlobresults or queued job results. Minor repo hygiene changes include fixing the ignored E2E output path and excluding the realtime browser test from the default Vitest config.Written by Cursor Bugbot for commit 64953d2. This will update automatically on new commits. Configure here.