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

sling-mock: make register Sling Models from classpath optional

Agile BoardAttach filesAttach ScreenshotVotersWatch issueWatchersCreate sub-taskLinkCloneUpdate Comment AuthorReplace String in CommentUpdate Comment VisibilityDelete Comments
    XMLWordPrintableJSON

Details

    • Improvement
    • Status: Closed
    • Major
    • Resolution: Fixed
    • Testing Sling Mock 1.9.12, Testing Sling Mock 2.2.18
    • Testing Sling Mock 2.2.20
    • Testing
    • None

    Description

      Registering all sling models from classpath leads to slow down tests dramatically.

      Steps to reproduce:

      1. Register several models.
      2. In context.afterSetup() register several custom injetors that implements StaticInjectAnnotationProcessorFactory

      Each injector registration leads to update every model (ModelAdapterFactory) that slow downs test dramatically, when you have 300+ models and 5-6 injectors that register in setup method before each test.

      Workaround is to register injectors before models.

      Please implement optional registration all sling models from classpath.

       

      Attachments

        Issue Links

        Activity

          This comment will be Viewable by All Users Viewable by All Users
          Cancel

          People

            sseifert Stefan Seifert
            bobi Andrey Bardashevsky
            Votes:
            0 Vote for this issue
            Watchers:
            1 Start watching this issue

            Dates

              Created:
              Updated:
              Resolved:

              Slack

                Issue deployment