このセクションでは、コード レビューをサポートする Parasoft 製品と SOAtest で利用できるコード レビュー機能について、その概要を紹介します。

このセクションの内容:

概要

Parasoft 社のコード レビューは、レビューの準備、通知、追跡を自動化することによって、レビューをより実用的かつ生産的にすることを目的として開発されました。コード レビューは、更新されたコードを自動的に特定し、担当レビューアーにコードを割り当てます。そして、問題がクローズするまで各レビュー項目の進行状況を追跡します。これによって、チームは強力なレビュー プロセスを確立し、確実にコードをレビューして発見したすべての問題を解決するようにできます。  

Parasoft 社のコード レビューは、次の一般的なコード レビュー ワークフローをサポートします。

  • コミット後:  カスタム ソース管理インターフェイスを介してソース リポジトリ中のコードの変更を自動的に特定します。変更されたコードとレビューアーの定義済みマッピングに基づいて、レビュー タスクが作成されます。
  • コミット前:  レビューに出すファイルを選択してデスクトップからコード レビューを開始するか、あるいはローカルに変更されたすべてのソース コードを自動的に特定することができます。
  • タスク駆動型:  特定の Parasoft Project Center タスクに関連するファイルを選択してデスクトップからコード レビューを開始できます。選択したファイルがソース管理システムにコミットされているかどうかは関係ありません。

なお、コード レビューを使用するには、Team Server と少なくとも 1 つの Parasoft Test Server Edition が必要です。

ワークフローの概要

2 種類の重要なワークフローがあります。「デフォルト vs. 制限付き」そして「コミット前 vs. コミット後 vs. タスク駆動型」です。

デフォルト vs. 制限付き

制限付きワークフローでは、レビューアーは各問題を承認しなければなりません。レビューアーの提案に作成者が同意しない場合、作成者はその問題についてレビューアーと話し合う必要があります。 

デフォルト ワークフローはもっとオープンです。レビューアーからコメントを受け取った作成者は、コメントの内容をコードに適用するか、またはコードをそのままにしておくことができます。

コミット前 vs. コミット後 vs. タスク駆動型

コミット前

コミット前のコード レビューは、ソース管理に追加する前にコードをレビューしたい開発チームに適しています。 新規のコードまたは変更済みのコードをレビューに出す準備ができたら、Parasoft Test からコード レビュー用テスト コンフィギュレーションを実行します。すると、必要なレビューがレビューアーに自動的に通知されます。 

コミット後

コミット後コード レビューは、ソース管理にコードがコミットされた後にコードをレビューしたい開発チームに適していいます。通常、コード レビュー用テスト コンフィギュレーションは定期的に (たとえば 24 時間ごとに) 自動的に実行するようにスケジュールされます。ソース管理リポジトリをスキャンし、レビューが必要なコードがあった場合、その情報を Team Server に送信して担当レビューアーに配布します。このワークフローの場合、コードの作成者は、コードをレビューしてもらうために特別なアクションを取る必要がありません。ソース管理システムにコードをコミットするだけで十分です。コード レビュー用テスト コンフィギュレーションを実行すると、必要なレビューが担当レビューアーに自動的に通知されます。 

コミット前のプロセスでは、ソース管理システムにコミットする前に承認が必要なコードに対して、コード レビュー パッケージが作成されます。コミット前のプロセスにおいて、作成者はパッケージが最終的に承認されたかどうかを必ず確認するべきです。承認を確認した時点で、作成者はファイルをコミットしてコード レビュー パッケージをクローズすることができます。

コミット前プロセスでレビューのためにコードをサブミットするチームの中には、次の操作を行うためにコミット後の夜間スキャンの実行を好むチームもあります。

  • 電子メールを生成し、割り当てられたコード レビュー タスクについて作成者とレビューアーに通知する。
  • ソース管理にコミットされたがコミット前の処理でレビューのためにサブミットされなかったコードの変更を特定する。

ワークフローの詳細

次の図は、利用できる重要なワークフローを表しています。

コミット前ワークフロー

コミット後ワークフロー

ビルトイン コード レビュー テスト コンフィギュレーション

  • Pre-Commit: ソース管理システムにコミットする「前」にコードをレビューしたいチームに適しています (「コミット前 vs. コミット後vs. タスク駆動型」を参照)。ローカルに追加または変更されたファイルをスキャンします。このテスト コンフィギュレーションを使用するには、コード レビューの設定で [スキャナー実行中にユーザー補助を表示] チェックボックスをオンにし、適切なレビューアーを作成者が指名できるようにする必要があります。
  • Post-Commit (Template): ソース管理システムにコミットした「後」にコードをレビューしたいチームに適しています (「コミット前 vs. コミット後vs. タスク駆動型」を参照)。前日に変更されたすべてのプロジェクト ファイルをスキャンします。このテスト コンフィギュレーションは、複製してカスタマイズしてから使用しなければなりません (例えば作成者とレビューアーのマッピングなどをカスタマイズします)。詳細については、「コミット後コード レビュー スキャンの構成と実行」 を参照してください。

  • Post-Commit (Assign All):  ソース管理システムにコミットした「後」にコードをレビューしたいチームに適しています (「コミット前 vs. コミット後vs. タスク駆動型」を参照)。前日に変更されたすべてのプロジェクト ファイルをスキャンします。このテスト コンフィギュレーションはカスタマイズせずに使用できます。ローカル コード レビュー ユーザーのマッピングを含みます。(あらゆる作成者について) スコープで発見されたすべてのリビジョンを現行ユーザーに割り当てます。

コード レビュー タスク / ステータス レポート

コード レビューの UI」で説明するように、コード レビューのタスクは GUI でレポートされます。コマンドラインまたは GUI からの解析で生成されたレポートの [コード レビュー] セクションにはコード レビューの詳細情報も表示されます。これらのレポートは、プロジェクトと作成者ごとにコード レビュー タスクの概要を提供します。また、保留中の課題とコード レビュー タスクについての情報も提供します。

例:

  • No labels