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

Fix GitHub Actions: move env, no need bundler #360

Merged
merged 1 commit into from
Apr 5, 2022

Conversation

takahashim
Copy link
Collaborator

@takahashim takahashim commented Apr 4, 2022

🎩 What? Why?

テストの実行結果ログが見えにくかったため、GitHub Actionsのrails-testを整理します。

bundle installはbundler-cache: trueでは不要だそうなので削除するのと、環境変数を個別のstep内で指定するのではなくjob全体で共有するように修正しています。

📋 Subtasks

  • Add CHANGELOG upgrade notes, if required
  • If there's a new public field, add it to GraphQL API
  • Add documentation regarding the feature
  • Add/modify seeds
  • Add tests
  • Another subtask

@takahashim
Copy link
Collaborator Author

テストは当初失敗していましたが、再実行させたところ通ったようでした。
どうもテスト環境でのlocaleの関係で、redirectさせるときにうしろに ?locale=en がついてしまうことがあるようでした。
何度も起こるようであればテストコードでurl queryを無視するような修正をします(が、意味があったりバグだったりする場合もあるため、問題なければそのままにしておきたい気持ちもあります…)。

@takahashim
Copy link
Collaborator Author

うーん、やっぱり失敗する時は失敗しますね… https://github.com/codeforjapan/decidim-cfj/runs/5814787081?check_suite_focus=true

@takahashim
Copy link
Collaborator Author

テストの件は #3598b0f3cf で対応しています。
必要であれば他のブランチにもcherry pickにて取り込みます。

@ayuki-joto
Copy link
Collaborator

ありがとうございます!

@ayuki-joto ayuki-joto merged commit 8289c48 into develop Apr 5, 2022
@takahashim takahashim deleted the fix-ga-rails-test branch April 11, 2022 07:39
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.

2 participants