Uploaded image for project: 'Ignite'
  1. Ignite
  2. IGNITE-10951 migration from Junit 3 to 4 phase 2 (follow-up to IGNITE-10173)
  3. IGNITE-10758

remove from tests scaffolding annotations "@RunWith(JUnit4.class)" and their respective imports

    XMLWordPrintableJSON

    Details

    • Type: Sub-task
    • Status: Resolved
    • Priority: Major
    • Resolution: Fixed
    • Affects Version/s: 2.8
    • Fix Version/s: 2.8
    • Component/s: None

      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 Eduard Shangareev 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.

        Attachments

          Issue Links

            Activity

              People

              • Assignee:
                oignatenko Oleg Ignatenko
                Reporter:
                oignatenko Oleg Ignatenko
              • Votes:
                0 Vote for this issue
                Watchers:
                4 Start watching this issue

                Dates

                • Created:
                  Updated:
                  Resolved:

                  Time Tracking

                  Estimated:
                  Original Estimate - Not Specified
                  Not Specified
                  Remaining:
                  Remaining Estimate - 0h
                  0h
                  Logged:
                  Time Spent - 20m
                  20m