Uploaded image for project: 'IMPALA'
  1. IMPALA
  2. IMPALA-4876

Remove _test suffix from test names that had been introduced to make names prefix-free

    Details

    • Type: Bug
    • Status: Resolved
    • Priority: Minor
    • Resolution: Fixed
    • Affects Version/s: Impala 2.9.0
    • Fix Version/s: Impala 2.9.0
    • Component/s: Infrastructure
    • Labels:
      None

      Description

      Prior to IMPALA-4735 we had added a _test suffix to some tests to make them individually selectable with impala-py.test's -k switch. Now they are not needed anymore and should be removed.

      ~/i1/tests(master) ✗$ git grep -E "def test_[^(]*_test\("
      custom_cluster/test_query_expiration.py:41:  def test_query_expiration_test(self, vector):
      query_test/test_insert.py:114:  def test_insert_test(self, vector):
      

        Activity

        Hide
        lv Lars Volker added a comment -

        IMPALA-4876: Remove _test suffix from test names

        After IMPALA-4735 has been fixed, this suffix is no longer needed to
        make test names prefix-free.

        Change-Id: Ie63d145c94c02ec67e81d0c51a33d20685fba73e
        Reviewed-on: http://gerrit.cloudera.org:8080/5886
        Reviewed-by: Jim Apple <jbapple-impala@apache.org>
        Tested-by: Impala Public Jenkins

        Show
        lv Lars Volker added a comment - IMPALA-4876 : Remove _test suffix from test names After IMPALA-4735 has been fixed, this suffix is no longer needed to make test names prefix-free. Change-Id: Ie63d145c94c02ec67e81d0c51a33d20685fba73e Reviewed-on: http://gerrit.cloudera.org:8080/5886 Reviewed-by: Jim Apple <jbapple-impala@apache.org> Tested-by: Impala Public Jenkins

          People

          • Assignee:
            lv Lars Volker
            Reporter:
            lv Lars Volker
          • Votes:
            0 Vote for this issue
            Watchers:
            1 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved:

              Development