テスト手法や複数人開発の詳しい人に質問です。
(CodeCompleteの内容は前提知識としてあるぐらい)
コードのマージ時にマージミスで重複行が発生してしまうのを防ぐ
良い方法ってご存じでしょうか。
具体的にはマージ時に、マージのミスで
call_method(foo)
if cond
call_method(foo)
end
みたいな重複行が発生することをチェックする仕組みがや取り組みが知りたいです。
なおテストケースではc1は網羅しててもメソッドの重複呼び出しだと場合によりテストが失敗しないので、テストをもっと書くべきというアドバイス以外でお願いします。
ベストアンサー
-
さぼてん
満足200pt
質問のようなことをやりたければ、Jenkinsを使ってる会社の人ということで回答しますが、静的解析ツールで重複しているコード数を把握し、増加を見るというのはどうでしょうか？
単なるマージで増加するのは普通はおかしいので、その都度チェックを入れればいい、という考えです。
そういうことをするのに、JavaだとPMDというツールが重複もチェックしてくれて、Jenkinsにはそれ用のPMD Pluginがあります。
Rubyには詳しくないですが、Flayというツールを使えば重複をチェックできるんじゃないんでしょうか？
残念ながらJenkinsにはPluginがないので、Plot Pluginなどを使えば増減を測定できるはずです。
-
その他の回答
-
a-kuma3
満足100pt
質問に書いたケースは、単純化してるから余計にそう感じるのかもしれませんが、
この手のマージのミスを防ぐのはテストでは無い、と思います。
さすがに Git なり、バージョン管理システムを使っているだろうとは、想像しますが、
commit の単位が大きすぎるんじゃないでしょうか？
例えば、チケットの単位でしか commit を認めてない、とか。
Unit テストが通る、というのは大前提になると思いますが、その範囲内で、細かく commit していれば、
特定のコードを if でくくった程度のマージがミスるというのは、ちょっと想像がつきません。
-
a-kuma3この質問へのコメント
質問の情報
- 登録日時
- 2011-10-28 16:56:29
- 終了日時
- 2011-11-02 09:13:26
- 回答条件
- 1人10回まで
この質問のカテゴリ
この質問に含まれるキーワード
人気の質問
-
この英文どうなってますか？ The boy seemed to be waiting for his mother to come.（その少年は母親が来…
匿名質問者
1
1
-
不定詞の形容詞的用法らしいんですけど、副詞的用法と書いたら✖️ですか？ I want to buy a book to read …匿名質問者11
-
日向陽葵という○V女優が好きなのですが似ている○V女優はいますか匿名質問者
-
共通テストで使われるプログラミング言語は共通テスト特有の言語と聞いたのですが、受験勉強で身につけたプ…匿名質問者1
-
内定してほしいポケモンはいますか？匿名質問者
それを曲解して、チケットの単位＝コミットの単位、としてるんじゃないかなあ、と。
redmine は使ってませんけど、ぼくの身近にも、似たような事例はありますし。
Mockでonceとかtwiceとかstrictに書けば避けられるのかなあ
可能だけどMockObjectの使い方として明らかに違う気もする
http://xunitpatterns.com/Mock%20Object.html
違うのですよねぇ。
解答の一つでありますが、ある程度高レベルのAPIのメソッド呼び出しの呼び出し回数を記録しておき、マージ前と後で比較してチェックすることで人的コストは高いですが、意図しない変更は解りそうですね。
マージ用ブランチのCI結果のtest coverageをみて、それをベースにレビューするという感じです。