weaver-test

weaver-test copied to clipboard

Includes two independent proposals per #614 1) `matches` partial pattern match combinator 2) `existsRight` combinator for asserting `ApplicativeError` Proposal 2 is more a strawman, I'm not confident in it at...

mcanlas

Suggestion: partial pattern match syntax

11

What do we think? Tried to provide more vibrant information by using `Show` Inspired by https://www.scalatest.org/user_guide/using_inside ```scala def matches[A](x: A)( f: PartialFunction[A, Expectations] )(implicit pos: SourceLocation, A: Show[A] = Show.fromToString[A]):...

mcanlas

Can't extend `Checkers` and `Discipline` in the same suite?

1

At least, I couldn't figure out how to do it. Seems like their hierarchies are conflicting? It seems like a common pattern to mix discipline tests with additional property tests....

armanbilge

Cats instances for Scalacheck `Gen`?

1

Specifically, to help with this: https://github.com/disneystreaming/weaver-test/blob/850a8a6aeb4875d916663092e362eb0bfc24bc99/docs/scalacheck.md?plain=1#L46-L53 Which is a reasonable setup IMO. Personally I'm just missing some of the handy combinators for composition. E.g. it would be nice to express...

armanbilge

`expect` broken with `inline def` and `opaque type`

1

```scala //> using lib "com.disneystreaming::weaver-cats:0.8.0" //> using testFramework "weaver.framework.CatsEffect" import weaver.* opaque type Foo = Int object Foo: inline def fromInt(x: Int): Foo = x inline def plus(x: Foo, y:...

armanbilge

Enacting The Big Wipe. This branch will keep the cross-publishing we currently have in master, and from it we will be releasing both 0.6.x and 0.7 patch versions for critical...

keynmol

Test Suites that use class loader/reflection fail on second run

3

We are using weaver for some internal libraries and found that some test suites succeed only on the first run and always fail on the second run. This impacts test...

rgueldem

If `minimumSuccessful` is set to 10 property tests will fail with "Discarded more inputs (0) than allowed" error message. [This](https://github.com/disneystreaming/weaver-test/blob/c99cced2af84d7f5598bb1773d6e4c0c49f1d991/modules/scalacheck/src/weaver/scalacheck/CheckConfig.scala#L15) calculation will result in 0 for `minimumSuccessful = 10` and...

wedens

ForEachTest resource

3

Add IOForEachSuite as a way to acquire a new instance of a resource for each test in a suite. Combining this with global makes it possible to do things that...

amumurst

Allow loading GlobalResources from libraries on the classpath.

2

As discussed in https://github.com/disneystreaming/weaver-test/issues/479. This is a minimal implmentation to allow GlobalResources to be reused in a submodule / external jar across projects. I've tested on a local project and...

aevanszen