test: MongoDB driver — normalizeConfig aliases and detectAuthMethod

[anandgupta42]

What does this PR do?

MongoDB driver support was added in v0.5.13 (commit abcaa1d, PR #482), but the config normalization aliases and auth method detection for MongoDB had zero test coverage. This PR adds targeted tests to close that gap.

1. normalizeConfig — MongoDB aliases in packages/drivers/src/normalize.ts (10 new tests)

The MONGODB_ALIASES map (lines 76–84) and the mongo type alias (lines 99–100) were added alongside the MongoDB driver but the existing test file (driver-normalize.test.ts) covered every other driver type except MongoDB. Without these tests, a regression in alias resolution would cause silent connection failures for users configuring MongoDB with dbt/SDK-style field names like uri, connectionString, authSource, replicaSet, etc.

New coverage includes:

• Each individual alias: connectionString, uri, url → connection_string; authSource → auth_source; replicaSet → replica_set; directConnection → direct_connection; connectTimeoutMS → connect_timeout; serverSelectionTimeoutMS → server_selection_timeout
• The mongo type alias resolves identically to mongodb
• A combined test verifying all aliases resolve correctly in a single realistic config object

2. detectAuthMethod — MongoDB auth detection in src/altimate/native/connections/registry.ts (4 new assertions)

The MongoDB-specific auth detection branch (line 229) was added but had no test assertions. Added coverage for:

• connection_string field → "connection_string"
• password field → "password" (caught by generic check at line 226)
• Bare mongodb type with no credentials → "connection_string" fallback (line 229)
• mongo type alias → same fallback behavior

Type of change

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

Issue for this PR

N/A — proactive test coverage for newly shipped MongoDB driver support

How did you verify your code works?

bun test test/altimate/driver-normalize.test.ts       # 88 pass (78 existing + 10 new)
bun test test/altimate/telemetry-safety.test.ts --test-name-pattern "detectAuthMethod"  # 2 pass (4 new assertions in existing test)

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_01U7mUcNzrnXfdtJSJxVTuQv

Summary by CodeRabbit

• Tests
  • Expanded test coverage for MongoDB configuration normalization, including alias property resolution and authentication method detection.

[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: 3f836a99-fea4-48c4-b8ed-7a8f6b9a0798

📥 Commits

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

📒 Files selected for processing (2)

• packages/opencode/test/altimate/driver-normalize.test.ts
• packages/opencode/test/altimate/telemetry-safety.test.ts

---

📝 Walkthrough

Walkthrough

This pull request adds comprehensive test coverage for MongoDB configuration normalization and authentication method detection, including alias property resolution and type alias handling without modifications to production code.

Changes

Cohort / File(s)	Summary
MongoDB Config Normalization Tests packages/opencode/test/altimate/driver-normalize.test.ts	New test suite verifying alias normalization for MongoDB configs, asserting that connection properties (connectionString, uri, url) normalize to connection_string and that configuration properties like authSource, replicaSet, directConnection, and timeout settings normalize to their snake_case equivalents with original alias fields removed.
Auth Method Detection Tests packages/opencode/test/altimate/telemetry-safety.test.ts	Extended test coverage for detectAuthMethod function with MongoDB scenarios, validating that type: "mongodb" with various auth configurations (connection_string, omitted, password) and the mongo type alias correctly resolve to appropriate auth methods.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~5 minutes

Possibly related PRs

• feat: add MongoDB driver support #482: Introduces MongoDB config alias mappings and registry changes for detectAuthMethod that are directly verified by the test additions in this PR.

Suggested labels

contributor

Poem

🐰 Config fields hop and dance,
Aliases normalize in perfect stance—
camelCase to snake_case they fly,
MongoDB's secrets safely comply! 🔐

🚥 Pre-merge checks | ✅ 3

✅ Passed checks (3 passed)

Check name	Status	Explanation
Title check	✅ Passed	The PR title accurately and concisely summarizes the main changes: adding MongoDB-specific tests for normalizeConfig aliases and detectAuthMethod.
Description check	✅ Passed	The PR description comprehensively covers all required template sections with detailed context, test coverage details, verification steps, and completed checklists.
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-1111

---

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.