test: warehouse connections — MongoDB auth + env var loading

[anandgupta42]

What does this PR do?

1. detectAuthMethod — MongoDB branch (src/altimate/native/connections/registry.ts) (2 new tests)

MongoDB driver support was added this week (commit abcaa1d, PR #482). The detectAuthMethod function gained a new mongodb/mongo branch that returns "connection_string" as the fallback auth method for MongoDB configs without a password — distinct from other driver types which fall through to "unknown". Zero tests existed for this new branch. Without these tests, the MongoDB-specific line could be deleted and no test would fail, silently mislabeling telemetry for all MongoDB users.

New coverage includes:

• { type: "mongodb", host: "..." } (no password) returns "connection_string" not "unknown"
• { type: "mongo", host: "..." } (alias) returns "connection_string" — verifies the || t === "mongo" half of the OR condition

2. loadFromEnv — ALTIMATE_CODE_CONN_* env var parsing (src/altimate/native/connections/registry.ts) (3 new tests)

CI/CD users configure warehouse connections via ALTIMATE_CODE_CONN_* environment variables. The loadFromEnv() function (lines 64–82) parses these into connection configs during load(). This code path had zero direct test coverage. If JSON parsing broke silently or the type field guard were removed, CI connections would vanish with no error.

New coverage includes:

• Valid JSON env var is parsed and available via getConfig() (includes lowercase name conversion)
• Invalid JSON env var is silently skipped (no crash, no config entry)
• JSON missing the required type field is rejected

Type of change

• New feature (non-breaking change which adds functionality)

Issue for this PR

N/A — proactive test coverage for newly added MongoDB driver support and untested CI/CD configuration path.

How did you verify your code works?

bun test test/altimate/connections.test.ts           # 42 pass
bun test test/altimate/warehouse-telemetry.test.ts   # 43 pass

Checklist

• I have added tests that prove my fix is effective or that my feature works
• New and existing unit tests pass locally with my changes

https://claude.ai/code/session_01QvuBUD1uLcEPSu6EugNcyf

Summary by CodeRabbit

• Tests
  • Enhanced test coverage for environment variable-based configuration loading and authentication method detection logic.

[claude]

Claude Code Review

This repository is configured for manual code reviews. Comment @claude review to trigger a review and subscribe this PR to future pushes, or @claude review once for a one-time review.

_{Tip: disable this comment in your organization's Code Review settings.}

[coderabbitai]

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info

⚙️ Run configuration

Configuration used: Repository UI

Review profile: CHILL

Plan: Pro

Run ID: adb2ddd9-0902-42b5-b3ee-edb2d2c6aa9b

📥 Commits

Reviewing files that changed from the base of the PR and between abcaa1d and ff293fd.

📒 Files selected for processing (2)

• packages/opencode/test/altimate/connections.test.ts
• packages/opencode/test/altimate/warehouse-telemetry.test.ts

---

📝 Walkthrough

Walkthrough

New test suites added to verify connection registry functionality: one tests environment variable loading into the registry with valid/invalid payloads, another tests MongoDB authentication method detection for fallback behavior. Both use proper isolation via beforeEach resets and finally cleanup blocks.

Changes

Cohort / File(s)	Summary
Connection Registry Tests packages/opencode/test/altimate/connections.test.ts	Added ConnectionRegistry: loadFromEnv test suite validating Registry.load() behavior with ALTIMATE_CODE_CONN_* environment variables, including positive paths (valid JSON parsing, field preservation) and negative paths (invalid JSON, missing required type field).
Warehouse Telemetry Tests packages/opencode/test/altimate/warehouse-telemetry.test.ts	Added two tests to warehouse telemetry: detectAuthMethod suite verifying Registry.detectAuthMethod returns "connection_string" for MongoDB driver types when both password and connection_string fields are absent.

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~12 minutes

Possibly related PRs

• feat: add MongoDB driver support #482: The added tests for Registry.detectAuthMethod verify behavior changes made to packages/opencode/src/altimate/native/connections/registry.ts in that PR.

Suggested labels

contributor

Poem

🐰 Through environments and registries deep,
These tests hop paths both steep and neat,
MongoDB's auth finds its way,
With fallbacks sure to save the day! 🔐

🚥 Pre-merge checks | ✅ 3

✅ Passed checks (3 passed)

Check name	Status	Explanation
Title check	✅ Passed	The title directly reflects the primary changes: adding tests for MongoDB authentication detection and environment variable loading for warehouse connections.
Description check	✅ Passed	The description provides comprehensive context, rationale, and test coverage for both changes, though it deviates from the template structure by not using the specified section headings.
Docstring Coverage	✅ Passed	No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

_{✏️ Tip: You can configure your own custom pre-merge checks in the settings.}

✨ Finishing Touches

📝 Generate docstrings

• Create stacked PR
• Commit on current branch

🧪 Generate unit tests (beta)

• Create PR with unit tests
• Commit unit tests in branch test/hourly-20260327-0413

---

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

• X
• Mastodon
• Reddit
• LinkedIn

_{Comment @coderabbitai help to get the list of available commands and usage tips.}

[anandgupta42]

Consolidated into #514. This PR's test coverage was either already on main or has been included in the consolidated PR.