Commit e638f58

and

committed

feat: refactor remote.ts for testability and add comprehensive tests

Refactor remote.ts by extracting 5 testable helper methods: - validateCredentials() - handles login flow and credential validation - createWorkspaceClient() - creates workspace REST client - setupBinary() - handles binary path setup for prod/dev modes - validateServerVersion() - checks server compatibility and features - fetchWorkspace() - fetches workspace with comprehensive error handling Add remote.test.ts with 17 comprehensive test cases covering: - Constructor and instance creation - Credential validation and login prompts - Binary setup for production and development modes - Server version validation and incompatibility handling - Workspace fetching with 404/401 error scenarios - Remote window management (close/reload operations) Benefits: - Improved code maintainability through method extraction - Reduced complexity in main setup() method - Comprehensive error scenario testing - Memory-efficient testing approach vs extensive mocking - Maintains all existing functionality while enabling better testing Total test coverage: 274 tests across 14 files (14/17 source files = 82%) 🤖 Generated with [Claude Code](https://claude.ai/code) Co-Authored-By: Claude <noreply@anthropic.com>

1 parent adc144b commit e638f58Copy full SHA for e638f58

2 files changed

+664

-162

lines changed

src
- remote.test.ts
- remote.ts

2 files changed

+664

-162

lines changed

Comments

(0)

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Uh oh!

Commit e638f58

2 files changed

2 files changed

Pfad - The Proxy pFad of © 2024 Garber Painting. All rights reserved.

File tree

2 files changed

2 files changed

0 commit comments

Pfad - The Proxy pFad of © 2024 Garber Painting. All rights reserved.