Skip to content

Add comprehensive unit tests for local invoke and start implementations#87

Merged
rsonghuster merged 1 commit intomasterfrom
add-test
Sep 23, 2025
Merged

Add comprehensive unit tests for local invoke and start implementations#87
rsonghuster merged 1 commit intomasterfrom
add-test

Conversation

@rsonghuster
Copy link
Copy Markdown
Contributor

  • Added new test files for model subcommand with full coverage
  • Added complete unit tests for local invoke implementations:
    • baseLocalInvoke_test.ts with tests for all methods
    • nodejsLocalInvoke_test.ts with runtime-specific tests
    • pythonLocalInvoke_test.ts with runtime-specific tests
  • Added complete unit tests for local start implementations:
    • baseLocalStart_test.ts with tests for all methods
    • nodejsLocalStart_test.ts with runtime-specific tests
    • pythonLocalStart_test.ts with runtime-specific tests
  • Fixed timing precision issue in utils_test.ts sleep test
  • Updated function.ts to handle runtime check properly

These changes significantly improve test coverage for the local development functionality.

- Added new test files for model subcommand with full coverage
- Added complete unit tests for local invoke implementations:
  * baseLocalInvoke_test.ts with tests for all methods
  * nodejsLocalInvoke_test.ts with runtime-specific tests
  * pythonLocalInvoke_test.ts with runtime-specific tests
- Added complete unit tests for local start implementations:
  * baseLocalStart_test.ts with tests for all methods
  * nodejsLocalStart_test.ts with runtime-specific tests
  * pythonLocalStart_test.ts with runtime-specific tests
- Fixed timing precision issue in utils_test.ts sleep test
- Updated function.ts to handle runtime check properly

These changes significantly improve test coverage for the local development functionality.
@rsonghuster rsonghuster merged commit 18865d0 into master Sep 23, 2025
4 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant