specs2 should take Future[MatchResult[T]]

[MasseGuillaume]

could not find implicit value for evidence parameter of type
org.specs2.execute.AsResult[scala.concurrent.Future[org.specs2.matcher.MatchResult[...]]]

when you have a Matcher[Future[T]] we can use await, but when we have a Future[MatchResult[T]] I think specs2 could handle it.

[etorreborre]

Done. This should be now as simple as (in 2.4-SNAPSHOT):

Future(1 must_== 1).await
Future(1 must_== 1).await(retries = 2, timeout = 1.second)

[MasseGuillaume]

Wow that was fast. Thanks!

[MasseGuillaume]

so it's

import org.specs2.time.NoTimeConversions

class _Spec extends Specification with NoTimeConversions {

I think it's better if you accept a specs2 duration and convert it internally.

---

also I think it's better if you let the user define the execution context

specs2/matcher/src/main/scala/org/specs2/matcher/FutureMatchers.scala

Line 6 in 7631010

import concurrent.ExecutionContext.Implicits.global

specs2

trait ConcurrentContext {
  implicit val context: ExecutionContext
}

trait FutureMatchers extends Expectations { context: ConcurrentContext =>
 // ...
}

user

class _Spec extends Specification with ConcurrentContext{
  implicit lazy val context = ???
}

so he could use

• ExecutionContext.Implicits.global
• play.api.libs.concurrent.Execution.Implicits.defaultContext
• etc

[etorreborre]

That's a very good suggestion and I added a trait to hold the execution context. On the Duration side of things I think that I'm going to remove specs2 conversions altogether. But I don't see how to avoid making it a breaking change :-(.

[MasseGuillaume]

Thanks, Looking good :)