Uploaded image for project: 'odlparent'
  1. odlparent
  2. ODLPARENT-275

Remove powermock declaration

XMLWordPrintable

    • Icon: Task Task
    • Resolution: Done
    • Icon: Medium Medium
    • 10.0.0
    • None
    • General
    • None

      Powermock is a nigh-dead upstream, with a long history of technical issues w.r.t. integration with other tools – notably Mockito and JaCoCo.

      The Mockito issues seem to be solved at this point, but JaCoCo integration still does not work with on-the-fly instrumentation – as noted in ODLPARENT-27. I believe it is time we removed Powermock from the global picture and let downstreams deal with it however they like.

      There are four projects alive at this point using powermock:

      • lispflowmapping
      • transportpce
      • ovsdb
      • unimgr

      Of these, transportpce manages powermock on its own (and is way behind at 2.0.2) and unimgr is still on odlparent-5 – hence they will not be negatively impacted by us removing the declaration.

      Lispflowmapping has only three tests which use Powermock and hence it is feasible to migrate.

      OVSDB has quite a few tests, but is also almost dead, so just declaring versions just as transportpce does should do the trick.

      Furthermore Mockito now has the ability to mock/spy static methods and final classes, which should work nicely as it is based on ByteBuddy.

            rovarga Robert Varga
            rovarga Robert Varga
            Votes:
            0 Vote for this issue
            Watchers:
            1 Start watching this issue

              Created:
              Updated:
              Resolved: