看看这样的单元测试，大家评价一下

ricmy

void  CTest_DB::TestPMDB_ClearFileData()
{
        TEST_CASE_TRACE("Begin");

        CString strSamplePath;
        strSamplePath.Format("%s\\BaseTestCase\\SwitchView",m_strCurrentPath);

        CString strTrace;
        strTrace.Format("Create a ItemFolder %s",strSamplePath);
        TEST_CASE_TRACE(strTrace);
        CItemFolder * pItemFolder;
        PMDB_CreateFolder(strSamplePath,pItemFolder);
        if (pItemFolder)
        {
                CString strOldFileName(_T("OCR Pro 4.jpg"));
                CItemFile* pOldItem = pItemFolder->GetItemFile(strOldFileName);
                strTrace.Format("Get %s ItemFile info %s",strOldFileName);
                TEST_CASE_TRACE(strTrace);
       
                CPropertyInfo * pInfo = pOldItem->GetPropertyInfo(PROPERTYID_KEYWORD);
                pInfo->SetProperData("Test PROPERTYID_KEYWORD");
                strTrace.Format("Add KeyWord Test PROPERTYID_KEYWORD",strOldFileName);
                TEST_CASE_TRACE(strTrace);

                pInfo = pOldItem->GetPropertyInfo(PROPERTYID_AUTHOR);
                pInfo->SetProperData("Test PROPERTYID_AUTHOR");
                strTrace.Format("Add Author  Test PROPERTYID_AUTHOR");
                TEST_CASE_TRACE(strTrace);

                pInfo = pOldItem->GetPropertyInfo(PROPERTYID_MEMORY);
                pInfo->SetProperData("Test PROPERTYID_MEMORY");
                strTrace.Format("Add Memory  Test PROPERTYID_MEMORY");
                TEST_CASE_TRACE(strTrace);

                pOldItem->UpdateProperty(FALSE);

                CString strFile;
                strFile.Format("%s\\OCR Pro 4.jpg",strSamplePath);
                strTrace.Format("Call PMDB_ClearFileData to Delete file info");
                TEST_CASE_TRACE(strTrace);
                PMDB_ClearFileData(strFile);
       
                pOldItem->UpdateProperty(TRUE);
                CString strData;
                pInfo = pOldItem->GetPropertyInfo(PROPERTYID_KEYWORD);
                strData = pInfo->GetProperData();
       
                strTrace.Format("Get File %s,KeyWord = %s",strFile,strData);
                TEST_CASE_TRACE(strTrace);
       
                TEST_CASE_ASSERT(strData.CompareNoCase("Test PROPERTYID_KEYWORD") != 0);

                pInfo = pOldItem->GetPropertyInfo(PROPERTYID_AUTHOR);
                strData = pInfo->GetProperData();
                strTrace.Format("Get File %s,AUTHOR = %s",strFile,strData);
                TEST_CASE_TRACE(strTrace);
                TEST_CASE_ASSERT(strData.CompareNoCase("Test PROPERTYID_AUTHOR") != 0);
       
                pInfo = pOldItem->GetPropertyInfo(PROPERTYID_MEMORY);
                strData = pInfo->GetProperData();
                strTrace.Format("Get File %s,MEMORY = %s",strFile,strData);
                TEST_CASE_TRACE(strTrace);

                TEST_CASE_ASSERT(strData.CompareNoCase("Test PROPERTYID_MEMORY") != 0);

                strTrace.Format("Free ItemFolder");
                TEST_CASE_TRACE(strTrace);
               
                delete pItemFolder;
                pItemFolder = NULL;
        }


        TEST_CASE_TRACE("End");
}

wheetle

没有注释

ricmy

TEST_CASE_TRACE 中trace 出来的信息，基本上就是注释

JPeanut

单元测试脚本,,最重要的一点,,就是简单,,简洁..
这样的测试脚本,,估计本身需要测试..
脚本太长了,,不易交流

ricmy

JPeanut  说的很好。

但是很多时候，一个函数是需要上下文的，而且测试这个函数还需要一定的环境
在上面的测试用例中有很多部分是用在准备测试环境
我来解释下上面的代码：
整个代码要测试的函数是   PMDB_ClearFileData(strFile);
这个函数的功能是从数据库中清除一个文件的信息

要测试这样的函数，你将需要有下面几步
1.为某个文件添加信息
2.调用测试函数（ PMDB_ClearFileData(strFile);）
3.获取文件信息以确认文件信息是否为NULL

这样的测试用例有没有办法改进呢？

JPeanut

准备条件一般写在在setup的，销毁用teardown
我是做java的，我的做法是自己写一个专门用于单元测试的jar提供开发人员编写测试脚本，
简化编写脚本的难度和工作量
实施下来，效果还可以，不过要抽出这样一个公共的jar，花了很多时间

ricmy

恩，很不错的建议。
对于一些可以抽出来的部分放在setup 中是一个很好的做法

我们测试的时候是针对对象来测试的，而不是静态的代码，不同的对象创建的方法是不一样的。
比如上面的例子中，要创建一个文件的对象（CItemFile），你首先需要创建一个目录对象（CItemFolder），只有通过目录对象你才可以获得要测试的文件对象（CItemFile* pOldItem = pItemFolder->GetItemFile(strOldFileName);）

在本测试用例中，你会发现测试环境不光是代码的部分，同时还需要在硬盘上建立一定的物理测试环境。
其中【strSamplePath.Format("%s\\BaseTestCase\\SwitchView",m_strCurrentPath);】就是要求必须才系统中存在这样一个目录，并且目录下具备一个名叫【   CString strOldFileName(_T("OCR Pro 4.jpg"));】的文件

相信大家在编写测试用例的时候，也遇到这样的问题

zixuxuwu

象这些你可以把它们都抽出来，单独作为一个函数，哪里需要的时候直接调用就可以了。这样就可以使你的测试代码中的代码简洁化，而且有问题的时候也好查清楚。