用easymock怎样测试该方法

0 0

用easymock怎样测试该方法10

待测方法:

public void moveDataToHistroyTable(Integer docId) throws IllegalAccessException, InvocationTargetException {
		DocDocument document = documentDao.get(docId);
		DocDocumentHst documentHst = new DocDocumentHst();
		BeanUtils.copyProperties(documentHst, document);
		documentHst.setDocDocumentHstId(document.getDocDocumentId());
		documentHst.setCommonDocEmergency(document.getCommonDocEmergency());
		documentHst.setCommonDocSecret(document.getCommonDocSecret());
		documentHst.setCommonDoctype(document.getCommonDoctype());
		documentHst.setDocSecrettimeUnit(document.getDocSecrettimeUnit());
		documentHst.setDocSpecialReturnreceiptType(document.getDocSpecialReturnreceiptType());
		documentHstDao.save(documentHst);
	}

测试方法:

@Test
	public void moveDataToHistroyTable() throws IllegalAccessException,
			InvocationTargetException {
		CommonDoctype commonDoctype = new CommonDoctype();
		commonDoctype.setDoctypeId(1);
		commonDoctype.setDoctypeCode("docType");

		CommonUnit unit = new CommonUnit();
		unit.setUnitId(1);
		unit.setUnitCode("4409131234");

		CommonUnit unit2 = new CommonUnit();
		unit.setUnitId(1);
		unit.setUnitCode("4409131244");

		DocDocument document = new DocDocument();
		document.setDocDocumentId(1);
		document.setDocCode("1111-ITDCL");
		document.setDocTitle("test");
		document.setCommonDoctype(commonDoctype);

		DocSend send = new DocSend();
		send.setDocSendId(1);
		send.setDocDocument(document);
		document.getDocSends().add(send);

		DocSendUnit sendUnit = new DocSendUnit();
		sendUnit.setCommonUnit(unit);
		sendUnit.setDocIsreceived(1);

		DocSendUnit sendUnit2 = new DocSendUnit();
		sendUnit.setCommonUnit(unit2);
		sendUnit.setDocIsreceived(1);

		send.getDocSendUnits().add(sendUnit);
		send.getDocSendUnits().add(sendUnit2);
		
		EasyMock.expect(mockDocumentDao.get(1)).andReturn(document);
		EasyMock.replay(mockDocumentDao);
		
		DocDocumentHst docHst = new DocDocumentHst();
		BeanUtils.copyProperties(docHst,document);
		docHst.setDocDocumentHstId(document.getDocDocumentId());
		docHst.setCommonDocEmergency(document.getCommonDocEmergency());
		docHst.setCommonDocSecret(document.getCommonDocSecret());
		docHst.setCommonDoctype(document.getCommonDoctype());
		docHst.setDocSecrettimeUnit(document.getDocSecrettimeUnit());
		docHst.setDocSpecialReturnreceiptType(document.getDocSpecialReturnreceiptType());
		mockDocumentHstDao.save(docHst);
		EasyMock.expectLastCall(); 
		EasyMock.replay(mockDocumentHstDao);
		
		this.pigeonholeSerivce.moveDataToHistroyTable(1);
		
		EasyMock.verify(mockDocumentDao);
		EasyMock.verify(mockDocumentHstDao);
	}

异常：

引用

java.lang.AssertionError:
Expectation failure on verify:
save(com.itdcl.adapter.pojo.DocDocumentHst@f39b3a): expected: 1, actual: 0
at org.easymock.internal.MocksControl.verify(MocksControl.java:100)
at org.easymock.EasyMock.verify(EasyMock.java:1597)
at com.itdcl.adapter.service.PigeonholeServiceTest.moveDataToHistroyTable(PigeonholeServiceTest.java:201)
at sun.reflect.NativeMethodAccessorImpl.invoke0(Native Method)
at sun.reflect.NativeMethodAccessorImpl.invoke(NativeMethodAccessorImpl.java:39)
at sun.reflect.DelegatingMethodAccessorImpl.invoke(DelegatingMethodAccessorImpl.java:25)
at java.lang.reflect.Method.invoke(Method.java:597)
at org.junit.internal.runners.TestMethod.invoke(TestMethod.java:59)
at org.junit.internal.runners.MethodRoadie.runTestMethod(MethodRoadie.java:98)
at org.junit.internal.runners.MethodRoadie$2.run(MethodRoadie.java:79)
at org.junit.internal.runners.MethodRoadie.runBeforesThenTestThenAfters(MethodRoadie.java:87)
at org.junit.internal.runners.MethodRoadie.runTest(MethodRoadie.java:77)
at org.junit.internal.runners.MethodRoadie.run(MethodRoadie.java:42)
at org.junit.internal.runners.JUnit4ClassRunner.invokeTestMethod(JUnit4ClassRunner.java:88)
at org.junit.internal.runners.JUnit4ClassRunner.runMethods(JUnit4ClassRunner.java:51)
at org.junit.internal.runners.JUnit4ClassRunner$1.run(JUnit4ClassRunner.java:44)
at org.junit.internal.runners.ClassRoadie.runUnprotected(ClassRoadie.java:27)
at org.junit.internal.runners.ClassRoadie.runProtected(ClassRoadie.java:37)
at org.junit.internal.runners.JUnit4ClassRunner.run(JUnit4ClassRunner.java:42)
at org.eclipse.jdt.internal.junit4.runner.JUnit4TestReference.run(JUnit4TestReference.java:38)
at org.eclipse.jdt.internal.junit.runner.TestExecution.run(TestExecution.java:38)
at org.eclipse.jdt.internal.junit.runner.RemoteTestRunner.runTests(RemoteTestRunner.java:460)
at org.eclipse.jdt.internal.junit.runner.RemoteTestRunner.runTests(RemoteTestRunner.java:673)
at org.eclipse.jdt.internal.junit.runner.RemoteTestRunner.run(RemoteTestRunner.java:386)
at org.eclipse.jdt.internal.junit.runner.RemoteTestRunner.main(RemoteTestRunner.java:196)

请问各位怎样修改该测试方法通过测试，谢谢！
问题补充
不错，通过啦。谢啦
问题补充

moonranger 写道

NicholasBugs 写道

这两行代码有问题：
改为
mockDocumentHstDao.save((DocDocumentHst)EasyMock.anyObject());
EasyMock.expectLastCall();
就可以了，这时候easyMock就不care你传的入参是什么了。

用anyObject貌似不合适吧？其实这个地方应该验证传到save方法里的DocDocumentHst是否正确。可以写一个自定义的argument matcher，在里面assert参数的正确性。

请指点一二，或完善这个测试方法也行。

软件测试单元测试

2010年12月31日 10:32

xuzhfa123
211
0 0 0

4个答案按时间排序按投票排序

0 0

而且再仔细想一想，Mock应该提供动态替换对象的功能，这样代码就应该看起来：

Mock.replace(originalObj, mockObj);
x.originalMethod();//函数里存在originalObj的局部变量
Expect...

大家觉得对不对？要不然可能总会出现这种得把源代码复制一份，将其中不好构造的对象换成MOCK对象的误用？
不过再一想，Mock目的既然是替换难构造的对象，不容易出现的方法返回，那是不是说明，被Mock替换的对象（函数直接和周围环境交互的地方）也尽量不应该出现在方法的局部变量里，毕竟局部变量，你就只能像LZ一样COPY一段代码来测试，而且还不能说明方法被测试了。

2010年12月31日 10:32

piao_bo_yi
70
0 0 1

添加评论

0 0

前两天我也碰到了和LZ相同的问题。仔细想一想，LZ的函数提供了三个职责，查询、转换、存储，而LZ的测试思路是写一个DAO的MOCK，拷贝原有函数代码来写测试，测试目的是Save方法是否调用，Save调用能保证验证方法的正确性吗（那只是你函数职责的一个而已，而且还只能保证他被调用）？并且再别人改出错误后，原有测试仍能继续使用？很明显，都不能。
所以，应将函数职责分开

public DocDocumentHst GetDDHstByDD(DocDocument document )
{
	DocDocumentHst documentHst = new DocDocumentHst();   
	BeanUtils.copyProperties(documentHst, document);   
        documentHst.setDocDocumentHstId(document.getDocDocumentId());           
	documentHst.setCommonDocEmergency(document.getCommonDocEmergency());   
        documentHst.setCommonDocSecret(document.getCommonDocSecret());   
       	documentHst.setCommonDoctype(document.getCommonDoctype());   
        documentHst.setDocSecrettimeUnit(document.getDocSecrettimeUnit());   
        documentHst.setDocSpecialReturnreceiptType(document.getDocSpecialReturnreceiptType());  

	return documentHst;
}

然后主要测试该转换函数（输入输出式测试），DAO的测试应该放到DAO中，经复合的函数只调用这三个被测试的函数，如果子部分全部正确，复合后的函数其实不用测试。这样的测试才有意义。COPY原有代码的测试行为我想是根本不行的。

2010年12月31日 10:32

piao_bo_yi
70
0 0 1

添加评论

0 0

NicholasBugs 写道

这两行代码有问题：
改为
mockDocumentHstDao.save((DocDocumentHst)EasyMock.anyObject());
EasyMock.expectLastCall();
就可以了，这时候easyMock就不care你传的入参是什么了。

用anyObject貌似不合适吧？其实这个地方应该验证传到save方法里的DocDocumentHst是否正确。可以写一个自定义的argument matcher，在里面assert参数的正确性。

2010年12月31日 10:32

moonranger
34
0 0 0

添加评论

0 0

这两行代码有问题：
mockDocumentHstDao.save(docHst);
EasyMock.expectLastCall();

在这里你预期的行为是：mockDocumentHstDao的save方法会被调用，并且入参是你new出来的一个对象docHst （com.itdcl.adapter.pojo.DocDocumentHst@f39b3a）。
而当你的测试用例执行的时候，你的mockDocumentHstDao接收到的是另一个DocDocumentHst对象，因此verify是不通过的。

改为
mockDocumentHstDao.save((DocDocumentHst)EasyMock.anyObject());
EasyMock.expectLastCall();
就可以了，这时候easyMock就不care你传的入参是什么了。

2010年12月31日 10:32