Skip to content

test: Dockerコンテナ起動時のREADME表示をテスト#1817

Open
Hiroshiba wants to merge 1 commit intoVOICEVOX:masterfrom
Hiroshiba:test/add-readme-display-check
Open

test: Dockerコンテナ起動時のREADME表示をテスト#1817
Hiroshiba wants to merge 1 commit intoVOICEVOX:masterfrom
Hiroshiba:test/add-readme-display-check

Conversation

@Hiroshiba
Copy link
Member

@Hiroshiba Hiroshiba commented Nov 1, 2025

内容

Dockerコンテナ起動時にREADMEが正しく表示されているかをテストするステップを追加しました。

  • コンテナのログから「利用規約」という文字列の存在を確認
  • warmup後、既存のAPIテストの前に実行

関連 Issue

#1815 (comment)

このPR関係ないですが、起動時にちゃんとREADMEが表示されるのかのテストがあったほうが良いのかもしれないですねぇ。

スクリーンショット・動画など

なし

その他

なし

@Hiroshiba Hiroshiba requested review from Copilot and tarepan November 1, 2025 06:11
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull Request Overview

This PR adds verification of README display in Docker container logs. The changes capture the container ID when starting the Docker container and add a new test step to verify that the README (specifically checking for "利用規約" - Terms of Use) appears in the container logs.

Key Changes:

  • Modified the container run step to capture and output the container ID
  • Added a new test step to verify README content appears in container logs

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@Hiroshiba Hiroshiba force-pushed the test/add-readme-display-check branch from 4c263da to 9ba73ef Compare November 1, 2025 06:14
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