-
Notifications
You must be signed in to change notification settings - Fork 313
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
スナップショットテストが更新されない(ことがある) #1724
Comments
そもそもWindows以外ではshouldUpdateSnapshotsをfalseにするのがいいような気がしました。 |
なるほどです。 個人的には、OS依存の描画の差もあるので、やっぱマルチOSでVRTしたい気持ちがちょっとあります。 今気づいたのですが、「実行のたびに謎の描画差がある」現状の原因、これかも……? |
こちらが実行できないとテストを更新できないため、優先度最大にさせていただきます。 |
でいろいろやっていたのですが、macでflakyになるのはそういうものという感じでした。 mac省くなら別にlinuxとwinは表示が一緒なので、両方テストするのは意味ない気がします。 |
不具合の内容
でVRTが導入されました!
コミットメッセージに
[update snapshots]
と書くとスナップショットを更新できるのですが、うまいこと更新されなかったのでissueを作りました。こちらがログです。(関係ないのがエラーで落ちてるのはopenapiのAPIが減ったからです)
その他
成果物を見る感じ、多分Windows用のスナップショットを更新するのはWindowsだけで、ubuntuとかは更新されてないスナップショットをアップロードしていて、マージする時に衝突して更新されたりされなかったりするんだと思います。
パッと思いつく解決策は、まあOSに従った成果物だけアップロードするようにするのが良いのかなと思いました。
(Windowsは多分
*-win32.png
?)もっと良い解決策があるかもと思ってとりあえずコメント待ちです!
The text was updated successfully, but these errors were encountered: