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;
}

 

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

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