[IGNITE-10758] remove from tests scaffolding annotations "@RunWith(JUnit4.class)" and their respective imports

• Type: Sub-task
• Status: Resolved
• Priority: Major
• Resolution: Fixed
• Affects Version/s: 2.8
• Fix Version/s: 2.8
• Labels: MakeTeamcityGreenAgain
• Assignee: Oleg Ignatenko
• Reporter: Oleg Ignatenko
• Created: 20/Dec/18 14:35
• Updated: 23/Feb/19 22:32
• Resolved: 30/Jan/19 13:02

Description

In the course of IGNITE-10175 and IGNITE-10176 many test classes were annotated @RunWith(JUnit4.class). This was necessary to allow gradual switching to JUnit 4 to let classes run under the version that worked for these.

After IGNITE-10177 is over, these scaffolding annotations will be not needed anymore (and will become even somewhat damaging by misleading readers to think that they serve some real purpose).

The task is to remove these annotations and respective import statements after IGNITE-10177 is merged to master. Note it was initially planned as a part of IGNITE-10177 but per discussion with EdShangGG we decided that it will be more convenient to do this in a separate ticket because this will make changes much easier to review, test and merge.

Thing worth paying attention is that mentioned annotations should be kept in one case - namely in root test class GridAbstractTest because over there these are necessary until JUnit 3 dependencies are wiped out from remaining cases listed in IGNITE-10739.

Issue Links

• depends upon: IGNITE-10177 cleanup Junit 3 from the project
• depends upon: IGNITE-10739 get rid of using JUnit 3 API in IgniteConfigVariationsAbstractTest
• is related to: IGNITE-11398 Remove leftover @RunWith(JUnit4.class)
• links to: GitHub Pull Request #5867