Windows Embedded CE 6 msmqadm工具的一个bug

最近继续使用C++做MSMQ的开发,最简单的方法就是看微软的源代码,在做MSMQ管理部分,参考reg.cxx,发现了一个bug,其实这个bug没什么大不了,为了证明自己“眼光敏捷”,吹毛求疵的把这个bug报告一下。

 

这bug在reg.cxx文件的273行,如果执行msmqadm register install 的时候,写注册表失败就会进入这段代码。我估计这个bug的出现是因为拷贝复制导致的,因为前后的代码都非常类似,如果把这段代码封装起来,那么就不会出现这个bug了。

dwValue = 1;
if (ERROR_SUCCESS != (hr = RegSetValueEx (hKey, L"Index", 0, REG_DWORD, (BYTE *)&dwValue, sizeof(DWORD)))) {
RegCloseKey (hKey);
rsc_wprintf (IDS_CANT_WRITE_KEY, L"Keep", hr);
return 1;
}

应该改为

dwValue = 1;
if (ERROR_SUCCESS != (hr = RegSetValueEx (hKey, L"Index", 0, REG_DWORD, (BYTE *)&dwValue, sizeof(DWORD)))) {
RegCloseKey (hKey);
rsc_wprintf (IDS_CANT_WRITE_KEY, L"Index", hr);
return 1;
}

 

image

我想这段代码没有走Unit Test,至少没有做写注册表失败的Unit Test。

其实Unit Test全面覆盖真的很难,最近自己也写很多Test cases,写的代码中,有一半为测试代码。其实还远远不够,因为要覆盖全部异常情况,测试代码应该是功能代码的好几倍。任重而道远呀。

posted @ 2010-08-24 10:18  Jake Lin  阅读(691)  评论(3编辑  收藏  举报