Merged
Conversation
Co-authored-by: Levy Tate <iLevyTate@users.noreply.github.com>
- E2E: beta-workflow.spec.js — 30 Playwright tests covering: Setup phase (smart folder CRUD via API and UI) Discover phase (file import UI, drag-drop, API) Organize phase (empty state, undo/redo API) Search/Knowledge OS (Ctrl+K modal, search input) Settings (panel open/close, sections, API) Cross-phase navigation and API surface validation - Component: BetaWorkflowInteraction.test.js — 24 Jest tests covering: Navigation rendering and dispatch interactions Smart folder state management actions Analysis state machine (start/progress/success/failure) Phase transition validation Settings toggle and loading state File state lifecycle tracking Notification system integration Mock state builder for all beta test scenarios
…nd Linux The Linux path test failed when run after the Windows test because jest.resetModules() did not fully isolate Babel-transpiled ES module state across evaluations. The module-level `isWindows` constant from the Windows test leaked into the Linux test's module evaluation. Fix: reorder Linux test first, use jest.isolateModules() for true module isolation, save/restore navigator in beforeEach/afterEach, and add writable flag to Object.defineProperty for reliable cleanup.
…d features New E2E test specs (54 tests total): - full-pipeline.spec.js (9): Analysis → organize → complete pipeline, smart folder routing, batch organize API, phase navigation - vision-ocr-analysis.spec.js (7): Vision/OCR API surface, model config, image/PDF file type support, analysis history tracking - chat-understand.spec.js (6): Chat API surface, Understand tab UI, chat input/send, new chat button, conversation history - knowledge-graph.spec.js (11): Relate tab UI, graph controls, legend, cluster toggle, add-to-graph search, knowledge/embedding APIs - settings-backup-restore.spec.js (7): Settings CRUD roundtrip, backup list, log export UI, settings sections validation - session-persistence.spec.js (5): Smart folder persistence across restart, settings persistence, undo/redo state, analysis history - model-switching.spec.js (9): Model listing, config read/write, text model update, connection status, AI settings UI, model selectors
|
Cursor Agent can help with this pull request. Just |
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
AGENTS.mdwith development environment setup, workarounds for cloud VMs, and links to test files.platformUtils.test.jsbug, ensuring all 6,109 automated tests now pass.Scope
docs,testing,e2e,components,coreType of Change
Test Plan
Automated
npm run format:checknpm run lintnpm test(All 6,109 tests now pass)Manual QA
TESTING.md)Manual QA notes:
App version "0.0"error), and large LLM inference/consecutive chat queries cause renderer crashes due to CPU memory limits.Checklist