Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Mockk junit5 #34

Merged
merged 3 commits into from Feb 25, 2018
Merged

Mockk junit5 #34

merged 3 commits into from Feb 25, 2018

Conversation

FredDeschenes
Copy link
Contributor

As mentioned in #23.
Take a good look at the build.gradle file please, I couldn't get Jacoco to run locally and there's probably a way to only use mockk-dsl but I couldn't get it to work.

@codecov-io
Copy link

Codecov Report

Merging #34 into master will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@            Coverage Diff            @@
##             master      #34   +/-   ##
=========================================
  Coverage     80.36%   80.36%           
  Complexity      516      516           
=========================================
  Files            62       62           
  Lines          1375     1375           
  Branches        220      220           
=========================================
  Hits           1105     1105           
  Misses          173      173           
  Partials         97       97

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 bdeb225...1b4ef8c. Read the comment docs.

@oleksiyp oleksiyp merged commit ae10396 into mockk:master Feb 25, 2018
@oleksiyp
Copy link
Collaborator

I merged and edited few things:

  • embedded it to main mockk sub-project
  • added auto registration (don't know if it works with most recent version of JUnit5, because information was removed from docs)
  • minor refactoring
    Released as v1.7.8

@FredDeschenes FredDeschenes deleted the mockk-junit5 branch February 26, 2018 14:20
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants