test: MongoDB normalization + detectAuthMethod coverage

[anandgupta42]

What does this PR do?

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

MongoDB driver support was just added in #482, but the config normalization layer had zero test coverage for MongoDB-specific field aliases. This is a P0 gap: users connecting via uri, url, or connectionString (the most common MongoDB connection patterns from Atlas docs, Mongoose, and official drivers) would silently get no connection string passed to the driver. New coverage includes:

• connectionString → connection_string alias resolution
• uri → connection_string alias resolution
• url → connection_string alias resolution (mongodb+srv URIs)
• First-value-wins: canonical connection_string takes precedence over uri
• mongo type alias resolves MongoDB-specific fields (authSource, uri)
• authSource → auth_source camelCase alias

2. detectAuthMethod — packages/opencode/src/altimate/native/connections/registry.ts (14 new tests)

This function feeds the auth_method field in warehouse_connect telemetry events. It had zero test coverage. Wrong values corrupt warehouse analytics and make it impossible to debug connection issues by auth type. New coverage includes:

• connection_string, key_pair (both snake_case and camelCase variants), sso (EXTERNALBROWSER + Okta URL), oauth, token, password detection
• File-based auth for DuckDB and SQLite ("file")
• MongoDB-specific default ("connection_string" when no password)
• MongoDB with password returns "password"
• Null/undefined guard returns "unknown"
• Empty config with no identifiable auth returns "unknown"

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 (#482)

How did you verify your code works?

bun test test/altimate/driver-normalize.test.ts   # 84 pass (78 existing + 6 new)
bun test test/altimate/connections.test.ts         # 53 pass (39 existing + 14 new)

Marker check passed: bun run script/upstream/analyze.ts --markers --base main --strict

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

Summary by CodeRabbit

• Tests
  • Enhanced test coverage for database authentication method detection across multiple connection types
  • Added validation tests for MongoDB connection configuration normalization

[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: 69902eab-b335-4452-a5dc-14c739c2478f

📥 Commits

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

📒 Files selected for processing (2)

• packages/opencode/test/altimate/connections.test.ts
• packages/opencode/test/altimate/driver-normalize.test.ts

---

📝 Walkthrough

Walkthrough

This PR adds comprehensive test coverage for authentication method detection and MongoDB connection configuration alias normalization. Tests validate proper handling of multiple authentication types (connection_string, key_pair, sso, oauth, token, password, file) and MongoDB-specific field alias mappings.

Changes

Cohort / File(s)	Summary
Authentication Detection Tests packages/opencode/test/altimate/connections.test.ts	Tests the detectAuthMethod function with assertions covering authentication method identification across multiple types: connection_string, key_pair, sso, oauth, token, password, file, and unknown for various database configs (Postgres, MongoDB, Snowflake, Databricks, DuckDB, SQLite).
MongoDB Configuration Normalization Tests packages/opencode/test/altimate/driver-normalize.test.ts	Tests MongoDB-specific alias normalization, validating that alias fields (connectionString, uri, url) map to canonical connection_string, that canonical fields take precedence, and that auth-related aliases (authSource) are normalized to snake_case equivalents (auth_source).

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~12 minutes

Possibly related PRs

• Altimate AI/altimate-code#482: Introduces MongoDB normalization and connection-registry logic that these tests validate, including alias mappings and authentication method detection.

Suggested labels

contributor

Poem

🐰 Hop hop, tests compile true,
Auth methods old and new detected through,
MongoDB aliases normalized with care,
Coverage spreads—zero doubts in the air! ✨

🚥 Pre-merge checks | ✅ 3

✅ Passed checks (3 passed)

Check name	Status	Explanation
Title check	✅ Passed	The pull request title accurately and concisely summarizes the main changes: adding test coverage for MongoDB normalization and detectAuthMethod functionality.
Description check	✅ Passed	The pull request description comprehensively covers all required sections: clear summary of changes with detailed rationale, test plan with verification steps, and completed checklist.
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-20260326-2315

---

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 #498. Tests deduplicated and merged into a single PR.