[ 
https://issues.apache.org/jira/browse/GIRAPH-51?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13155357#comment-13155357
 ] 

Jakob Homan commented on GIRAPH-51:
-----------------------------------

bq. I think we have to separate the unit tests into two categories:
Excellent summary. Totally agree.

Patch looks great. Barring a couple minor things, I think it's ready to go:
* Need completed javadoc for MockUtils::prepareVertex()
* Javadoc typo in ReflectionUtils::setField: /**t ry -> /** try
* Javadoc typo in ReflectionUtils::findDeclaredField: it's -> its
* Javadoc typo in MockUtils::MockedEnvironment: enviroment -> environment

                
> Provide unit testing tool for Giraph algorithms
> -----------------------------------------------
>
>                 Key: GIRAPH-51
>                 URL: https://issues.apache.org/jira/browse/GIRAPH-51
>             Project: Giraph
>          Issue Type: Improvement
>            Reporter: Jakob Homan
>            Assignee: Sebastian Schelter
>         Attachments: GIRAPH-51-2.patch, GIRAPH-51.patch
>
>
> It would be nice to have a little tool, similar to MRUnit, that would allow 
> Giraph application writers to quickly unit test their algorithms.  The tool 
> could take a Vertex implementation, a set of input and expected output and 
> verify that after the specified number of supersteps, we've gotten what we 
> expect.

--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators: 
https://issues.apache.org/jira/secure/ContactAdministrators!default.jspa
For more information on JIRA, see: http://www.atlassian.com/software/jira

        

Reply via email to