crunch-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Jesse Anderson (JIRA)" <>
Subject [jira] [Commented] (CRUNCH-322) StoreLastEmitter Should Be Part of crunch-test
Date Wed, 15 Jan 2014 23:22:21 GMT


Jesse Anderson commented on CRUNCH-322:

Then I'd recommend changing the unit tests in src/test/java/org/apache/crunch/fn/ that currently
use StoreLastEmitter to start using InMemoryEmitter. That was how I finally found some unit
test examples for Crunch.

I'd also add a something about unit testing to the new user guide.

> StoreLastEmitter Should Be Part of crunch-test
> ----------------------------------------------
>                 Key: CRUNCH-322
>                 URL:
>             Project: Crunch
>          Issue Type: Improvement
>          Components: Core
>    Affects Versions: 0.9.0
>            Reporter: Jesse Anderson
>            Assignee: Josh Wills
> There aren't many ways of unit testing Crunch that are built in. I'm using crunch-core/src/test/java/org/apache/crunch/fn/
to test outputs from DoFN classes.
> This class is not distributed with the crunch-core JAR because it is in the test package.
This class should be moved to the non-test source for crunch-test.

This message was sent by Atlassian JIRA

View raw message