Uploaded image for project: 'Sling'
  1. Sling
  2. SLING-7166

Deprecate Commons Testing

    XMLWordPrintableJSON

Details

    • Improvement
    • Status: Open
    • Major
    • Resolution: Unresolved
    • Commons Testing 2.1.2
    • Commons Testing 2.1.4
    • Testing
    • None

    Description

      The module Commons Testing seems to be rather old and not too actively maintained. For most (if not all) of its classes there are nowadays better replacements:

      For the individual packages I consider the following alternatives the better choices

      1. o.a.s.commons.testing.integration: Teleporter or the org.apache.sling.testing.clients should be used preferably instantiated through Sling HTTP testing rules.
      2. o.a.s.commons.testing.jcr: jcr-mock should be used instead
      3. o.a.s.commons.testing.junit: the retry logic from the Sling HTTP Testing Client should be used.
      4. # o.a.s.commons.testing.junit.categories: o.a.s.testing.junit.rules.category.SlowRunningTest from Sling HTTP testing rules. should be used instead.
      5. o.a.s.commons.testing.osgi: osgi-mock should be used instead
      6. o.a.s.commons.testing.sling: sling-mock should be used instead
      7. org.apache.sling.commons.testing.util: if really useful can maybe moved to sling-mock as well, but probably not that useful. Currently TestStringUtil is not used at all and JavascriptEngine is only used from o.a.s.commons.testing.integration.

      In addition there are certain limitations in that module which are not easy to fix:

      1. o.a.s.commons.testing.jcr uses Jackrabbit 2 only and never Oak, that means that the ITs based on those classes are pretty far away from what we ship now in Sling.
      2. o.a.s.commons.testing.jcr is currently not compatible with Java 9 (https://issues.apache.org/jira/browse/SLING-7159)

      The according discussion around this can be found in https://www.mail-archive.com/dev@sling.apache.org/msg70983.html.

      Attachments

        Issue Links

          There are no Sub-Tasks for this issue.

          Activity

            People

              kwin Konrad Windszus
              kwin Konrad Windszus
              Votes:
              0 Vote for this issue
              Watchers:
              1 Start watching this issue

              Dates

                Created:
                Updated: