ALM 開発者の 1 日: 作業の中断、バグの修正、コードレビューの実行

[アーティクル]
02/21/2013

作業の 1 種類のスレッド間で、作業のコンテキストを切り替えることができる強力な 1 クリックの方法は、Team Foundation Server で Visual Studio Premium と Visual Studio Ultimate で使用できます。また、チームメンバーはコードについて提案された変更に関するメッセージ交換簡単にできます。このトピックでは、1 日架空のアジャイルチームのメンバーに従ってチュートリアルを続行これらの機能について説明します。

Peter は、バックログ項目のタスクを実行するコードを記述でビジー状態です。ただし、自分のユーザーが、そのブロックには、それをすぐに修正するバグを検索します。Bob は Alice にする中断し、バグを修正を処理します。Mallory はジュリアに修正の確認を行うように尋ね、レビュー、チェックイン修正および再開が、元のタスクを実行した後。

[!メモ]

Peter を使用して自分の作業とコードレビューの機能は Visual Studio Premium と Visual Studio Ultimateでのみ使用できます。

このトピックの内容

現在の作業を中断し、バグの作業を開始します
バグを検索します。
コードレビューを要求します。
(または拒否) コードレビューを受け入れ、実行します。
コードレビューに受信して応答します
テストおよびコードを修正
修正をチェックイン
タスクの作業の再開

現在の作業を中断します。

Peter がバックログ項目を操作しているときに、ジュリアは、バグをブロックを話し合うことがあります。これは Peter を理解しているため、Mallory はバグを修正するタスクを作成し、自身に割り当てる領域にあります。Mallory は修正の作業をすばやく開始することになります。

これが新しいバグで作業を開始する前に Peter は、自分の現在の動作がチームのサーバー上の安全な場所で取得専用ことが必要になる場合があります。[担当作業] のページで、Peter は保存するに中断を選択します Team Foundation Server () :

作成したコードが、テスト、およびそのほかのファイルへの変更を含むすべての作業は。
Visual Studio の状態の開いているソリューション、ウィンドウ、ブレークポイント、ウォッチウィンドウで変数やそのほかのビット。

自分のワークスペースのクリーンであるため、Peter は [使用できる作業項目] から **[処理中の作業]**に新しいタスクをドラッグします。Mallory は修正を調査して記述する準備ができました。

[!メモ]

の動作は、コンテキストと同時に作業のページの実行中に示す作業項目にリンクします。中断と [再開]を使用して、さまざまなタスク間ですばやく切り替えることができます。は、開いているソリューションファイルと、コードの変更と Visual Studio のレイアウトはすべて一緒に切り替えます。

現在の作業を中断し、別のタスクの作業を開始するには

一部の作業の中断

使用するチームプロジェクトに接続する必要**:接続** とチームプロジェクトに接続します。:
1. [チームエクスプローラー]では、[ホーム]を選択し、**[担当作業]**を選択します。
中断上記のタスク:
1. [処理中の作業] のセクションでは、中断を選択します。
2. 表示されるダイアログボックスで中断された作業のセットを付ける指定し、を中断のボタンをクリックすると、名前の。既定の名前は、の現在進行中の作業項目です。
新しいタスク作業の開始、バグなどの作業項目:
1. 作業項目を選択する前に、必要な可能性があります:
  - **[使用できる作業項目]**の下に新規のをクリックして、新しいタスク、またはそのほかの作業項目を作成します。; または
  - **[使用できる作業項目]**の下の別のクエリを選択します。
2. [使用できる作業項目] から **[処理中の作業]**に作業項目をドラッグします。
  
  また、を **[中断されている作業]**のからコントロールをドラッグする前によって中断する作業項目へ切り替えることができます。

ヒント
現在進行中の作業項目は、現在のコードの変更と Visual Studio の状態にリンクします。1 個のタスク間で切り替えると適切な項目が実行中の状態であることを Visual Studio での作業を整理するのに役立つ確認します。

バグを検索します。

Peter は、バグ作業項目を開き、読み取ります。テストチームのメンバーによって作成された説明に従って、支払いされたの請求書は、無給ように、不適切にフラグが設定されます。バグ作業項目に添付されたラボ環境のスナップショットがあります。Peter は、IntelliTrace ログを使用してテストが実行された仮想マシンを開く、" 不適切な請求書、ついて説明します。Mallory は、次のメソッドの違反をトレース:

    public class LocalMath
    {       
        public static bool EqualTo(double a, double b)
        {
          return a == b;
        }

IntelliTrace ログから、Peter は、パラメーターがあってもごくわずか異なるため、メソッドは false を返すことがわかります。Peter は、この種の丸め誤差が浮動小数点演算で場合であること、および参照が等値で浮動小数点数をテストする望ましくないことを認識します。

エラーを示すために、テストをインクリメントします。

バグがあるとき、単体テストにすき間があるか、またはテストがユーザーの実際のニーズに一致しないことを示します。したがって、バグを修正、Peter がこのエラーの存在を示すテストを追加します。

 
        // Added 2012-02-02 for bug 654321:
        /// <summary>
        /// Make sure that number equality test allows for 
        /// small rounding errors.
        /// </summary>
        [TestMethod]
        public void TestDoublesEqual()
        {
            // We allow a rounding error of 1 in 1000000:
            TestEqual(1, 1e-7, true); // Less than allowed error
            TestEqual(1, 1e-5, false); // More than allowed error
            TestEqual(1000, 1e-7, true); // Less than allowed error
            TestEqual(1000, 1e-5, false); // More than allowed error
        }
        private void TestEqual(double value, double error, bool result)
        {
            // Try different combinations of error and value:
            Assert.IsTrue(result == LocalMath.EqualTo(value + error, value));
            Assert.IsTrue(result == LocalMath.EqualTo(value, value + error));
            Assert.IsTrue(result == LocalMath.EqualTo(value - error, value));
            Assert.IsTrue(result == LocalMath.EqualTo(value, value - error));
        }

Mallory は、テストを実行し、予想どおりに失敗します。

等価テストの失敗を示す単体テストエクスプローラー

テストのパスを設定します

Peter は、コードを修正:

        public static bool EqualTo(double a, double b)
        {
            // Allow for rounding errors.
            // For example, a == 2.0 and b = 1.99999999999

            const double allowedError = 1/1000000;
            return System.Math.Abs(a - b) < allowedError;
        }

テストのパス:

等価テストの成功を示す単体テストエクスプローラー

コードレビューを要求します。

Peter はバグとの間の修正に成立しますが、Mallory は自分の作業をまだチェックイン。自分のチームは、全体的なコード品質を向上し、より多くのバグを作成するリスクを軽減するコードレビューを使用して、Peter は、自分のチームジュリア関連付けるとアダムからコードレビューを要求するためにチームエクスプローラーを使用します。

コードレビューを要求します。

コードレビューの要求

[チームエクスプローラー]では、[担当作業] のページで、**[レビューの要求]**を選択します。

[新しいコードレビュー] のページが表示されます。
一つ以上のレビュー担当者を指定します。
レビューの名前を指定します。
区分パスを指定します。
すべてのレビュー担当者にコメントを指定します。
**[要求の送信]**を選択します。

レビュー担当者は、電子メールで要求の通知されます。

中断された処理、シェルブセット、または変更セットのコードレビューを要求できます。

コードレビューを受け入れるか拒否します。

ジュリアは、コードレビューの要求を受け取り、受け取ります。Alice はコードを見直しましたり、ファイル、およびコードブロックレベルにあるコメントを記述し、Peter にコードレビューを返します。アダムは、コードをレビューするには、ビジー状態で、拒否します。

自分のコメントでは、ジュリアは、テストが誤っていることを示しています。有効なエラーは、入力値の指定された部分が、定数量必要があります。このテストは、エラーをインクリメントする必要があります。

            // We allow a rounding error of 1 in 1000000
            // as a fraction of the value:
            TestEqual(1, 1e-7, true); // Less than allowed error
            TestEqual(1, 1e-5, false); // More than allowed error
            TestEqual(1000, 1000*1e-7, true); // Less than allowed error
            TestEqual(1000, 1000*1e-5, false); // More than allowed error

ヒント
チームメンバーがついては、フォーカスとしてテストを使用することに注意してください。テストが正しく、十分である場合は、コードでもあります。コードとは異なり、各テストは別のケースを表します。したがって、テストは、多くについて、およびコードをより簡単です。

コードレビューを実行するには

コードレビューの受け入れと応答コードレビューの実行

[チームエクスプローラー]では、[担当作業] のページで、マイコードのレビューと要求 のセクションに移動し、要求を開きます。
[コードレビュー] のページで、あります:
- レビューを行うかどうかを作成者に通知するために [同意する] か 同意しない を選択します。
- code–review の要求に他のレビュー担当者を追加するに [レビューアーの追加] を選択します。
- この作業項目の更新された各ファイルへの変更を確認します。
- 作成者と他のレビュー担当者と変更についてするに [コメント] を展開します。
  - **全体的なコメントを追加します。*****を選択します。
    
    または
    
    コードブロックを選択し、ショートカットメニューの [コメントの追加] を選択します。
  - 作成者と、の機能を他のレビュー担当者に表示できるようにするにコメントを送信します。*** を選択します。
- コードがより多くの作業を必要とするかどうかを示す、のレビューを完了するに send および完了 を選択します。

コードレビューに応答します

Peter はジュリアからコードレビューに受け取り、応答します。

コードレビューに応答するには

使うとコードのレビュー担当者と開発者はコメントを頻繁に交換できます。レビューの作成者が閉じると終了します。ディスカッションをそれぞれ使用して、他の参加要素は、電子メールで通知されます。

コードレビューへの対応

[チームエクスプローラー]では、[担当作業] のページで、コードレビューと要求 のセクションに移動し、要求をダブルクリックします。

また、要求のショートカットメニューを開き、**[開く]**を選択できます。
必要に応じてそれらのコメントと応答を参照してください。コメントに応答するには、**[返信]を入力、コメントを表示して、を選択します。[OK]**にボックスをオンにします。、コメントを送信するには、**コメントを送信します。*****を選択します。
ファイルを表示し、コメントを含むコードブロックを表示するには、またはファイルを編集することは [コメント] のセクションに、移動します。[ファイル] の部分で、ファイルのショートカットメニューを開き、比較 (読み取り専用) か **[ファイルの編集]**を選択します。
、およびそのほかのレビュー担当者が互いのコメントに応答が終了され、レビューを終了準備が整ったとき **[レビューを閉じる]**をクリックし、次のいずれかを選択する:
- レビューが終了したことを示す完了します。***。
- または
- 示す破棄は、レビューはキャンセルされます。

テストおよびコードを修正

Alice が示すようにジュリアのコメント、Peter を読み取る自分の単体テストを修正します。テストで、失敗します。これにより、コードがまだ正しくないことを示します。

Peter は、コードを修正:

        /// <summary>
        /// Returns true if two numbers are equal.
        /// </summary>
        public static bool EqualTo(double a, double b)
        {
            // Allow for rounding errors.
            const double allowedErrorMultiple = 1/1000000;
            double allowedError = (System.Math.Abs(a) + System.Math.Abs(b)) * allowedErrorMultiple/2;
            return System.Math.Abs(a - b) < allowedError;
        }

テストは再びパス:

等価テストの成功を示す単体テストエクスプローラー

ヒント
バグを修正するには、コードの開発と同じ手順に従います。失敗するテストを記述し、テストを渡らせます。テストに成功した場合にのみコードとテストをチェックインします。

Peter は、バグが検出されたテストケースへの自分の注意を切り替えます。バグを再現する手順は、テストケースの作業項目で明確に記述されています。Mallory は手順に従って、請求書が正しく表示されることを確認します。

修正をチェックイン

Peter のチェックイン固定コードと単体テスト。バグの状態は [解決済み]に自動的に設定され、[担当者] の値は、バグを検出したテストチームのメンバーに自動的に再度割り当てられます。そのチームメンバーが、バグを確認し、作業項目を修正するを閉じます。

修正をチェックイン

バグを修正するための更新プログラムのチェックイン

[チームエクスプローラー]では、[担当作業] のページで、**[チェックイン]**を選択します。
ように [保留中の変更] ページの内容を確認します:
- **[含まれる変更]**のすべての関連する変更が一覧表示
- 関連するすべての作業項目は **[関連作業項目]**に示します。
それらが変更されたファイルとフォルダーのバージョン管理履歴を表示するとき、チームがこれらの変更の目的を理解できるように [コメント] を指定します。
[チェックイン] を選択します。

タスクの作業の再開

Peter の再開が Bob のタスクを実行します。Bob は Alice のコード変更がすべて開いているウィンドウ、ブレークポイントやウォッチウィンドウの変数のような状態の上位ビットとともに自分のワークスペースに復元するため、すぐに使用できるように戻れます。

タスクの作業を再開します。

タスクの再開と完了

[チームエクスプローラー]では、[担当作業] のページで、Suspended & Shelved の作業 のリストを取得します。項目のショートカットメニューを開きます。2 個のオプションがあります:
- 中断された作業を再度開き、自動的に自分のワークスペースの保留中の変更を中断する場合は、**[再開]**を選択します。
- ワークスペースの保留中の変更が中断された作業を既にマージする場合は、**[処理中とマージ]**を選択します。

作業を再開したとき

作業項目を中断すると影響を受けるペイン。

作業を再び開くと、Visual Studio が復元されます:

は、開いているソリューション
コードの変更
開いているウィンドウの状態と位置
ブレークポイント
ウォッチウィンドウの変数と式
ブックマーク

ALM 開発者の 1 日: 作業の中断、バグの修正、コード レビューの実行

現在の作業を中断します。

現在の作業を中断し、別のタスクの作業を開始するには

バグを検索します。

エラーを示すために、テストをインクリメントします。

テストのパスを設定します

コード レビューを要求します。

コード レビューを要求します。

コード レビューを受け入れるか拒否します。

コード レビューを実行するには

コード レビューに応答します

コード レビューに応答するには

テストおよびコードを修正

修正をチェックイン

修正をチェックイン

タスクの作業の再開

タスクの作業を再開します。

作業を再開したとき

その他のリソース

ALM 開発者の 1 日: 作業の中断、バグの修正、コードレビューの実行

コードレビューを要求します。

コードレビューを要求します。

コードレビューを受け入れるか拒否します。

コードレビューを実行するには

コードレビューに応答します

コードレビューに応答するには