ZIO Test: Use Suite Execution Strategy in ProvideSomeLayerShared #3256
Conversation
| * Applies the function `f` to each element of the `Iterable[A]` and returns | ||
| * the result in a new `List[B]` using the specified execution strategy. | ||
| */ | ||
| private def foreachExec[R, E, A, B]( |
There was a problem hiding this comment.
This could be worth upstreaming to ZIO Core. It provides a nice way of abstracting over execution strategies.
|
@adamgfraser I like this as-is. But it does strike me there's an alternate API we should consider: sealed trait ExecutionStrategy[+A]
...
foreachExec(Parallel(List(1, 2, 3, 4))) { element =>
...
} |
|
@jdegoes Yes. I think that might not work as well when used with combinators that are methods on a trait versus a companion object. For example in Conceptually it seems like it might also be valuable sometimes to be able to talk about an execution strategy separate from any concrete collection (e.g. tell an application whether it is supposed to do whatever it does sequentially or in parallel). This could be handled by doing |
|
@adamgfraser Sounds good to me! |
In
provideSomeLayerSharedwe need to create one effect that models execution of all tests in the spec to provide the layer to. This PR updates the implementation to use the execution strategy for each suite in the composed effect.