[S2JUnit4] @Parameters使用時に@BeforeClass,@AfterClassがクラスで1度だけ実行されるように修正しました

[manhole]

S2JUnitで@Parametersを使用する際に@BeforeClass,@AfterClassがテストメソッドごとに実行されていましたが、 クラスで1度だけ実行されるように修正しました。 (JUnitと動きを合わせました)

[koichik]

コミットログのS2JUnitをS2JUnit4に(できればコミットログの先頭に [S2JUnit4] とラベル風に) 直して force pushしてもらえますか？

[manhole]

コミットログを直しました。

[koichik]

@manhole コメントした些細な点以外はいいと思います。

@nakamura-to 念のためレビューお願いします。

[nakamura-to]

問題ないと思います

[manhole]

@koichik @nakamura-to ありがとうございます。 コメント頂いた箇所を直しました。

[koichik]

@manhole あー、できれば追加の3コミットは rebase -i で 3d5acb7 にsquashしてほしいなぁ。。。 (そのあとでforce push)

[manhole]

squashしてforce commitしました。

(修正したのがわかりづらくなるかと思ってsquashしていませんでした。)

[koichik]

ありがとうございます、マージしました。

Node.jsなんかだとこちら側で勝手にsquashすることもありだった (そしてマージコミット作らずrebaseする) のですが、プルリクのコミット (sha) を勝手に変えてはいけない文化もあるようだし悩ましいですねー。

[manhole]

マージありがとうございます!

はい、悩ましいです ＞ プルリク 自分からはsquashに踏み切れなかったのですが、レビューアから促していただけて気が楽になりました。