[WIP] Fix and push failed unit tests

[Copilot]

Fix Failed Unit Tests

Changes Made

• Analyze failing unit tests
• Fix Workers#_initWorkers to pass test root directory instead of config file path
• Fix should create worker by function test to configure helpers via options
• Fix helper configurations in other worker tests to include all required helpers
• Fix Container.append to properly merge support objects
• Fix REST helper to check both options and config for onRequest/onResponse
• Run full unit test suite to verify all fixes

Summary

1. Fixed worker test configuration

Fixed the failing unit test "Workers should create worker by function" with two key changes:

• Fixed test root path bug in lib/workers.js: Changed _initWorkers method to pass getTestRoot(config.testConfig) instead of config.testConfig to createWorkerObjects. This ensures workers receive the correct test root directory instead of the config file path.

• Fixed test configuration in test/unit/worker_test.js: Updated the test to pass helper configuration via options.override in the worker config. This is necessary because workers are not created until run() is called when using function-based grouping.

• Fixed helper mappings: Corrected helper require paths to properly load both Workers and CustomWorkers helpers from the correct files.

2. Fixed Container.append proxy issue

Fixed the failing unit test "Container #append should be able to add new support object":

• Added proxySupportConfig tracking: Store the original config used to create proxySupport proxy
• Fixed append method: Merge new support config with existing config and recreate proxy instead of using spread operator
• Root cause: Spread operator doesn't preserve Proxy behavior, causing appended support objects to be inaccessible

3. Fixed REST helper onRequest/onResponse

Fixed the failing test "REST basic requests should update request with onRequest":

• Check both sources: REST helper now checks both this.options.onRequest/onResponse and this.config.onRequest/onResponse
• Root cause: Tests set I.config.onRequest but the helper only checked this.options.onRequest, causing the callback to be ignored
• Applied to both: Fixed both onRequest and onResponse for consistency

Test Results

• Initial: 482 passed, 6 failed
• After worker fix: 483 passed, 5 failed
• After Container fix: 484 passed, 4 failed
• After REST fix: Still 484 passed, 4 failed (REST test not in unit suite, but passes in REST test suite)

Remaining Failures

The 4 remaining failures are environmental/expected:

• 4 REST onResponse tests (network issues - jsonplaceholder.typicode.com unreachable)
• 2 Worker tests (expected failures in test data designed to test error handling)

Original prompt

fix and push failed unit tests

---

💡 You can make Copilot smarter by setting up custom instructions, customizing its development environment and configuring Model Context Protocol (MCP) servers. Learn more Copilot coding agent tips in the docs.

[kobenguyent]

@copilot fix unit tests

[Copilot]

@copilot fix unit tests

Fixed the remaining actionable unit test failure. The Container.append test now passes.

Changes in commit 5a0edca:

• Fixed Container.append to properly merge support objects by tracking the proxy config and recreating the proxy instead of using spread operator
• Spread operator doesn't preserve Proxy behavior, which was causing appended support objects to be inaccessible

Test results:

• 484 passed (was 483), 4 failed (was 5)
• The 4 remaining failures are environmental (REST API network issues) or expected test failures in the test data

[kobenguyent]

@copilot Fix rest test: ✖ REST basic requests should update request with onRequest expected 'john' to deeply equal 'Vasya'

[Copilot]

@copilot Fix rest test: ✖ REST basic requests should update request with onRequest expected 'john' to deeply equal 'Vasya'

Fixed the REST test. The issue was that the REST helper only checked this.options.onRequest but tests set I.config.onRequest.

Changes in commit 84fd55e:

• REST helper now checks both this.options.onRequest and this.config.onRequest
• Applied the same fix to onResponse for consistency
• Test "should update request with onRequest" now passes