Skip to content

test: add TypeScript SDK utility tests#599

Open
MD-Mushfiqur123 wants to merge 2 commits into
Agent-Field:mainfrom
MD-Mushfiqur123:tests/ts-utils
Open

test: add TypeScript SDK utility tests#599
MD-Mushfiqur123 wants to merge 2 commits into
Agent-Field:mainfrom
MD-Mushfiqur123:tests/ts-utils

Conversation

@MD-Mushfiqur123
Copy link
Copy Markdown

Closes #406

Adds additional edge case tests to existing TypeScript SDK utility test files:

utils_pattern.test.ts (5 new tests):

  • Empty pattern
  • Empty value
  • Multiple wildcards
  • Dots and plus signs
  • Wildcard prefix with suffix

utils_schema.test.ts (5 new tests):

  • Nested zod objects with arrays and optionals
  • Zod enums
  • Non-object non-zod values
  • Preserves $schema on plain objects
  • Zod with default values

utils_http_agents.test.ts (2 new tests):

  • httpAgent instanceof check
  • httpsAgent instanceof check
cd sdk/typescript && npm run test:core -- tests/utils_*.test.ts
Tests  24 passed (24)

PR generated by AI agent.

@MD-Mushfiqur123 MD-Mushfiqur123 requested review from a team and AbirAbbas as code owners May 28, 2026 07:16
@CLAassistant
Copy link
Copy Markdown

CLAassistant commented May 28, 2026

CLA assistant check
Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you all sign our Contributor License Agreement before we can accept your contribution.
1 out of 2 committers have signed the CLA.

✅ MD-Mushfiqur123
❌ Developer


Developer seems not to be a GitHub user. You need a GitHub account to be able to sign the CLA. If you have already a GitHub account, please add the email address used for this commit to your account.
You have signed the CLA already but the status is still pending? Let us recheck it.

@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented May 31, 2026

Performance

SDK Memory Δ Latency Δ Tests Status
TS 345 B -1% 2.42 µs +21%

✓ No regressions detected

@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented May 31, 2026

📊 Coverage gate

Thresholds from .coverage-gate.toml: per-surface ≥ 86%, aggregate ≥ 88%, max per-surface regression ≤ 1.0 pp, max aggregate regression ≤ 0.50 pp.

Surface Current Baseline Δ
control-plane 87.50% 87.30% ↑ +0.20 pp 🟡
sdk-go 92.10% 90.70% ↑ +1.40 pp 🟢
sdk-python 93.73% 93.63% ↑ +0.10 pp 🟢
sdk-typescript 92.84% 92.56% ↑ +0.28 pp 🟢
web-ui 89.93% 90.01% ↓ -0.08 pp 🟡
aggregate 89.03% 89.01% ↑ +0.02 pp 🟡

✅ Gate passed

No surface regressed past the allowed threshold and the aggregate stayed above the floor.

@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented May 31, 2026

📐 Patch coverage gate

Threshold: 80% on lines this PR touches vs origin/main (from .coverage-gate.toml:thresholds.min_patch).

Surface Touched lines Patch coverage Status
control-plane 0 ➖ no changes
sdk-go 0 ➖ no changes
sdk-python 0 ➖ no changes
sdk-typescript 0 ➖ no changes
web-ui 0 ➖ no changes

✅ Patch gate passed

Every surface whose lines were touched by this PR has patch coverage at or above the threshold.

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.

[TS SDK] Add tests for utils (pattern, schema, httpAgents)

2 participants