Skip to content

Conversation

laktek
Copy link
Contributor

@laktek laktek commented Jul 2, 2025

  • Updated invoke method to include response object in both success and error cases
  • Added test cases to verify response object is present in function invocations
  • Ensures compliance with FunctionsResponse type definition

Fixes #65 . Supersedes #98

- Updated invoke method to include response object in both success and error cases
- Added test cases to verify response object is present in function invocations
- Ensures compliance with FunctionsResponse type definition
@laktek laktek requested a review from grdsdev July 2, 2025 05:31
@coveralls
Copy link

Pull Request Test Coverage Report for Build 16016917499

Details

  • 5 of 5 (100.0%) changed or added relevant lines in 2 files are covered.
  • No unchanged relevant lines lost coverage.
  • Overall coverage increased (+0.08%) to 92.391%

Totals Coverage Status
Change from base Build 15999277351: 0.08%
Covered Lines: 223
Relevant Lines: 236

💛 - Coveralls

@laktek laktek merged commit 99cda28 into main Jul 2, 2025
3 checks passed
@laktek laktek deleted the fix-functions-response-type branch July 2, 2025 09:14
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.

Negative tests for edge functions with Deno
3 participants