Details

    • Type: Sub-task
    • Status: Resolved
    • Priority: Major
    • Resolution: Fixed
    • Affects Version/s: 1.0.0
    • Fix Version/s: 1.1.0
    • Component/s: tests
    • Labels:
      None

      Description

      I want to make the smoke tests code to be a bit more clear and easier to maintain. It would involve things like removing multiple strings concats, and some other things.
      One of the objective is to make sure we don't compile test framework out of the source code.

        Activity

        Hide
        cos Konstantin Boudnik added a comment -

        Something along these lines, but more work for the rest of the components needs to be done.

        Show
        cos Konstantin Boudnik added a comment - Something along these lines, but more work for the rest of the components needs to be done.
        Hide
        cos Konstantin Boudnik added a comment -

        Cleaned up some more, removed a bunch of redundant and unused code.
        While at it, fixed broken compilation of the Kite smoke test
        Groovy version is defined in one place now

        Show
        cos Konstantin Boudnik added a comment - Cleaned up some more, removed a bunch of redundant and unused code. While at it, fixed broken compilation of the Kite smoke test Groovy version is defined in one place now
        Hide
        cos Konstantin Boudnik added a comment -

        Latest patch also fixes broken by BIGTOP-2172 smoke-tests.sh and bumps up ITEST default version to 1.0.0 so the mapreduce tests won't rely on the compilation of iTest from the source.

        Show
        cos Konstantin Boudnik added a comment - Latest patch also fixes broken by BIGTOP-2172 smoke-tests.sh and bumps up ITEST default version to 1.0.0 so the mapreduce tests won't rely on the compilation of iTest from the source.
        Hide
        cos Konstantin Boudnik added a comment -

        I think it is ready for ppl to take a look at.

        Show
        cos Konstantin Boudnik added a comment - I think it is ready for ppl to take a look at.
        Hide
        oflebbe Olaf Flebbe added a comment -

        Just discovered this, looking into it.

        Show
        oflebbe Olaf Flebbe added a comment - Just discovered this, looking into it.
        Hide
        oflebbe Olaf Flebbe added a comment -

        Works for me, +1

        Show
        oflebbe Olaf Flebbe added a comment - Works for me, +1
        Hide
        cos Konstantin Boudnik added a comment -

        Pushed to the master - thanks for the review

        Show
        cos Konstantin Boudnik added a comment - Pushed to the master - thanks for the review

          People

          • Assignee:
            cos Konstantin Boudnik
            Reporter:
            cos Konstantin Boudnik
          • Votes:
            0 Vote for this issue
            Watchers:
            2 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved:

              Development