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

General testing code clean-ups #434

Open
wants to merge 3 commits into
base: kotlin-experiments
Choose a base branch
from

Conversation

garyttierney
Copy link
Contributor

@garyttierney garyttierney commented Jul 19, 2019

A WIP to clean up some of the current test code. This patch also introduces support for reading from the game cache during plugin and integration tests. This shouldn't be an issue for most contributors, who can test against their own cache. However, we are unable to distribute the same cache we are using due to copyright restrictions.

@codecov
Copy link

codecov bot commented Jul 19, 2019

Codecov Report

Merging #434 into kotlin-experiments will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@                  Coverage Diff                  @@
##             kotlin-experiments     #434   +/-   ##
=====================================================
  Coverage                 23.29%   23.29%           
  Complexity                  809      809           
=====================================================
  Files                       655      655           
  Lines                     11343    11343           
  Branches                   1591     1591           
=====================================================
  Hits                       2642     2642           
  Misses                     8446     8446           
  Partials                    255      255

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 11cee7e...a5a369d. Read the comment docs.

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.

1 participant