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
[S2JUnit4] @Parameters使用時に@BeforeClass,@AfterClassがクラスで1度だけ実行されるように修正しました #7
Conversation
コミットログのS2JUnitをS2JUnit4に(できればコミットログの先頭に [S2JUnit4] とラベル風に) 直して force pushしてもらえますか? |
76a207c
to
3d5acb7
Compare
コミットログを直しました。 |
@manhole コメントした些細な点以外はいいと思います。 @nakamura-to 念のためレビューお願いします。 |
問題ないと思います |
@koichik @nakamura-to ありがとうございます。 |
Expected :ac(0:1,1)dc(1:2,4)dc(2:3,9)db Actual :ac(0:1,1)dbac(1:2,4)dbac(2:3,9)db
d831084
to
d9e3846
Compare
squashしてforce commitしました。 (修正したのがわかりづらくなるかと思ってsquashしていませんでした。) |
[S2JUnit4] @parameters使用時に@BeforeClass,@afterclassがクラスで1度だけ実行されるように修正しました
ありがとうございます、マージしました。 Node.jsなんかだとこちら側で勝手にsquashすることもありだった (そしてマージコミット作らずrebaseする) のですが、プルリクのコミット (sha) を勝手に変えてはいけない文化もあるようだし悩ましいですねー。 |
マージありがとうございます! はい、悩ましいです > プルリク |
S2JUnitで@parametersを使用する際に@BeforeClass,@afterclassがテストメソッドごとに実行されていましたが、
クラスで1度だけ実行されるように修正しました。
(JUnitと動きを合わせました)