[jira] [Commented] (GEODE-1441) Distributed transaction unit tests confusing

["Darrel Schneider (JIRA)" <ji...@apache.org>]

    [ https://issues.apache.org/jira/browse/GEODE-1441?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15301151#comment-15301151 ] 

Darrel Schneider commented on GEODE-1441:
-----------------------------------------

These unit tests are part of GEODE-16.
The current plan, as I understand it, was to leave the partial implementation of GEODE-16 in the develop branch.
These tests are part of that partial implementation and they do provide test coverage of it.
I think the idea was to have unit tests that use the partial implementation of GEODE-16 so that it would not rot. If that is correct then these tests should not be removed but left as is.


> Distributed transaction unit tests confusing
> --------------------------------------------
>
>                 Key: GEODE-1441
>                 URL: https://issues.apache.org/jira/browse/GEODE-1441
>             Project: Geode
>          Issue Type: Bug
>          Components: transactions
>            Reporter: Kirk Lund
>
> Fix this mess...
> {noformat}
> @Category(DistributedTest.class)
> public class PRDistTXWithVersionsDUnitTest extends PRTransactionWithVersionsDUnitTest {
>   @Override
>   public Properties getDistributedSystemProperties() {
>     Properties props = super.getDistributedSystemProperties();
>     props.setProperty(DistributionConfig.DISTRIBUTED_TRANSACTIONS_NAME, "true");
>     return props;
>   }
>   
>   @Ignore("[DISTTX] TODO test overridden and intentionally left blank as they fail.")
>   @Test
>   public void testBasicPRTransactionRedundancy0() {
>   }
>   @Ignore("[DISTTX] TODO test overridden and intentionally left blank as they fail.")
>   @Test
>   public void testBasicPRTransactionRedundancy1() {
>   }
>   @Ignore("[DISTTX] TODO test overridden and intentionally left blank as they fail.")
>   @Test
>   public void testBasicPRTransactionRedundancy2() {
>   }
>   @Ignore("[DISTTX] TODO test overridden and intentionally left blank as they fail.")
>   @Test
>   public void testBasicPRTransactionNoDataRedundancy0() {
>   }
>   @Ignore("[DISTTX] TODO test overridden and intentionally left blank as they fail.")
>   @Test
>   public void testBasicPRTransactionNoDataRedundancy1() {
>   }
>   @Ignore("[DISTTX] TODO test overridden and intentionally left blank as they fail.")
>   @Test
>   public void testBasicPRTransactionNoDataRedundancy2() {
>   }
> }
> {noformat}



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)