Skip to content

#365 を実装する(スプリント m8e) - #492

Merged
takemi-ohama merged 1 commit into
sprint/m8efrom
feat/issue-365-m8e
Oct 15, 2026
Merged

takemi-ohama merged 1 commit into
sprint/m8efrom
feat/issue-365-m8e

Conversation

@takemi-ohama

@takemi-ohama takemi-ohama commented Oct 15, 2026 •

Copy link
Copy Markdown
Contributor

tests/editor/test_opener.py を直しました。open_editor のテストで例に使っていた作成者個人のプロジェクト名を、中立の名前 myapp-ml に置き換えています(#365)。変更はテストだけで、本体の動作は変わりません。

利用者向けの変化

  • 無し

移行の手順

  • 無し

変更の要点

テスト

ステップ exit 結果
test-limited 0 ok。範囲テスト 1 本を戦略 local-full で実行(132.6 秒)
design-tests 3 stopped。#365 には設計文書が無いため、3 つの確認はどれも該当しなかった

設計と違う点

  • 該当なし(設計文書が無い)

変更

tests/editor/test_opener.py の test_open_editor_remote_ssh_empty_ssh_host_launches_flat_uri_with_context(Remote-SSH で DEVBASE_EDITOR_SSH_HOST= を空にしたときにフラット URI で開くテスト)で、open_editor に渡す project_name・workdir と、URI の末尾のアサーションに残っていた作成者の個人のプロジェクト名を myapp-ml(workdir="/work/myapp-ml")へ置き換えた。課題に書かれていた行番号(991・999 行)は、その後の変更でずれており、実際は 1281・1289 行だった。

確かめる内容(@ssh-remote+ を含まず workdir で終わる URI・settings.context が wsl・「同名」の案内が出ない)は変えていない。#294 の置き換えの表への 1 行は別に追記済み。テストだけの変更のため CHANGELOG.md は書いていない。

手で確かめたこと

  • git grep -i で置き換えた語が追跡されているファイルに 0 件
  • env -u DEVBASE_ROOT uv run --locked pytest tests/ -q: 4402 passed, 32 skipped
  • CI と同じ lint(ruff check --select=E9,F63,F7,F82 lib): 通過
  • 固有の語の検査(python3 .github/scripts/proper_term_check.py): 既定の出所では「語の一覧が無いため飛ばした」となり、検査したとは数えていない。置き換えた語 1 語だけを入れた一時の一覧を --terms で渡して走らせ、当たり 0 件(対象 437 ファイル)を確かめた

モード: standard / 通した工程: 実装 → 完了判定 → Pull Request

🤖 Generated with Claude Code

Remote-SSH で DEVBASE_EDITOR_SSH_HOST を空にしたときのフラット URI のテストで、
project_name・workdir と URI の末尾のアサーションに作成者の個人のプロジェクト名が
残っていた。#294 の置き換えの表に足した myapp-ml へ揃える。確かめる内容は変えない。

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
@takemi-ohama
takemi-ohama marked this pull request as ready for review October 15, 2026 08:57
@takemi-ohama
takemi-ohama merged commit bd73624 into sprint/m8e Oct 15, 2026
10 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant