Maven
  1. Maven
  2. MNG-3118

Test-classes should come before classes in the classpath

    Details

    • Type: Improvement Improvement
    • Status: Closed Closed
    • Priority: Major Major
    • Resolution: Fixed
    • Affects Version/s: 2.0.7
    • Fix Version/s: 2.0.8, 3.0-alpha-1
    • Component/s: General
    • Labels:
      None
    • Patch Submitted:
      Yes
    • Number of attachments :
      1

      Description

      Currently maven-project creates the test classpath in the order: classes, tests-classes, dependencies.
      It would be better if test-classes came first because sometimes it is useful for a test class to replace a main class during testing. The opposite case is not normally true (i.e. one would not want to override a test class with one of the main classes).

        Issue Links

          Activity

          Paul Gier made changes -
          Field Original Value New Value
          Attachment MNG-3118-maven-project-r558713.patch [ 28567 ]
          Paul Gier made changes -
          Link This issue relates to SUREFIRE-61 [ SUREFIRE-61 ]
          Brett Porter made changes -
          Fix Version/s 2.0.8 [ 13139 ]
          Kenney Westerhof made changes -
          Fix Version/s 2.1-alpha-1 [ 13143 ]
          Resolution Fixed [ 1 ]
          Status Open [ 1 ] Closed [ 6 ]
          Jim Sellers made changes -
          Link This issue is depended upon by MECLIPSE-318 [ MECLIPSE-318 ]
          Jim Sellers made changes -
          Link This issue is depended upon by MECLIPSE-318 [ MECLIPSE-318 ]
          Jim Sellers made changes -
          Link This issue relates to MECLIPSE-318 [ MECLIPSE-318 ]
          Benjamin Bentmann made changes -
          Link This issue is duplicated by MNG-2365 [ MNG-2365 ]
          Benjamin Bentmann made changes -
          Link This issue relates to MECLIPSE-388 [ MECLIPSE-388 ]

            People

            • Assignee:
              Unassigned
              Reporter:
              Paul Gier
            • Votes:
              0 Vote for this issue
              Watchers:
              5 Start watching this issue

              Dates

              • Created:
                Updated:
                Resolved: