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

Ilya Kasnacheev commented on IGNITE-10978:
------------------------------------------

[~kbolyandra]
"Tests that cache value is copied for get, interceptor and invoke closure." why 
do we remove those tests? Sounds like a legitimate scenario. Do we have this 
kind of test in other place?

Same question about deployment. It also seems to test task deployment. Do we 
have those tested in other place? Please explain it on per file basis.

> Remove unused tests marked with unclear todo
> --------------------------------------------
>
>                 Key: IGNITE-10978
>                 URL: https://issues.apache.org/jira/browse/IGNITE-10978
>             Project: Ignite
>          Issue Type: Task
>    Affects Versions: 2.7
>            Reporter: Dmitriy Pavlov
>            Assignee: Konstantin Bolyandra
>            Priority: Major
>              Labels: newbie
>             Fix For: 2.8
>
>          Time Spent: 0.5h
>  Remaining Estimate: 0h
>
> {noformat}
> IgniteCacheTestSuite3
> // TODO GG-11141.
> //        
> GridTestUtils.addTestIfNeeded(suite,GridCacheDeploymentSelfTest.class, 
> ignoredTests);
> //        
> GridTestUtils.addTestIfNeeded(suite,GridCacheDeploymentOffHeapSelfTest.class, 
> ignoredTests);
> //        
> GridTestUtils.addTestIfNeeded(suite,GridCacheDeploymentOffHeapValuesSelfTest.class,
>  ignoredTests);
> IgniteCacheInterceptorSelfTestSuite
> // TODO GG-11141.
> //        
> GridTestUtils.addTestIfNeeded(suite,GridCacheOnCopyFlagTxPartitionedSelfTest.class,
>  ignoredTests);
> //        
> GridTestUtils.addTestIfNeeded(suite,GridCacheOnCopyFlagReplicatedSelfTest.class,
>  ignoredTests);
> //        
> GridTestUtils.addTestIfNeeded(suite,GridCacheOnCopyFlagLocalSelfTest.class, 
> ignoredTests);
> //        
> GridTestUtils.addTestIfNeeded(suite,GridCacheOnCopyFlagAtomicSelfTest.class, 
> ignoredTests);
> {noformat}
> Test classes not used, so there is no reason to keep it



--
This message was sent by Atlassian JIRA
(v7.6.3#76005)

Reply via email to