Details

    • Type: Bug
    • Status: Resolved
    • Priority: Major
    • Resolution: Fixed
    • Affects Version/s: None
    • Fix Version/s: 1.4.0
    • Component/s: Build
    • Labels:
      None

      Description

      It is because, the unit tests in flume-ng-tests module requires other modules to have been packaged as jars. When mvn test is invoked, no module is packaged (they are just compiled and tested).

      The solution is to run the tests in flume-ng-tests module in package phase or later.

      1. FLUME-2011.patch
        0.7 kB
        Thiruvalluvan M. G.

        Activity

        Hide
        thiru_mg Thiruvalluvan M. G. added a comment -

        This patch "postpones" the tests of flume-ng-tests module until package.

        If the user invokes mvn package or mvn xyz where xyz is a phase after package phase of Maven, the behavior is identical to that existed before.

        If the user invokes a phase earlier than package, no tests will be run for flume-ng-tests modules. That is the build failure seen so far does not show up and the build succeeds.

        Show
        thiru_mg Thiruvalluvan M. G. added a comment - This patch "postpones" the tests of flume-ng-tests module until package. If the user invokes mvn package or mvn xyz where xyz is a phase after package phase of Maven, the behavior is identical to that existed before. If the user invokes a phase earlier than package , no tests will be run for flume-ng-tests modules. That is the build failure seen so far does not show up and the build succeeds.
        Hide
        mpercy Mike Percy added a comment -

        +1, looks good

        Show
        mpercy Mike Percy added a comment - +1, looks good
        Hide
        mpercy Mike Percy added a comment -

        I agree this was annoying, thanks for the fix

        Show
        mpercy Mike Percy added a comment - I agree this was annoying, thanks for the fix
        Hide
        mpercy Mike Percy added a comment -

        Patch pushed to trunk & flume-1.4 branches.

        Thiruvalluvan M. G.: I just added you to the Flume contributors list in JIRA, so now you are able to assign Flume JIRAs to yourself.

        Regards,
        Mike

        Show
        mpercy Mike Percy added a comment - Patch pushed to trunk & flume-1.4 branches. Thiruvalluvan M. G. : I just added you to the Flume contributors list in JIRA, so now you are able to assign Flume JIRAs to yourself. Regards, Mike
        Hide
        hudson Hudson added a comment -

        Integrated in flume-trunk #399 (See https://builds.apache.org/job/flume-trunk/399/)
        FLUME-2011. "mvn test" fails without package build (Revision ea87b1192a839cd376075eba351da7514cb66952)

        Result = UNSTABLE
        mpercy : http://git-wip-us.apache.org/repos/asf/flume/repo?p=flume.git&a=commit&h=ea87b1192a839cd376075eba351da7514cb66952
        Files :

        • flume-ng-tests/pom.xml
        Show
        hudson Hudson added a comment - Integrated in flume-trunk #399 (See https://builds.apache.org/job/flume-trunk/399/ ) FLUME-2011 . "mvn test" fails without package build (Revision ea87b1192a839cd376075eba351da7514cb66952) Result = UNSTABLE mpercy : http://git-wip-us.apache.org/repos/asf/flume/repo?p=flume.git&a=commit&h=ea87b1192a839cd376075eba351da7514cb66952 Files : flume-ng-tests/pom.xml

          People

          • Assignee:
            thiru_mg Thiruvalluvan M. G.
            Reporter:
            thiru_mg Thiruvalluvan M. G.
          • Votes:
            0 Vote for this issue
            Watchers:
            3 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved:

              Development