You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@geode.apache.org by Ken Howe <kh...@pivotal.io> on 2016/02/18 00:19:41 UTC

Review Request 43681: GEODE-855: Create unit test class for Fragment class

-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/43681/
-----------------------------------------------------------

Review request for geode, Darrel Schneider and Sai Boorlagadda.


Repository: geode


Description
-------

GEODE-855: Adding more unit tests for Fragment class


GEODE-855: more unit tests for complete coverage of Fragment class


GEODE-855: Checking system property in unit test


GEODE-855: Refacotring in SimpleMemoryAllocatorImpl to allow resetting the singleton allocator for testing. Updated tests to use the memory allocator reset capability.


Revert "GEODE-855: Refacotring in SimpleMemoryAllocatorImpl to allow resetting"

This reverts commit 7bb94a9d6ed196090b5cfe616eecc18469530f79.

GEODE-855: Removed a test that didn't add coverage for Fragment


Diffs
-----

  gemfire-core/src/test/java/com/gemstone/gemfire/internal/offheap/FragmentJUnitTest.java PRE-CREATION 

Diff: https://reviews.apache.org/r/43681/diff/


Testing
-------

precheckin


Thanks,

Ken Howe


Re: Review Request 43681: GEODE-855: Create unit test class for Fragment class

Posted by Darrel Schneider <ds...@pivotal.io>.
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/43681/#review119558
-----------------------------------------------------------


Ship it!




Ship It!

- Darrel Schneider


On Feb. 17, 2016, 3:19 p.m., Ken Howe wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/43681/
> -----------------------------------------------------------
> 
> (Updated Feb. 17, 2016, 3:19 p.m.)
> 
> 
> Review request for geode, Darrel Schneider and Sai Boorlagadda.
> 
> 
> Repository: geode
> 
> 
> Description
> -------
> 
> GEODE-855: Adding more unit tests for Fragment class
> 
> 
> GEODE-855: more unit tests for complete coverage of Fragment class
> 
> 
> GEODE-855: Checking system property in unit test
> 
> 
> GEODE-855: Refacotring in SimpleMemoryAllocatorImpl to allow resetting the singleton allocator for testing. Updated tests to use the memory allocator reset capability.
> 
> 
> Revert "GEODE-855: Refacotring in SimpleMemoryAllocatorImpl to allow resetting"
> 
> This reverts commit 7bb94a9d6ed196090b5cfe616eecc18469530f79.
> 
> GEODE-855: Removed a test that didn't add coverage for Fragment
> 
> 
> Diffs
> -----
> 
>   gemfire-core/src/test/java/com/gemstone/gemfire/internal/offheap/FragmentJUnitTest.java PRE-CREATION 
> 
> Diff: https://reviews.apache.org/r/43681/diff/
> 
> 
> Testing
> -------
> 
> precheckin
> 
> 
> Thanks,
> 
> Ken Howe
> 
>