test: Stage 2 unit coverage — split convertors/csvConverter + AzureResourceMetadata tests

[bk201-]

Summary

Stage 2 of the unit test coverage plan: split large utility files into focused modules (barrel pattern keeps public import paths unchanged) and add per-module unit tests.

Changes

src/utils/convertors (was convertors.ts, 663 lines)

Split into focused modules behind a barrel (index.ts), so all existing import sites resolve unchanged:

• types.ts — shared types and constants
• json.ts — queryResultToJSON
• table.ts — queryResultToTable + table-building helpers
• tree.ts — queryResultToTree + tree-building helpers
• metrics.ts — queryMetricsToTable, indexMetricsToTableItem, queryMetricsToJSON

Tests split per module with shared testFixtures.ts: json.test.ts, table.test.ts, tree.test.ts, metrics.test.ts (23 tests).

src/utils/csvConverter (was csvConverter.ts)

Split into:

• escape.ts — getCsvSeparator, escapeCsvValue
• metrics.ts — queryMetricsToCsv
• table.ts — queryResultToCsv

Tests: escape.test.ts, metrics.test.ts, table.test.ts (10 tests).

src/cosmosdb/AzureResourceMetadata.ts

Added AzureResourceMetadata.test.ts covering the documentEndpoint and isServerless getters (6 tests). No production changes — uses a test subclass to access the protected constructor.

Out of scope

• Migration files — owned by another contributor.
• Theme utils (src/webviews/theme/utils/) — verbatim vendored code from CSSWG CSS Color 4 / ThreeJS / Fluent Blocks; testing third-party reference implementations has low value.

Validation

• npm run l10n — clean
• npm run prettier-fix — clean
• npm run lint — 0 errors
• vitest — 39 tests passing

[github-actions]

🎭 E2E Tests (Playwright + VS Code)

Commit: d95f5bd
Pull Request: #3151 test: Stage 2 unit coverage — split convertors/csvConverter + AzureResourceMetadata tests

🧪 Result

• E2E Tests: ✅ success

📥 Artifacts (run)

• e2e-results-1 — 202 B
• e2e-html-report-1 — 198.8 KB

Tip: the HTML report artifact contains a self-contained Playwright report.
Download the zip, extract, and open index.html — or run
npx playwright show-report <extracted-dir> for the interactive view.

[github-code-quality]

Code Coverage Overview

Languages: TypeScript

TypeScript / code-coverage/vitest

The overall coverage in the dev/dshilov/unit-tes... branch is 82%. The coverage in the main branch is 73%.

Show a code coverage summary of the most impacted files.

File	main 89d42bc	dev/dshilov/unit-tes... 525bf60	+/-
src/chat/Cosmos...tionsService.ts	53%	72%	+19%
packages/nosql-...ticsProvider.ts	38%	96%	+58%
packages/nosql-...ticsProvider.ts	29%	90%	+61%
packages/nosql-...eryDecorator.ts	5%	83%	+78%
packages/nosql-...streamParser.ts	5%	88%	+83%
packages/nosql-...eryDecorator.ts	0%	86%	+86%
src/services/Se...tingsService.ts	0%	94%	+94%
src/utils/convertors/tree.ts	0%	95%	+95%
src/utils/convertors/table.ts	0%	96%	+96%
src/utils/conve...tors/metrics.ts	0%	100%	+100%

---

_{Updated June 12, 2026 14:44 UTC
Code Coverage is in Public Preview. Learn more and provide us with your feedback.}

[github-actions]

🔨 Build, Lint & Test

🔗 Source

• Commit: 525bf60
• Pull Request: #3151 test: Stage 2 unit coverage — split convertors/csvConverter + AzureResourceMetadata tests

📦 Package Information

• Version: 0.35.1
• Preview: True
• VSIX File: vscode-cosmosdb-0.35.1.vsix
• VSIX Size: 3.34 MB
• Artifact: dev-dshilov-unit-tests-stage2-0.35.1-525bf60.zip

🧪 Test Results

• Unit Tests: ✅ success
• Integration Tests (extension host): ✅ success

📥 Artifacts (run)

• dev-dshilov-unit-tests-stage2-0.35.1-525bf60 — 3.34 MB

✅ Build Status

Build and local tests passed. See sibling comments below for E2E and NoSQL integration results.

[github-actions]

🔬 NoSQL language-service integration

Commit: 525bf60
Pull Request: #3151 test: Stage 2 unit coverage — split convertors/csvConverter + AzureResourceMetadata tests

🧪 Result

• NoSQL integration tests: ✅ success

📥 Artifacts (run)

• nosql-junit-report-1 — 4.5 KB