Skip to content

test(core): cover operator env path resolution - #1431

Open
seonghobae wants to merge 1 commit into
developfrom
codex/pr1285-env-paths-clean
Open

test(core): cover operator env path resolution#1431
seonghobae wants to merge 1 commit into
developfrom
codex/pr1285-env-paths-clean

Conversation

@seonghobae

@seonghobae seonghobae commented Aug 20, 2026

Copy link
Copy Markdown
Contributor

Scope

Add focused contract coverage for operator_env_file_paths, including default ordered paths, home expansion, custom paths, and the ~ boundary. Production behavior is unchanged.

This is the clean replacement for #1285's valid test slice; the original branch contained unrelated repository-wide deletions and is not merge-safe.

Verification

  • PYTHONPATH=. python3 -m pytest -q backend/tests/test_env_paths.py
  • ruff check backend/tests/test_env_paths.py
  • git diff --check

Open in Devin Review

@coderabbitai

coderabbitai Bot commented Aug 20, 2026

Copy link
Copy Markdown
Contributor

Warning

Review limit reached

@seonghobae, you've reached your PR review limit, so we couldn't start this review.

Next review available in: 40 minutes

Limit details: You’ve used the included review currently available.

You've used all free OSS reviews for now. Wait for the free limit to reset to keep reviewing this public repository.

How can I continue?

Wait for the limit to reset, then comment @coderabbitai review or push new commits to the PR.

An organization admin can change what happens after included review limits in Billing.

How do review limits work?

CodeRabbit enforces per-developer PR review limits within each organization.

For paid Pro and Pro+ reviews, CodeRabbit uses a developer's included PR review attempts over the past 7 days to set the current hourly allowance. At typical activity levels, the full plan allowance applies. Higher sustained activity can lower the allowance until earlier attempts leave the 7-day window.

Please refer docs for additional details.

Review details
⚙️ Run configuration

Configuration used: Path: .coderabbit.yaml

Review profile: CHILL

Plan: Pro Plus

Run ID: c18de1b6-b47a-44e9-bf39-94b919e589af

📥 Commits

Reviewing files that changed from the base of the PR and between 81c1056 and e058f8c.

📒 Files selected for processing (1)
  • backend/tests/test_env_paths.py

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

Comment @coderabbitai help to get the list of available commands.

@devin-ai-integration devin-ai-integration Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

✅ Devin Review: No Issues Found

Devin Review analyzed this PR and found no bugs or issues to report.

Open in Devin Review

@seonghobae

Copy link
Copy Markdown
Contributor Author

Current-head review completed for 10446594.

The added tests exercise both default and custom operator env-file path resolution under an isolated HOME, including the existing configured path contract. No production behavior change is included in this test-only PR.

Verification: python -m pytest backend/tests/test_env_paths.py -q → 5 passed; Ruff and git diff --check passed.

@seonghobae
seonghobae enabled auto-merge (squash) August 20, 2026 18:11
@seonghobae
seonghobae force-pushed the codex/pr1285-env-paths-clean branch from 1044659 to e058f8c Compare August 21, 2026 02:48
@opencode-agent opencode-agent Bot added priority: medium Normal-priority or P2 work status: needs-review Open pull request requiring current-head review or checks type: test Test coverage, fixtures, fuzzing, or validation labels Aug 22, 2026
@opencode-agent
opencode-agent Bot disabled auto-merge August 31, 2026 06:43
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

maintenance priority: medium Normal-priority or P2 work status: needs-review Open pull request requiring current-head review or checks type: test Test coverage, fixtures, fuzzing, or validation

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant