test: openbao_root fixture が DEVBASE_ROOT を setenv しないため、使う側が忘れると実環境を読む

[takemi-ohama]

何を見つけたか

tests/conftest.py の openbao_root fixture（tests/conftest.py:369）は tmp のリポジトリを作るが、DEVBASE_ROOT を setenv しない。os.environ から DEVBASE_ROOT を読む経路（lib/devbase/commands/container.py の _check_group_consistency / _grouped_store など）をこの fixture で試すテストは、各自で monkeypatch.setenv('DEVBASE_ROOT', ...) する必要がある。

実行環境によってはシェルが DEVBASE_ROOT を持ち、pytest がそれを継承するため、忘れると実環境の devbase の設定を読む。ホストの Mac のシェルは持ち、dev コンテナの中は持たない。環境で再現したりしなかったりする。PLAN62（#185）のテストでは明示的に setenv して回避した。

実環境を読むことの確認

外側に secrets/backend.yml（layout: group）を持つ偽の root を用意し、openbao_root だけを使うテストを DEVBASE_ROOT=<偽の root> で実行すると、_grouped_store() が tmp ではなく偽の root を指す。

E  assert PosixPath('.../fakeroot') == PosixPath('/tmp/pytest-of-ubuntu/pytest-0/test_...0')
E  _grouped_store().root = PosixPath('.../fakeroot')

今は赤が出ない

シェルの DEVBASE_ROOT を変えても、テスト全体の結果は変わらない。

実行	結果
env -u DEVBASE_ROOT pytest -q	1 failed, 2861 passed, 1 skipped
DEVBASE_ROOT=<偽の root> pytest -q	1 failed, 2861 passed, 1 skipped

1 failed は tests/snapshot/test_restore_incremental.py::test_a_generation_with_swapped_directories_restores_completely で、本件とは関わらない。

穴は完全に潜在している。 今あるテストは自前で setenv しているため赤にならず、新しく書いたテストが setenv を忘れたときに初めて実環境を読む。見つかるのは、実環境の projects/ や secrets/backend.yml を変えた誰かの手元で、無関係のテストが落ちたときである。

修正レイヤー

現象レイヤー: tests/conftest.py の openbao_root fixture。

修正レイヤー: pytest のセッション全体の環境の隔離（tests/conftest.py の autouse fixture）。

fixture 1 つに setenv を足しても、同じ穴は他にも開いたままになる。実際、

• tests/ には monkeypatch.setenv('DEVBASE_ROOT', ...) を各テストが自分で書いている箇所が 32 か所 / 15 ファイルある
• DEVBASE_ROOT を setenv しない tmp の root を作る fixture は openbao_root だけではない（tests/env/test_backend_config.py の root、tests/env/test_runtime.py の root、tests/env/test_secret_store_backend.py の root、tests/plugin/test_migrator.py の devbase_root、tests/commands/test_project_migrate_config.py の devbase_root など）
• tests/cli/conftest.py は同じ危険（継承した DEVBASE_ROOT で実環境の projects/ を見る）を CLI のテスト向けに別途 docstring と wrapper の代入で塞いでいる
• tests/conftest.py の autouse fixture は _release_shared_secret_store（tests/conftest.py:322）の 1 つだけで、環境変数は隔離していない

pyproject.toml の [tool.pytest.ini_options] にも環境変数の設定は無く、pytest は素の環境を継承する。

採る手: 統合（consolidate_duplication）。tests/conftest.py に autouse fixture を置き、DEVBASE_ROOT をセッション全体で固定する。各テストの明示の setenv はそのまま後勝ちで働くため、既存のテストは書き換えなくてよい。

実装の選択肢

案	内容	既存テストへの影響
A	autouse で monkeypatch.delenv('DEVBASE_ROOT', raising=False)	無し
B	autouse で tmp のディレクトリへ setenv	無し

B を推す。 _require_devbase_root の「未設定」分岐を試すテストが、その場で明示的に delenv して書けるようになる。A だと未設定が既定になり、設定済みの分岐を試す側が毎回 setenv を書くことになって現状と変わらない。

完了の目安: autouse で DEVBASE_ROOT を隔離した状態で、テスト全体が 1 failed（上記の既存の失敗）, 2861 passed, 1 skipped のまま変わらないこと。

由来

PLAN62（#185 / PR #206）の完了判定。範囲外と判断した。

進行

モード: light / 作業ツリー: .worktrees/feature/v3.7.0-test-env-isolation

• 要求と受け入れ条件 — 2026-09-22 06:15
• 作業場所の用意 — 2026-09-22 06:17
• 設計
• 素材の収集と出典の確定
• ドキュメント再構成
• ドキュメントレビュー
• 計画
• 実装 — 2026-09-22 06:18
• 構造改善
• 実装レビュー — 2026-09-22 06:33
• 完了判定 — 2026-09-22 06:45
• Pull Request — 2026-09-22 06:17
• 確定仕様化
• 後片付け
• 配布
• 体裁レビュー
• リリース後テスト
• 振り返り

[takemi-ohama]

要求と受け入れ条件

モード light（tests/ だけを触り、本番の振る舞いも本番コードの構造も変えない）。

目的

pytest が実行したシェルから DEVBASE_ROOT を継承するため、tmp の root を作る fixture を使う
テストが自前で setenv を忘れると、実環境の devbase の設定（projects/ と
secrets/backend.yml）を読む。この穴を fixture 1 つではなくセッション全体で塞ぐ。

前提

• 採るのは本文の 案 B（autouse で tmp のディレクトリへ setenv）。案 A（delenv）ではない。
  理由は本文のとおり、_require_devbase_root の「未設定」分岐を試す側がその場で delenv
  して書けるようにするため
• 既存の 32 か所 / 15 ファイルの monkeypatch.setenv('DEVBASE_ROOT', ...) は書き換えない
  （autouse より後に走り、後勝ちでそのまま働く）
• 隔離する環境変数は DEVBASE_ROOT 1 つに限る。他の変数（DEVBASE_ACCOUNT_GROUP
COMPOSE_PROJECT_NAME DEVBASE_AGE_KEY_FILE XDG_CONFIG_HOME EDITOR SHELL
DEVBASE_OPEN_INDEX DEV_SERVICE_NAME PWD）へは広げず、別課題として起票する

受け入れ条件

• AC1: tests/conftest.py に autouse の fixture があり、テストが何も設定しなければ
  os.environ['DEVBASE_ROOT'] が pytest の tmp の下のディレクトリを指す
  （実環境の devbase の root ではない）
• AC2: AC1 のディレクトリは実在し、secrets/backend.yml も projects/ も持たない
  （空の root として振る舞う）
• AC3: テストの側の monkeypatch.setenv('DEVBASE_ROOT', ...) が後勝ちで働く。既存の
  32 か所を 1 行も書き換えずに済む
• AC4: テストの側の monkeypatch.delenv('DEVBASE_ROOT', raising=False) が働く
  （未設定の分岐を試す道が残る）
• AC5: env -u DEVBASE_ROOT uv run pytest -q の結果が
  1 failed, 2861 passed, 1 skipped である（1 failed は既存の
  tests/snapshot/test_restore_incremental.py::test_a_generation_with_swapped_directories_restores_completely）
• AC6: DEVBASE_ROOT=<外側の偽の root> uv run pytest -q の結果が AC5 と同一である。
  偽の root は secrets/backend.yml（version: 2 / layout: group）と projects/ を持つ
• AC7: 修正後の差分は tests/ の中だけである（lib/ bin/ containers/ を触らない）

検証手段

条件	コマンド
AC1〜AC4	差分の読みと、AC5 / AC6 が同一であること
AC5	env -u DEVBASE_ROOT uv run pytest -q
AC6	DEVBASE_ROOT=<偽の root> uv run pytest -q
AC7	git diff --stat

release/v3.7.0 を base にした Pull Request では CI が 1 件も動かない（#216）。上の 2 つは
手元で実行し、出力を Pull Request の Test plan へ載せる。

含む / 含まない

含む	tests/conftest.py への autouse fixture の追加
含まない	既存 32 か所の setenv の整理、DEVBASE_ROOT 以外の環境変数の隔離、tests/cli/conftest.py の wrapper 側の仕組みの変更、pyproject.toml の env 設定の導入

境界

• 常に行う: 変更後にテスト全体を 2 条件で実行する
• 行わない: 既存テストの書き換え、新しいテストの追加（完了の目安が件数を
  2861 passed のままと定めているため、fixture 自体を確かめるテストも足さない）、
  lib/ への変更

[takemi-ohama]

設計の工程は通さない。/ndf:design の「触る領域を決める」の表のうち「すべての変更」以外の領域が 1 つも当たらないため（tests/conftest.py に autouse fixture を 1 つ足すだけで、構成要素の配置・依存・反映先も、型・クラスも、永続データも、呼び出される約束も、画面も、非機能の条件も変わらない。lib/ は触らない）。