Skip to content
This repository has been archived by the owner on Jan 20, 2022. It is now read-only.

Feature/use arbitrary in tests #249

Merged
merged 3 commits into from
Sep 24, 2013
Merged

Conversation

ianoc
Copy link
Collaborator

@ianoc ianoc commented Sep 23, 2013

No description provided.

…calacheck idiomatic.

use more scalacheck than rand for generation
…/use_arbitrary_in_tests

Conflicts:
	summingbird-core/src/test/scala/com/twitter/summingbird/TestGraphGenerators.scala
implicit def genSource2: Gen[KeyedProducer[Storm, Int, Int]] = Gen.choose(1,100000000).map(x => IdentityKeyedProducer(Storm.source(TraversableSpout(List[(Int, Int)]()))))

implicit def arbSource1: Arbitrary[Producer[Storm, Int]] = Arbitrary(Gen.listOfN(5000,arbInt.arbitrary).map{x: List[Int] => Storm.source(TraversableSpout(x))})
implicit def arbSource2: Arbitrary[KeyedProducer[Storm, Int, Int]] = Arbitrary(Gen.listOfN(5000,arbitaryIntPair.arbitrary).map{x: List[(Int, Int)] => IdentityKeyedProducer(Storm.source(TraversableSpout(x)))})
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

replace arbitraryIntPair.arbitrary with Arbitrary[(Int, Int)].arbitrary and delete arbitraryIntPair

johnynek added a commit that referenced this pull request Sep 24, 2013
@johnynek johnynek merged commit 18ef238 into develop Sep 24, 2013
@johnynek johnynek deleted the feature/use_arbitrary_in_tests branch September 24, 2013 01:31
snoble pushed a commit to snoble/summingbird that referenced this pull request Sep 8, 2017
Moving Trampoline flatMap impl into trait.
# for free to subscribe to this conversation on GitHub. Already have an account? #.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants