Details

    • Type: Bug Bug
    • Status: Open
    • Priority: Major Major
    • Resolution: Unresolved
    • Affects Version/s: 0.8.0
    • Fix Version/s: None
    • Component/s: None
    • Labels:

      Description

      There is a lot of unused stuff lying in system_test. If we have converted all the bash tests to python, we should delete the unused code.

      Also there is commented code at a few places that should be deleted if it is not going to be used. For instance, search for brokerLogCksumDict in system_test/utils/kafka_system_test_utils.py.

        Activity

        Hide
        Swapnil Ghike added a comment -

        And we should also do something about this trippy diff that pops up every time one tries to run a system test

        diff --git a/system_test/testcase_to_run.json b/system_test/testcase_to_run.json
        index 8252860..c6cf17e 100644
        — a/system_test/testcase_to_run.json
        +++ b/system_test/testcase_to_run.json
        @@ -1,5 +1,5 @@

        { "ReplicaBasicTest" : [ - "testcase_0001" + "testcase_1" ] }
        Show
        Swapnil Ghike added a comment - And we should also do something about this trippy diff that pops up every time one tries to run a system test diff --git a/system_test/testcase_to_run.json b/system_test/testcase_to_run.json index 8252860..c6cf17e 100644 — a/system_test/testcase_to_run.json +++ b/system_test/testcase_to_run.json @@ -1,5 +1,5 @@ { "ReplicaBasicTest" : [ - "testcase_0001" + "testcase_1" ] }

          People

          • Assignee:
            John Fung
            Reporter:
            Swapnil Ghike
          • Votes:
            0 Vote for this issue
            Watchers:
            1 Start watching this issue

            Dates

            • Created:
              Updated:

              Development