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

TAMAYA-288: Additional test coverage #14

Closed
wants to merge 1 commit into from

Conversation

peculater
Copy link
Contributor

Looks huge and scary, but it's honestly just adding tests.

Things that this PR does:

  • Significant increase in test coverage, especially mutation coverage
    ** API from 57% to 92%
    ** Core from 38% to 82%
    ** SPI-Support from ??% (mutation coverage disabled) to 86%
  • Tests now succeed with OpenJDK, Hotspot, and OpenJ9.
  • Generated a series of additional bug tickets. :-)

What this does not do:

  • Touch any implementation code. This is only tests.
  • Reduce code duplication, especially between API/Core and SPI-Support.
  • Have a great answer for OSGI testing. It ends up being a pile of mocks.

Things that this PR does
* Significant increase in test coverage, especially mutation coverage
** API from 57% to 92%
** Core from 38% to 82%
** SPI-Support from ??% (mutation coverage disabled) to 86%
* Tests now succeed with OpenJDK, Hotspot, and OpenJ9.
* Generated a series of additional bug tickets. :-)

What this does not do:
* Touch any implementation code.  This is only tests.
* Reduce code duplication, especially between API/Core and SPI-Support.
* Have a great answer for OSGI testing.  It ends up being a pile of mocks.
@asfgit asfgit closed this in 125eed2 Feb 25, 2018
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
1 participant