langfuse-js icon indicating copy to clipboard operation
langfuse-js copied to clipboard

chore: add tests for query params

Open maxdeichmann opened this issue 1 year ago • 1 comments

[!IMPORTANT] Add tests and modify encodeQueryParams to handle special characters and arrays, and comment out a test in langfuse-integration-node.spec.ts.

  • Tests:
    • Add tests for encodeQueryParams in utils.spec.ts to handle special characters, arrays, and null/undefined values.
  • Functions:
    • Modify encodeQueryParams in utils.ts to encode special characters and arrays correctly.
  • Misc:
    • Comment out create span test in langfuse-integration-node.spec.ts.

This description was created by Ellipsis for dfedb379510e99d43ce5ad08354c8c3b92a35958. It will automatically update as commits are pushed.

maxdeichmann avatar Oct 09 '24 00:10 maxdeichmann

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Updated (UTC)
langfuse-js ✅ Ready (Inspect) Visit Preview Oct 9, 2024 0:21am

vercel[bot] avatar Oct 09 '24 00:10 vercel[bot]