test: Remove Docker tests, add comprehensive Incus coverage

[sebyx07]

Summary

• Delete Docker-specific test files (DockerBackend.test.js, DockerClient.test.js)
• Update ContainerPool tests to remove Docker backward compat references
• Update remaining test files that reference Docker to use Incus
• Update integration and E2E tests to reference Incus instead of Docker
• Add comprehensive IncusBackend unit tests (75 tests) covering:
  • Container lifecycle (create, start, stop, exec, health check)
  • Resource configuration (memory, CPU, nesting)
  • Workspace path resolution (absolute and relative)
  • User handling edge cases (root, numeric UIDs, invalid users)
  • Start timeout behavior and retry logic
  • Custom remote image parsing
  • Package installation and validation
  • Container lookup and listing
  • Error wrapping and interface compliance
• Add comprehensive IncusClient unit tests (49 tests) covering:
  • HTTP requests to Incus Unix socket (POST, PUT, GET, DELETE)
  • Async operation handling with polling
  • Exec command with stdout/stderr log retrieval
  • File push/pull operations with header validation
  • Error handling (socket not found, connection refused, timeouts, API errors)
  • URL encoding and operation ID extraction

Test plan

• All 75 IncusBackend unit tests pass
• All 49 IncusClient unit tests pass
• Full unit test suite passes (5,599 tests, 0 failures)
• ESLint + Prettier linting passes

🤖 Generated with Claude Code

Greptile Overview

Greptile Summary

Completes the Docker-to-Incus migration by removing all Docker references from tests and adding comprehensive test coverage for Incus backend components.

Key changes:

• Added 105 new tests in IncusBackend.test.js covering workspace path resolution, user handling edge cases (root, numeric UIDs, invalid users), start timeout behavior with retries, custom remote image parsing, and container lookup edge cases
• Added 158 new tests in IncusClient.test.js covering stdout/stderr log retrieval from exec operations, file push with optional uid/gid/mode headers, and error handling edge cases (missing error fields, non-Failure status with err field)
• Updated all E2E test comments to reference Incus containers instead of Docker
• Removed backward compatibility comments about Docker from tool test files
• Added export verification tests for IncusBackend and IncusClient in main index test

Confidence Score: 5/5

• This PR is safe to merge with no identified issues
• Test-only changes that add comprehensive coverage without modifying production code. All 124 new tests follow established patterns, properly mock dependencies, and verify expected behavior. Comment updates correctly reflect the Docker-to-Incus migration. The PR description indicates all 5,599 tests pass with no failures.
• No files require special attention

Important Files Changed

Filename	Overview
src/index.test.js	Added export tests for IncusBackend and IncusClient to verify they are properly exported
test/unit/execution/IncusBackend.test.js	Added 105 new lines of tests covering workspace paths, user handling, start timeout, container lookup, and custom remote image parsing
test/unit/execution/IncusClient.test.js	Added 158 new lines of tests covering stdout/stderr log retrieval, file push header handling, and error edge cases

_{Last reviewed commit: 34c769b}