Accumulo
  1. Accumulo
  2. ACCUMULO-1944

Cobertura not working for functional tests in 1.5.x and earlier

    Details

    • Type: Bug Bug
    • Status: Resolved
    • Priority: Major Major
    • Resolution: Fixed
    • Affects Version/s: 1.4.4, 1.5.0
    • Fix Version/s: 1.4.5, 1.5.1
    • Component/s: test
    • Labels:

      Description

      The -C argument to test/system/auto/run.py should enable running instrumented functional tests to analyze coverage using Cobertura. The capability is not functional; code is not instrumented even when Cobertura is available, and tests are not run with the instrumented code.

      1. ACCUMULO-1944-1.5.v2.patch
        9 kB
        Bill Havanki
      2. ACCUMULO-1944-1.5.patch
        9 kB
        Bill Havanki
      3. ACCUMULO-1944-1.4.v2.patch
        9 kB
        Bill Havanki
      4. ACCUMULO-1944-1.4.patch
        9 kB
        Bill Havanki

        Issue Links

          Activity

          Hide
          Josh Elser added a comment -

          Cool! HTH.

          Show
          Josh Elser added a comment - Cool! HTH.
          Hide
          Bill Havanki added a comment -

          Thanks for the feedback, Josh! I had made two separate patches, one for 1.4 and another for 1.5. So, you are correct, I applied both. I can understand that they come out as duplicates since 1.4 gets merged into 1.5 during the process. That merge piece was something I didn't really get until Mike Drob explained it today.

          Next time, I'll try to resolve it during the merge. That makes a lot of sense now. Thanks again!

          Show
          Bill Havanki added a comment - Thanks for the feedback, Josh! I had made two separate patches, one for 1.4 and another for 1.5. So, you are correct, I applied both. I can understand that they come out as duplicates since 1.4 gets merged into 1.5 during the process. That merge piece was something I didn't really get until Mike Drob explained it today. Next time, I'll try to resolve it during the merge. That makes a lot of sense now. Thanks again!
          Hide
          Josh Elser added a comment -

          I'm a little confused – I'm looking at the 1.5.1-SNAPSHOT branch: 198b1abd9e81d1af55220bf22a0b4e6ec1d18048 and 8669b80f98da34da1b709d1ed4f14647cc1f952c seem to be duplicated by 950f144c8e3517053af704189f0fdfd769487dfd and c23126a86d23566b154747d153bf489a09684c97, respectively. They seem to be the (about) the same changes. I'm guessing that you applied both the 1.4 and the 1.5 patches?

          In the future, you should just be applying the patch to the lowest-version branch which the issue will be fixed in, and then merge it through. This will be a change for you in process as you can now correctly resolve merge conflicts during the merge itself. I think Sean Busbey and Mike Drob came up with an alternative plan that we might want to look into adopting that would allow contributors to give that first patch, and then subsequent fixes for the merge conflicts while avoiding the duplicate commits.

          Having these duplicated commits is what we want to try to avoid as much as possible as tools like git-bisect break down really quickly. Not a big deal that we need to do anything about now, but I just wanted to make sure you're aware

          Show
          Josh Elser added a comment - I'm a little confused – I'm looking at the 1.5.1-SNAPSHOT branch: 198b1abd9e81d1af55220bf22a0b4e6ec1d18048 and 8669b80f98da34da1b709d1ed4f14647cc1f952c seem to be duplicated by 950f144c8e3517053af704189f0fdfd769487dfd and c23126a86d23566b154747d153bf489a09684c97, respectively. They seem to be the (about) the same changes. I'm guessing that you applied both the 1.4 and the 1.5 patches? In the future, you should just be applying the patch to the lowest-version branch which the issue will be fixed in, and then merge it through. This will be a change for you in process as you can now correctly resolve merge conflicts during the merge itself. I think Sean Busbey and Mike Drob came up with an alternative plan that we might want to look into adopting that would allow contributors to give that first patch, and then subsequent fixes for the merge conflicts while avoiding the duplicate commits. Having these duplicated commits is what we want to try to avoid as much as possible as tools like git-bisect break down really quickly. Not a big deal that we need to do anything about now, but I just wanted to make sure you're aware
          Hide
          ASF subversion and git services added a comment -

          Commit c23126a86d23566b154747d153bf489a09684c97 in branch refs/heads/master from Bill Havanki
          [ https://git-wip-us.apache.org/repos/asf?p=accumulo.git;h=c23126a ]

          ACCUMULO-1944 Restore test runs when Cobertura not installed

          The changes in the prior commit to re-enable test coverage assume that Cobertura
          is available. When it isn't, functional test runs fail. This commit allows runs
          to proceed when Cobertura is not installed.

          Show
          ASF subversion and git services added a comment - Commit c23126a86d23566b154747d153bf489a09684c97 in branch refs/heads/master from Bill Havanki [ https://git-wip-us.apache.org/repos/asf?p=accumulo.git;h=c23126a ] ACCUMULO-1944 Restore test runs when Cobertura not installed The changes in the prior commit to re-enable test coverage assume that Cobertura is available. When it isn't, functional test runs fail. This commit allows runs to proceed when Cobertura is not installed.
          Hide
          ASF subversion and git services added a comment -

          Commit 950f144c8e3517053af704189f0fdfd769487dfd in branch refs/heads/master from Bill Havanki
          [ https://git-wip-us.apache.org/repos/asf?p=accumulo.git;h=950f144 ]

          ACCUMULO-1944 Fix coverage for functional tests

          The -C flag for test/system/auto/run.py did not work. This changeset
          allows it to work again if a functional Cobertura installation is
          placed under $ACCUMULO_HOME/lib/test/cobertura.

          The code for producing instrumented Accumulo code was inactive and
          out of date. It was reworked so that, if -C is passed, the Accumulo
          JARs are instrumented and placed into a location ahead of their
          standard location in the test classpath. (If -C is not passed, any
          instrumented JARs are removed.) The classpath is also dynamically
          adjusted to include whatever Cobertura JAR is available; its name
          includes a version number (as of 2.0.x).

          The command-line scripts shipped with Cobertura 2.0.x are out of
          date and do not work out of the box. Pull request #102 was submitted
          to cobertura/cobertura on Github to fix the problem; in lieu of that,
          the scripts must be manually updated to use a correct classpath.

          Tested with Cobertura 2.0.3 / 2.0.4-SNAPSHOT with needed script
          updates.

          Show
          ASF subversion and git services added a comment - Commit 950f144c8e3517053af704189f0fdfd769487dfd in branch refs/heads/master from Bill Havanki [ https://git-wip-us.apache.org/repos/asf?p=accumulo.git;h=950f144 ] ACCUMULO-1944 Fix coverage for functional tests The -C flag for test/system/auto/run.py did not work. This changeset allows it to work again if a functional Cobertura installation is placed under $ACCUMULO_HOME/lib/test/cobertura. The code for producing instrumented Accumulo code was inactive and out of date. It was reworked so that, if -C is passed, the Accumulo JARs are instrumented and placed into a location ahead of their standard location in the test classpath. (If -C is not passed, any instrumented JARs are removed.) The classpath is also dynamically adjusted to include whatever Cobertura JAR is available; its name includes a version number (as of 2.0.x). The command-line scripts shipped with Cobertura 2.0.x are out of date and do not work out of the box. Pull request #102 was submitted to cobertura/cobertura on Github to fix the problem; in lieu of that, the scripts must be manually updated to use a correct classpath. Tested with Cobertura 2.0.3 / 2.0.4-SNAPSHOT with needed script updates.
          Hide
          Bill Havanki added a comment - - edited

          I applied the patches. I didn't apply any changes to 1.6.x or master, as the updates don't make sense for them - the test framework changed.

          Since this is my first commit, I'd be perfectly fine if anyone feels the need to double-check what I did. Mike Drob gave me a good tutorial on the process, so hopefully I got it all correct. Thanks.

          Show
          Bill Havanki added a comment - - edited I applied the patches. I didn't apply any changes to 1.6.x or master, as the updates don't make sense for them - the test framework changed. Since this is my first commit, I'd be perfectly fine if anyone feels the need to double-check what I did. Mike Drob gave me a good tutorial on the process, so hopefully I got it all correct. Thanks.
          Hide
          ASF subversion and git services added a comment -

          Commit c23126a86d23566b154747d153bf489a09684c97 in branch refs/heads/1.6.0-SNAPSHOT from Bill Havanki
          [ https://git-wip-us.apache.org/repos/asf?p=accumulo.git;h=c23126a ]

          ACCUMULO-1944 Restore test runs when Cobertura not installed

          The changes in the prior commit to re-enable test coverage assume that Cobertura
          is available. When it isn't, functional test runs fail. This commit allows runs
          to proceed when Cobertura is not installed.

          Show
          ASF subversion and git services added a comment - Commit c23126a86d23566b154747d153bf489a09684c97 in branch refs/heads/1.6.0-SNAPSHOT from Bill Havanki [ https://git-wip-us.apache.org/repos/asf?p=accumulo.git;h=c23126a ] ACCUMULO-1944 Restore test runs when Cobertura not installed The changes in the prior commit to re-enable test coverage assume that Cobertura is available. When it isn't, functional test runs fail. This commit allows runs to proceed when Cobertura is not installed.
          Hide
          ASF subversion and git services added a comment -

          Commit 950f144c8e3517053af704189f0fdfd769487dfd in branch refs/heads/1.6.0-SNAPSHOT from Bill Havanki
          [ https://git-wip-us.apache.org/repos/asf?p=accumulo.git;h=950f144 ]

          ACCUMULO-1944 Fix coverage for functional tests

          The -C flag for test/system/auto/run.py did not work. This changeset
          allows it to work again if a functional Cobertura installation is
          placed under $ACCUMULO_HOME/lib/test/cobertura.

          The code for producing instrumented Accumulo code was inactive and
          out of date. It was reworked so that, if -C is passed, the Accumulo
          JARs are instrumented and placed into a location ahead of their
          standard location in the test classpath. (If -C is not passed, any
          instrumented JARs are removed.) The classpath is also dynamically
          adjusted to include whatever Cobertura JAR is available; its name
          includes a version number (as of 2.0.x).

          The command-line scripts shipped with Cobertura 2.0.x are out of
          date and do not work out of the box. Pull request #102 was submitted
          to cobertura/cobertura on Github to fix the problem; in lieu of that,
          the scripts must be manually updated to use a correct classpath.

          Tested with Cobertura 2.0.3 / 2.0.4-SNAPSHOT with needed script
          updates.

          Show
          ASF subversion and git services added a comment - Commit 950f144c8e3517053af704189f0fdfd769487dfd in branch refs/heads/1.6.0-SNAPSHOT from Bill Havanki [ https://git-wip-us.apache.org/repos/asf?p=accumulo.git;h=950f144 ] ACCUMULO-1944 Fix coverage for functional tests The -C flag for test/system/auto/run.py did not work. This changeset allows it to work again if a functional Cobertura installation is placed under $ACCUMULO_HOME/lib/test/cobertura. The code for producing instrumented Accumulo code was inactive and out of date. It was reworked so that, if -C is passed, the Accumulo JARs are instrumented and placed into a location ahead of their standard location in the test classpath. (If -C is not passed, any instrumented JARs are removed.) The classpath is also dynamically adjusted to include whatever Cobertura JAR is available; its name includes a version number (as of 2.0.x). The command-line scripts shipped with Cobertura 2.0.x are out of date and do not work out of the box. Pull request #102 was submitted to cobertura/cobertura on Github to fix the problem; in lieu of that, the scripts must be manually updated to use a correct classpath. Tested with Cobertura 2.0.3 / 2.0.4-SNAPSHOT with needed script updates.
          Hide
          ASF subversion and git services added a comment -

          Commit 8669b80f98da34da1b709d1ed4f14647cc1f952c in branch refs/heads/1.6.0-SNAPSHOT from Bill Havanki
          [ https://git-wip-us.apache.org/repos/asf?p=accumulo.git;h=8669b80 ]

          ACCUMULO-1944 Restore test runs when Cobertura not installed

          The changes in the prior commit to re-enable test coverage assume that Cobertura
          is available. When it isn't, functional test runs fail. This commit allows runs
          to proceed when Cobertura is not installed.

          Show
          ASF subversion and git services added a comment - Commit 8669b80f98da34da1b709d1ed4f14647cc1f952c in branch refs/heads/1.6.0-SNAPSHOT from Bill Havanki [ https://git-wip-us.apache.org/repos/asf?p=accumulo.git;h=8669b80 ] ACCUMULO-1944 Restore test runs when Cobertura not installed The changes in the prior commit to re-enable test coverage assume that Cobertura is available. When it isn't, functional test runs fail. This commit allows runs to proceed when Cobertura is not installed.
          Hide
          ASF subversion and git services added a comment -

          Commit c23126a86d23566b154747d153bf489a09684c97 in branch refs/heads/1.5.1-SNAPSHOT from Bill Havanki
          [ https://git-wip-us.apache.org/repos/asf?p=accumulo.git;h=c23126a ]

          ACCUMULO-1944 Restore test runs when Cobertura not installed

          The changes in the prior commit to re-enable test coverage assume that Cobertura
          is available. When it isn't, functional test runs fail. This commit allows runs
          to proceed when Cobertura is not installed.

          Show
          ASF subversion and git services added a comment - Commit c23126a86d23566b154747d153bf489a09684c97 in branch refs/heads/1.5.1-SNAPSHOT from Bill Havanki [ https://git-wip-us.apache.org/repos/asf?p=accumulo.git;h=c23126a ] ACCUMULO-1944 Restore test runs when Cobertura not installed The changes in the prior commit to re-enable test coverage assume that Cobertura is available. When it isn't, functional test runs fail. This commit allows runs to proceed when Cobertura is not installed.
          Hide
          ASF subversion and git services added a comment -

          Commit 950f144c8e3517053af704189f0fdfd769487dfd in branch refs/heads/1.5.1-SNAPSHOT from Bill Havanki
          [ https://git-wip-us.apache.org/repos/asf?p=accumulo.git;h=950f144 ]

          ACCUMULO-1944 Fix coverage for functional tests

          The -C flag for test/system/auto/run.py did not work. This changeset
          allows it to work again if a functional Cobertura installation is
          placed under $ACCUMULO_HOME/lib/test/cobertura.

          The code for producing instrumented Accumulo code was inactive and
          out of date. It was reworked so that, if -C is passed, the Accumulo
          JARs are instrumented and placed into a location ahead of their
          standard location in the test classpath. (If -C is not passed, any
          instrumented JARs are removed.) The classpath is also dynamically
          adjusted to include whatever Cobertura JAR is available; its name
          includes a version number (as of 2.0.x).

          The command-line scripts shipped with Cobertura 2.0.x are out of
          date and do not work out of the box. Pull request #102 was submitted
          to cobertura/cobertura on Github to fix the problem; in lieu of that,
          the scripts must be manually updated to use a correct classpath.

          Tested with Cobertura 2.0.3 / 2.0.4-SNAPSHOT with needed script
          updates.

          Show
          ASF subversion and git services added a comment - Commit 950f144c8e3517053af704189f0fdfd769487dfd in branch refs/heads/1.5.1-SNAPSHOT from Bill Havanki [ https://git-wip-us.apache.org/repos/asf?p=accumulo.git;h=950f144 ] ACCUMULO-1944 Fix coverage for functional tests The -C flag for test/system/auto/run.py did not work. This changeset allows it to work again if a functional Cobertura installation is placed under $ACCUMULO_HOME/lib/test/cobertura. The code for producing instrumented Accumulo code was inactive and out of date. It was reworked so that, if -C is passed, the Accumulo JARs are instrumented and placed into a location ahead of their standard location in the test classpath. (If -C is not passed, any instrumented JARs are removed.) The classpath is also dynamically adjusted to include whatever Cobertura JAR is available; its name includes a version number (as of 2.0.x). The command-line scripts shipped with Cobertura 2.0.x are out of date and do not work out of the box. Pull request #102 was submitted to cobertura/cobertura on Github to fix the problem; in lieu of that, the scripts must be manually updated to use a correct classpath. Tested with Cobertura 2.0.3 / 2.0.4-SNAPSHOT with needed script updates.
          Hide
          ASF subversion and git services added a comment -

          Commit 8669b80f98da34da1b709d1ed4f14647cc1f952c in branch refs/heads/1.5.1-SNAPSHOT from Bill Havanki
          [ https://git-wip-us.apache.org/repos/asf?p=accumulo.git;h=8669b80 ]

          ACCUMULO-1944 Restore test runs when Cobertura not installed

          The changes in the prior commit to re-enable test coverage assume that Cobertura
          is available. When it isn't, functional test runs fail. This commit allows runs
          to proceed when Cobertura is not installed.

          Show
          ASF subversion and git services added a comment - Commit 8669b80f98da34da1b709d1ed4f14647cc1f952c in branch refs/heads/1.5.1-SNAPSHOT from Bill Havanki [ https://git-wip-us.apache.org/repos/asf?p=accumulo.git;h=8669b80 ] ACCUMULO-1944 Restore test runs when Cobertura not installed The changes in the prior commit to re-enable test coverage assume that Cobertura is available. When it isn't, functional test runs fail. This commit allows runs to proceed when Cobertura is not installed.
          Hide
          ASF subversion and git services added a comment -

          Commit 198b1abd9e81d1af55220bf22a0b4e6ec1d18048 in branch refs/heads/1.5.1-SNAPSHOT from Bill Havanki
          [ https://git-wip-us.apache.org/repos/asf?p=accumulo.git;h=198b1ab ]

          ACCUMULO-1944 Fix coverage for functional tests

          The -C flag for test/system/auto/run.py did not work. This changeset
          allows it to work again if a functional Cobertura installation is
          placed under $ACCUMULO_HOME/lib/test/cobertura.

          The code for producing instrumented Accumulo code was inactive and
          out of date. It was reworked so that, if -C is passed, the Accumulo
          JARs are instrumented and placed into a location ahead of their
          standard location in the test classpath. (If -C is not passed, any
          instrumented JARs are removed.) The classpath is also dynamically
          adjusted to include whatever Cobertura JAR is available; its name
          includes a version number (as of 2.0.x).

          The command-line scripts shipped with Cobertura 2.0.x are out of
          date and do not work out of the box. Pull request #102 was submitted
          to cobertura/cobertura on Github to fix the problem; in lieu of that,
          the scripts must be manually updated to use a correct classpath.

          Tested with Cobertura 2.0.3 / 2.0.4-SNAPSHOT with needed script
          updates.

          Show
          ASF subversion and git services added a comment - Commit 198b1abd9e81d1af55220bf22a0b4e6ec1d18048 in branch refs/heads/1.5.1-SNAPSHOT from Bill Havanki [ https://git-wip-us.apache.org/repos/asf?p=accumulo.git;h=198b1ab ] ACCUMULO-1944 Fix coverage for functional tests The -C flag for test/system/auto/run.py did not work. This changeset allows it to work again if a functional Cobertura installation is placed under $ACCUMULO_HOME/lib/test/cobertura. The code for producing instrumented Accumulo code was inactive and out of date. It was reworked so that, if -C is passed, the Accumulo JARs are instrumented and placed into a location ahead of their standard location in the test classpath. (If -C is not passed, any instrumented JARs are removed.) The classpath is also dynamically adjusted to include whatever Cobertura JAR is available; its name includes a version number (as of 2.0.x). The command-line scripts shipped with Cobertura 2.0.x are out of date and do not work out of the box. Pull request #102 was submitted to cobertura/cobertura on Github to fix the problem; in lieu of that, the scripts must be manually updated to use a correct classpath. Tested with Cobertura 2.0.3 / 2.0.4-SNAPSHOT with needed script updates.
          Hide
          ASF subversion and git services added a comment -

          Commit 8669b80f98da34da1b709d1ed4f14647cc1f952c in branch refs/heads/1.4.5-SNAPSHOT from Bill Havanki
          [ https://git-wip-us.apache.org/repos/asf?p=accumulo.git;h=8669b80 ]

          ACCUMULO-1944 Restore test runs when Cobertura not installed

          The changes in the prior commit to re-enable test coverage assume that Cobertura
          is available. When it isn't, functional test runs fail. This commit allows runs
          to proceed when Cobertura is not installed.

          Show
          ASF subversion and git services added a comment - Commit 8669b80f98da34da1b709d1ed4f14647cc1f952c in branch refs/heads/1.4.5-SNAPSHOT from Bill Havanki [ https://git-wip-us.apache.org/repos/asf?p=accumulo.git;h=8669b80 ] ACCUMULO-1944 Restore test runs when Cobertura not installed The changes in the prior commit to re-enable test coverage assume that Cobertura is available. When it isn't, functional test runs fail. This commit allows runs to proceed when Cobertura is not installed.
          Hide
          ASF subversion and git services added a comment -

          Commit 198b1abd9e81d1af55220bf22a0b4e6ec1d18048 in branch refs/heads/1.4.5-SNAPSHOT from Bill Havanki
          [ https://git-wip-us.apache.org/repos/asf?p=accumulo.git;h=198b1ab ]

          ACCUMULO-1944 Fix coverage for functional tests

          The -C flag for test/system/auto/run.py did not work. This changeset
          allows it to work again if a functional Cobertura installation is
          placed under $ACCUMULO_HOME/lib/test/cobertura.

          The code for producing instrumented Accumulo code was inactive and
          out of date. It was reworked so that, if -C is passed, the Accumulo
          JARs are instrumented and placed into a location ahead of their
          standard location in the test classpath. (If -C is not passed, any
          instrumented JARs are removed.) The classpath is also dynamically
          adjusted to include whatever Cobertura JAR is available; its name
          includes a version number (as of 2.0.x).

          The command-line scripts shipped with Cobertura 2.0.x are out of
          date and do not work out of the box. Pull request #102 was submitted
          to cobertura/cobertura on Github to fix the problem; in lieu of that,
          the scripts must be manually updated to use a correct classpath.

          Tested with Cobertura 2.0.3 / 2.0.4-SNAPSHOT with needed script
          updates.

          Show
          ASF subversion and git services added a comment - Commit 198b1abd9e81d1af55220bf22a0b4e6ec1d18048 in branch refs/heads/1.4.5-SNAPSHOT from Bill Havanki [ https://git-wip-us.apache.org/repos/asf?p=accumulo.git;h=198b1ab ] ACCUMULO-1944 Fix coverage for functional tests The -C flag for test/system/auto/run.py did not work. This changeset allows it to work again if a functional Cobertura installation is placed under $ACCUMULO_HOME/lib/test/cobertura. The code for producing instrumented Accumulo code was inactive and out of date. It was reworked so that, if -C is passed, the Accumulo JARs are instrumented and placed into a location ahead of their standard location in the test classpath. (If -C is not passed, any instrumented JARs are removed.) The classpath is also dynamically adjusted to include whatever Cobertura JAR is available; its name includes a version number (as of 2.0.x). The command-line scripts shipped with Cobertura 2.0.x are out of date and do not work out of the box. Pull request #102 was submitted to cobertura/cobertura on Github to fix the problem; in lieu of that, the scripts must be manually updated to use a correct classpath. Tested with Cobertura 2.0.3 / 2.0.4-SNAPSHOT with needed script updates.
          Hide
          Bill Havanki added a comment -

          Updated patch for 1.5.x, as prior patch no longer applies.

          Show
          Bill Havanki added a comment - Updated patch for 1.5.x, as prior patch no longer applies.
          Hide
          Bill Havanki added a comment -

          Updated patch for 1.4.x, as prior patch no longer applies.

          Show
          Bill Havanki added a comment - Updated patch for 1.4.x, as prior patch no longer applies.
          Hide
          Bill Havanki added a comment -

          Patches available for 1.4.x and 1.5.x. Testing for the 1.5.x patch revealed that the original patch broke running functional tests when Cobertura is not installed, so I've submitted a new patch for 1.4.x with the required updates.

          Show
          Bill Havanki added a comment - Patches available for 1.4.x and 1.5.x. Testing for the 1.5.x patch revealed that the original patch broke running functional tests when Cobertura is not installed, so I've submitted a new patch for 1.4.x with the required updates.
          Hide
          Bill Havanki added a comment -

          Current patch only applies to 1.4.5-SNAPSHOT. Working on testing patch for 1.5.1-SNAPSHOT.

          Show
          Bill Havanki added a comment - Current patch only applies to 1.4.5-SNAPSHOT. Working on testing patch for 1.5.1-SNAPSHOT.
          Hide
          Bill Havanki added a comment -

          I had success with it last week, so I don't mind submitting changes. I bet the overall technique will be similar for any tests that use the Cobertura scripts instead of the Maven plugin. Those scripts were also broken (in Cobertura 2.0.x), so I have hopefully repaired those as well (submitted a pull request for it, will link to this ticket).

          Show
          Bill Havanki added a comment - I had success with it last week, so I don't mind submitting changes. I bet the overall technique will be similar for any tests that use the Cobertura scripts instead of the Maven plugin. Those scripts were also broken (in Cobertura 2.0.x), so I have hopefully repaired those as well (submitted a pull request for it, will link to this ticket).
          Hide
          Eric Newton added a comment -

          I have not tried to run coverage for a long time. The last time I did, it was painfully slow, with many false failures due to timing issues. I discourage you from putting any time into fixing this in the python test driver since it no longer exists. Instead, I would put the effort into getting coverage to work for the integration tests.

          Show
          Eric Newton added a comment - I have not tried to run coverage for a long time. The last time I did, it was painfully slow, with many false failures due to timing issues. I discourage you from putting any time into fixing this in the python test driver since it no longer exists. Instead, I would put the effort into getting coverage to work for the integration tests.

            People

            • Assignee:
              Bill Havanki
              Reporter:
              Bill Havanki
            • Votes:
              1 Vote for this issue
              Watchers:
              3 Start watching this issue

              Dates

              • Created:
                Updated:
                Resolved:

                Development