[转载]基于TFS实践敏捷-修复Bug和执行代码评审
本主题阐释了这些功能,以继续这一关注虚拟敏捷团队成员的一天的教程。
Peter 忙于编写一些代码以完成积压工作 (backlog) 项任务。但是,他的同事发现了一个阻碍他们工作的 Bug,他想立即修复此 Bug。 他暂停了手中的工作并修复此 Bug。 他请求 Julia 审查修复情况,并在审查后检查修复结果并恢复其初始任务。
说明 |
Peter 使用的“我的工作”和“代码评审”功能仅在 Visual Studio 高级专业版和 Visual Studio 旗舰版中可用。 |
主题内容
- 挂起当前工作并开始处理 Bug
- 调查 Bug
- 请求代码评审
- 接受并执行(或拒绝)代码评审
- 接收并响应代码评审
- 修复测试和代码
- 签入修复
- 继续处理任务
当 Peter 处理积压工作 (backlog) 项时,Julia 过来讨论困扰她的 Bug。 这是 Peter 所熟悉的区域,因此他创建了修复 Bug 的任务,并将其分配给自己。 他决定立即开始修复工作。
在 Peter 开始处理新的 Bug 前,他想确保其当前工作保留在团队服务器上的安全位置。 在“我的工作”页上,Peter 选择“挂起”进行保存(在 Team Foundation Server 上):
- 他完成的所有工作包括对代码、测试和其他文件的更改。
- 打开解决方案、窗口、断点、监视窗口变量和其他 Visual Studio 状态的位。
限制已清理工作区,Peter 将新的任务从“可用工作项”拖动到“正在进行的工作”。 他已准备好研究和编写修复。
说明 |
你的工作上下文链接到在“我的工作”页中显示为“正在进行”的工作项。 通过使用“挂起”和“继续”,可在不同的任务间快速切换。 打开的解决方案和文件、代码更改以及 Visual Studio 布局都会一起切换。 |
挂起当前工作并开始执行另一项任务
- 连接:如果尚未连接到要在其中工作的团队项目,请连接到团队项目:
- 在“团队资源管理器”中,选择“主页”,然后选择“我的工作”。
- 挂起你的当前任务:
- 在“正在进行的工作”部分中,选择“挂起”。
- 在显示的框中,指定你要为此挂起的工作集提供的名称,然后选择“挂起”按钮。 默认名称是当前正在进行的工作项。
- 开始新任务、Bug 或其他工作项上的工作:
- 在选择工作项之前,你可能需要:
- 通过选择“可用工作项”下的“新建”,创建新任务或其他工作项;或
- 在“可用工作项”下选择不同查询。
- 将工作项从“可用工作项”拖动到“正在进行的工作”。
或者,可以从“挂起的工作”项下拖出之前已挂起的工作项来切换到该工作项。
提示 |
当前正在进行的工作项将链接到当前代码更改和 |
Peter 打开并阅读 Bug 工作项。 根据测试团队成员编写的说明,已付款的发票有时会被错误地标记为未付款。 有一个附加到 Bug 工作项的实验室环境快照。 Peter 可以打开运行测试的虚拟机,查看错误发票和 IntelliTrace 日志。 他跟踪以下方法存在的错误:
public class LocalMath
{
public static bool EqualTo(double a, double b)
{
return a == b;
}
从 IntelliTrace 日志中,Peter 发现此方法有时会因参数存在极小的差异而返回 false。 Peter 知道此类舍入误差在浮点算法中是不可避免的,同时,利用此方法测试浮点数字的相等性也较为不妥。
增加测试以显示错误
在找到 Bug 后,就会显示单元测试数量不足或测试不符合用户的实际需求。 因此,在修复 Bug 之前,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(, 1e-, true); // Less than allowed error
TestEqual(, 1e-, false); // More than allowed error
TestEqual(, 1e-, true); // Less than allowed error
TestEqual(, 1e-, 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));
}
他运行了该测试,结果如预期一样以失败告终。
让测试通过
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 = /;
return System.Math.Abs(a - b) < allowedError;
}
测试现在通过:
Peter 对自己对 Bug 的修复感到满意,但是尚未签入其工作。 Peter 的团队使用代码评审来提高整体代码质量并减小因创建更多 Bug 带来的风险,因此,他使用团队资源管理器从团队同事 Julia 和 Adam 那里请求代码评审。
请求代码评审
- 在“团队资源管理器”中的“我的工作”页上,选择“请求审查”。“新代码审阅”页将出现。
- 指定一个或多个审阅者。
- 指定评审的名称。
- 指定区域路径。
- 指定审阅者的注释。
- 选择“提交请求”。
审阅者将通过电子邮件接收请求通知。
你还可以请求对挂起的工作、搁置集或变更集进行代码评审。 若要查看变更集的列表,请打开“源代码管理资源管理器”,然后选择“历史记录”按钮。
Julia 接收并接受代码评审请求。 她评审代码,并在文件和代码块级别编写注释,然后将代码评审反馈给 Peter。 Adam 太忙,无法评审代码,因此拒绝了请求。
Julia 在其注释中指出测试是错误的。 允许的误差应为输入值的指定部分,而不是常数。 因此,测试应将错误值乘以值。
// We allow a rounding error of 1 in 1000000
// as a fraction of the value:
TestEqual(, 1e-, true); // Less than allowed error
TestEqual(, 1e-, false); // More than allowed error
TestEqual(, *1e-, true); // Less than allowed error
TestEqual(, *1e-, false); // More than allowed error
提示 |
请注意,团队成员会将测试用作讨论焦点。 如果测试正确且足够,则代码也会如此。 与代码不同,每个测试表示单独的大小写。 因此,讨论测试通常比讨论代码更容易。 |
执行代码评审
- 在“团队资源管理器”中的“我的工作”页上,转到“我的代码评审和请求”部分并打开该请求。
- 在“代码评审”页上,你可以:
- 选择“接受”或“拒绝”以通知作者你是否将执行评审。
- 选择“添加审阅者”以将其他审阅者添加到代码审阅请求。
- 查看对每个已为此工作项更新的文件的更改。
- 展开“注释”以与作者和其他审阅者讨论更改。
- 选择“添加总体评价”
- 或 -
选择一个代码块,然后从快捷菜单中选择“添加注释”。 - 选择“发送评论”以使你的提交对作者和其他审阅者可见。
- 选择“发送和完成”以完成审阅,指出是否需要进一步修改代码。
Peter
接收并响应了来自 Julia 的代码评审。
代码的审阅者和作者可以根据需要交换注释。
作者一旦关闭评审,则评审就将结束。 每次有人参与讨论时,其他参与者都会收到电子邮件通知。
- 在“团队资源管理器”中的“我的工作”页上,转到“代码评审和请求”部分,然后双击该请求。
你还可以打开请求的快捷菜单并选择“打开”。 - 阅读评论并根据需要进行答复。
若要答复评论,请选择“答复”,在出现的框中输入你的评论,然后选择“确定”。 若要发送你的评论,请选择“发送评论”。 - 若要查看文件并查看包含注释的代码块或者要编辑文件,请转到“注释”部分。
在“文件”子节中,打开文件的快捷菜单并选择“比较(只读)”或“编辑文件”。 - 如果你和其他审阅者已回复完彼此的评论并准备关闭审阅,请单击“关闭审阅”,然后选择:
- “完成”可表明已完成审阅。
- - 或 -
- “放弃”可表明正在取消审阅。
阅读完 Julia
的注释后,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 = /;
double allowedError = (System.Math.Abs(a) + System.Math.Abs(b)) * allowedErrorMultiple/;
return System.Math.Abs(a - b) < allowedError;
}
测试再次通过:
提示 |
若要修复此 Bug,请按照代码开发中的相同做法进行操作。 编写会失败的测试,然后设法让该测试通过。 仅在测试通过时,签入代码和测试。 |
Peter 现在将注意力转移到发现 Bug 的测试用例上。 在测试用例工作项中清楚说明了 Bug 的重现步骤。 他按照这些步骤操作,并发现已正确列出了发票。
Peter 签入已修复代码和单元测试。 Bug 的状态自动设置为“已解决”,并且“指派给”值将自动重新分配给发现 Bug 的测试团队的成员。 该团队成员将验证 Bug 是否已修复并关闭工作项。
- 在“团队资源管理器”中,在“我的工作”页上,选择“签入”。
- 查看“挂起的更改”页的内容以确保:
- 所有相关更改都在“包含的更改”中列出
- 所有相关工作项都在“相关工作项”中列出。
- 指定“注释”以在你的团队查看已更改文件和文件夹的版本控制历史记录时助其了解这些更改的目的。
- 选择“签入”。
Peter 继续处理自己的任务。 他很快就可以继续工作,因为他的所有代码更改连同重要的状态图位(如打开窗口、断点及观看窗口变量)都已还原至工作区。
继续任务中的工作
- 在“团队资源管理器”中的“我的工作”页上,查找“挂起的和搁置的工作”列表。 打开项的快捷菜单。 你有两个选择:
- 若要继续挂起的工作并自动挂起工作区中任何挂起的更改,请选择“继续”。
- 若要将挂起的工作与工作区中已挂起的更改合并,请选择“与“正在进行”部分合并”。
当你继续工作时
当你继续工作时,Visual
Studio 会还原:
- 打开的解决方案
- 代码更改
- 打开的窗口的状态和位置
- 断点
- 监视窗口变量与表达式
- 书签
如果测试团队发现此
bug,他们会将一些测试用例链接到此 bug,并且将重新运行它们。 了解详细信息。
博客转自:《ALM开发人员生活中的一天:暂停工作、修复Bug和执行代码评审》
[转载]基于TFS实践敏捷-修复Bug和执行代码评审的更多相关文章
- [转载]基于TFS实践敏捷-项目管理
最近,一直想让团队加入敏捷开发,但TFS2010用下来,并不是令人满意,还好,TFS2012横空出世了.相比TFS2010,TFS2012改进太多了,主要体现在以下方面: Team Web Acces ...
- [转载]基于TFS实践敏捷-实现用户场景
您是新用户的 Visual Studio 应用程序生命周期管理 (ALM) 和 Team Foundation Server (TFS) 吗? 您想知道如何您和您的团队可以获得最大受益的这些工具来生成 ...
- [转载]基于TFS实践敏捷-工作项跟踪
工作项跟踪(1) 可跟踪性是软件过程的重要能力,TFS主要是以工作项来实现过程的可跟踪性.曾有人问:"你们实际项目里的工作项是怎么样的?能不能让我们看看?"我也一直很好奇别的公司T ...
- [转载]基于TFS实践敏捷-Scrum模式运用
根据Forrester Research今年第二季度的一份研究报告,在超过1000名专业开发人员中,采用敏捷模式进行软件开发的已经有10.9%采用了Scrum模式,在所有的敏捷开发模式中名列首位,而在 ...
- 基于TFS实践敏捷-可视化管理
TFS是基于微软平台一套不错的系统,支持源码管理+运行调试+持续集成+自动化测试+Bug管理+代码评审+任务项管理+文档管理+沟通管理.基于TFS 2015实践看板管理,让团队的数据可视化,让大家更多 ...
- [转载]基于Redis的Bloomfilter去重(附Python代码)
前言: “去重”是日常工作中会经常用到的一项技能,在爬虫领域更是常用,并且规模一般都比较大.去重需要考虑两个点:去重的数据量.去重速度.为了保持较快的去重速度,一般选择在内存中进行去重. 数据量不大时 ...
- 基于TFS的.net技术路线的云平台DevOps实践
DevOps是近几年非常流行的系统研发管理模式,很多公司都或多或少在践行DevOps.那么,今天就说说特来电云平台在DevOps方面的实践吧. 说DevOps,不得不说DevOps的具体含义.那么,D ...
- 敏捷开发中高质量 Java 代码开发实践
Java 项目开发过程中,由于开发人员的经验.代码风格各不相同,以及缺乏统一的标准和管理流程,往往导致整个项目的代码质量较差,难于维护,需要较大的测试投入 和周期等问题. 这些问题在一个项目组初建.需 ...
- 敏捷测试(6)--基于story的敏捷基础知识
基于story的敏捷基础知识----需求管理(三) (3)每日站会 站会的目的有三个: (1)周知进度 仅从用户故事和任务的层面周知进度,任务进度只有两种状态:完成或未完成(完成百分比). (2)周知 ...
随机推荐
- 屏幕适配1(Android 增强版百分比布局库 为了适配而扩展)
转载请标明出处: http://blog.csdn.net/lmj623565791/article/details/46767825: 本文出自:[张鸿洋的博客] 这篇博文写得太好了,让我很激动,分 ...
- 在docker里部署网络服务
之前试着玩玩docker有一阵子了,今天算是头一回正式在docker里部署网络服务. 本来想和lxc差不多的东西那自然是手到擒来,没想到还是改了很多. 第一个遇到的问题是,远程连到docker宿主机干 ...
- Win10 无法完全关机问题
Win10是重新安装的.开机运行时间长了或者跑的东西多了.关机,键盘灯还是亮的,要强制按电源键关机才行. 问题解决:从网上搜以为是显卡驱动问题,上官网更新最新驱动,结果还是关不了机.偶然间搜到是Int ...
- 直接代码POST数据调用WebService
ps:使用过webservice的童鞋大概都明白它是基于Soap协议交换数据的,同时Soap协议是对HTTP协议的扩展,其实我们就可以认为调用一个WEB服务就是通过http协议GET或POST数据的过 ...
- 如何基于RabbitMQ实现优先级队列
概述 由于种种原因,RabbitMQ到目前为止,官方还没有实现优先级队列,只实现了Consumer的优先级处理. 但是,迫于种种原因,应用层面上又需要优先级队列,因此需求来了:如何为RabbitMQ加 ...
- 在Linux上以服务的方式运行ASP.NET Core站点
更新:用supervisor是更好的解决方法,详见 Linux下为 dotnet 创建守护进程 要在生成环境下在Linux服务器上跑ASP.NET Core站点,首先要解决的问题是以服务的方式运行AS ...
- 我的ORM之三 -- 更新
我的ORM索引 更新语法 var 影响行数 = dbr.表.Update(实体).Where(条件).Execute(); 实体类型: 更新的实体类型和添加的实体类型一样,有三类: 1. 任何C#类. ...
- Linux 网络编程(多路复用)
服务器端代码 #include<stdio.h> #include<stdlib.h> #include<string.h> #include<sys/soc ...
- VS2012下配置MPI
并行处理结课实验,要用到MPI编程,我的电脑和VS2012都是64位的,以为MPICH也得是64位才行,结果饶了很大的弯——配置正确,添加引用之后,仍然无法识别MPI函数. 后来换了个32位的MPIC ...
- Javascript事件模型系列(四)我所理解的javascript自定义事件
被我拖延了将近一个月的javascript事件模型系列终于迎来了第四篇,也是我计划中的最后一篇,说来太惭愧了,本来计划一到两个星期写完的,谁知中间遇到了很多事情,公司的个人的,搞的自己心烦意乱浮躁了一 ...