Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

macOSのみ、e2eテストを実行しない #2006

Merged
merged 7 commits into from
Apr 25, 2024

Conversation

weweweok
Copy link
Contributor

内容

2024年4月25日現在、github actionsにて最新バージョンのmacOSでe2eテストが落ちるようです。

https://discord.com/channels/879570910208733277/893889888208977960/1232870129809883156

応急措置としてテストをスキップします。

関連 Issue

ref #2003

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

その他

@weweweok weweweok requested a review from a team as a code owner April 25, 2024 02:46
@weweweok weweweok requested review from y-chan and removed request for a team April 25, 2024 02:46
Copy link
Member

@Hiroshiba Hiroshiba left a comment

Choose a reason for hiding this comment

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

うおお!!ありがとうございます!!!

mac全部でテストを外すのは問題に気付けないかもなので、もし可能だったら落ちてるテストだけskipするとか可能でしょうか 🙇 🙇 🙇
こんな感じで特定テストの特定OSだけスキップできます!

test.skip(process.platform !== "win32", "Windows以外のためスキップします");

今のところ落ちてるのは「再生ボタンを押して再生できる」と「アップデートが通知されたりスキップしたりできる」の2つだけかなと。
ファイル全体検索すれば該当テストが見つかると思います。
もしよければ・・・!!! 🙇

@weweweok weweweok changed the title add: macosのみe2eテストを実行しない macOSのみ、e2eテストを実行しない Apr 25, 2024
@weweweok
Copy link
Contributor Author

weweweok commented Apr 25, 2024

mac全部でテストを外すのは問題に気付けないかもなので、もし可能だったら落ちてるテストだけskipするとか可能でしょうか

こちらのほうが、色々問題が合った時に対応しやすそうです!

該当箇所の範囲だけ、skipを追加しました。

Copy link
Member

@Hiroshiba Hiroshiba left a comment

Choose a reason for hiding this comment

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

LGTM!!

すみません、助かりました!
issueを作ったのでそのリンクも追加するコミットを足させていただきます!

@Hiroshiba Hiroshiba merged commit 177ecee into VOICEVOX:main Apr 25, 2024
9 checks passed
sevenc-nanashi pushed a commit to sevenc-nanashi/voicevox that referenced this pull request Apr 28, 2024
* add: macosのみe2eテストを実行しない

* fix: 誤字修正

* add: 特定のテストケースのみスキップさせる

* delete: 特定のテストケースのみスキップできればよさそう

* issueへのリンクを追加

* フォーマッターを間違えたのを修正

* 修正漏れ

---------

Co-authored-by: Hiroshiba <hihokaruta@gmail.com>
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.

None yet

2 participants