You are viewing a plain text version of this content. The canonical link for it is here.
Posted to issues@ignite.apache.org by "Ryabov Dmitrii (JIRA)" <ji...@apache.org> on 2019/01/21 09:05:00 UTC

[jira] [Commented] (IGNITE-10978) Remove unused tests marked with unclear todo

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

Ryabov Dmitrii commented on IGNITE-10978:
-----------------------------------------

[~kbolyandra], [~dpavlov], hello,

I think we should uncomment tests instead of removing. As I see, we have some of these classes in the codebase, so, I propose to uncomment existing tests and run [TeamCity|https://ci.ignite.apache.org/].

As for nonexisting tests - they could be renamed. Konstantin, please, find a commit where they were removed or renamed?

> 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: 10m
>  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)