Skip to content
This repository has been archived by the owner on May 27, 2020. It is now read-only.

Split Harness into Test and Section, add testOnly #28

Merged
merged 5 commits into from
Oct 15, 2018

Conversation

edmundnoble
Copy link
Contributor

No description provided.

@codecov-io
Copy link

codecov-io commented Sep 24, 2018

Codecov Report

Merging #28 into master will increase coverage by 3.42%.
The diff coverage is 87.5%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master      #28      +/-   ##
==========================================
+ Coverage   92.15%   95.57%   +3.42%     
==========================================
  Files          17       17              
  Lines         586      588       +2     
  Branches       24       23       -1     
==========================================
+ Hits          540      562      +22     
+ Misses         46       26      -20
Impacted Files Coverage Δ
core/shared/src/main/scala/testz/package.scala 100% <ø> (ø) ⬆️
specs2/shared/src/main/scala/testz/specs2.scala 0% <0%> (ø) ⬆️
runner/shared/src/main/scala/testz/runner.scala 100% <100%> (ø) ⬆️
tests/src/main/scala/testz/PropertySuite.scala 100% <100%> (ø) ⬆️
tests/src/main/scala/testz/ExtrasSuite.scala 100% <100%> (ø) ⬆️
tests/src/main/scala/testz/CoreSuite.scala 92.1% <100%> (ø) ⬆️
tests/src/main/scala/testz/StdlibSuite.scala 100% <100%> (ø) ⬆️
tests/src/main/scala/testz/ScalazSuite.scala 99.01% <100%> (+1.87%) ⬆️
tests/src/main/scala/testz/RunnerSuite.scala 100% <100%> (ø) ⬆️
tests/src/main/scala/testz/Main.scala 100% <100%> (ø) ⬆️
... and 7 more

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update d373669...f2e22d1. Read the comment docs.

Copy link
Member

@hrhino hrhino left a comment

Choose a reason for hiding this comment

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

I've gotta say, this looks way awesome. 💯

@@ -38,7 +46,7 @@ sealed abstract class Result

final class Fail private() extends Result {
override val toString: String = "Fail"
override def equals(other: Any): Boolean = other.asInstanceOf[AnyRef] eq this
override def equals(other: Any): Boolean = other.isInstanceOf[Fail]
Copy link
Member

Choose a reason for hiding this comment

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

is this faster?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

It's more portable from what I can tell.

@edmundnoble edmundnoble merged commit cecd77f into scalaz:master Oct 15, 2018
@xuwei-k xuwei-k mentioned this pull request Apr 22, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants