[#APF-750] one jmock question: no need for dao.verify - AppFuse JIRA

AppFuse

one jmock question: no need for dao.verify

Created: 17/May/07 01:45 AM Updated: 30/Aug/07 01:57 AM Resolved: 30/Aug/07 01:57 AM

Component/s:

Build, Test, or Deploy Process

Affects Version/s:

2.0-M4

Fix Version/s:

2.0-RC1

File Attachments:

patch.txt (11 kB)

Description

« Hide

In every method of PersonManagerImplTest, there is a line

dao.verify();

but it seems that the line is not necessary, super class
MockObjectTestCase will do the verify job automatically !

so, just remove the line !!

Sort Order:

[ Permalink | « Hide ]

Matt Raible added a comment - 01/Jun/07 10:23 PM

Hopefully we can solve ~~APF-661~~ before this one. That way, we only need to update the code in SVN and it'll be updated in the tutorial.

[ Permalink | « Hide ]

Trent Bartlem added a comment - 09/Jul/07 04:52 AM

No promises, but hope this helps.

[ Permalink | « Hide ]

Matt Raible added a comment - 13/Aug/07 11:44 PM

Thanks for the patch Trent - I've applied it in SVN.

http://fisheye4.cenqua.com/changelog/appfuse?cs=2832

This issue remains open as all the tutorials still need to have this change made to them.

[ Permalink | « Hide ]

Matt Raible added a comment - 30/Aug/07 01:57 AM

Resolved manually for demos since Snippet plugin seems to have issues (~~APF-661~~).