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

Add: VRT(e2eテスト)を追加 #294

Open
wants to merge 4 commits into
base: main
Choose a base branch
from

Conversation

sevenc-nanashi
Copy link
Contributor

@sevenc-nanashi sevenc-nanashi commented Feb 17, 2024

image

VRTを追加します。(写真は失敗例)

Copy link
Contributor

@peacock0803sz peacock0803sz left a comment

Choose a reason for hiding this comment

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

どこかに結果の画像を出せると良さそうな

tests/vrt.spec.ts-snapshots/home-page-linux.png Outdated Show resolved Hide resolved
@sevenc-nanashi sevenc-nanashi changed the title Add: e2eテストを追加 Add: VRT(e2eテスト)を追加 Feb 17, 2024
@peacock0803sz
Copy link
Contributor

CI落ちてるのはなんでなんでしょ....?

pnpm run build
- name: Upload playwright report
Copy link
Contributor

Choose a reason for hiding this comment

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

zipでレポート上げられても見づらい気が。画像をコメントなりって可能なんでしょうか

Copy link
Contributor Author

Choose a reason for hiding this comment

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

どうなんでしょう?
そういうActionがあればいいんですが...

Copy link
Contributor

@peacock0803sz peacock0803sz Feb 20, 2024

Choose a reason for hiding this comment

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

actions/github-script で頑張るしかないか...このPRでやるのは厳しいですよね

@sevenc-nanashi
Copy link
Contributor Author

image
image

GitHub Actionsのブラウザに日本語フォントが入っていないせいでにほんごテキストを描画出来ず死亡、ですね

@peacock0803sz
Copy link
Contributor

であればGHAでテスト前にNoto Sans CJK?あたりを適当にapt installすれば良い気がしますね

GitHub Actionsのブラウザに日本語フォントが入っていないせいでにほんごテキストを描画出来ず死亡、ですね

@peacock0803sz
Copy link
Contributor

@sevenc-nanashi こちらどうしましょう。目的としてはPRなりでビルド結果の差分が出ていることを検知できれば良いと思っているのですが...

@sevenc-nanashi
Copy link
Contributor Author

VRTまで行かなくても、画面に差分が発生していたら自動で変化をPRコメントする、が良い感じになる...んですかね?

@peacock0803sz
Copy link
Contributor

が理想ですね。方法としては、画面までいかずとも静的ファイルの差分でも良いのかなと思ったりもしています

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