Uploaded image for project: 'Magnolia Test Framework'
  1. Magnolia Test Framework
  2. MGNLTEST-102

Refactor MagnoliaRestClient making it aware of public / author instances

    XMLWordPrintable

Details

    • Improvement
    • Resolution: Fixed
    • Neutral
    • 1.0.2
    • None
    • None

    Description

      MagnoliaTestClient currently uses various - partially conflicting - ways to refer to a Magnolia instance:

      • MagnoliaRestClient#httpHost refers to testSettings.magnoliaHostUrl()
      • Various methods refer to directly to either Instance.DIRECT_AUTHOR or Instance.DIRECT_PUBLIC
      • Various methods take a String instanceContext argument.

      Optimally a MagnoliaRestCient should be bound to an Instance when instantiating instead.

       

      Additionally we should revisit the exception semantics of the methods of the rest clients. Many methods don't throw the declared exceptions at all. Some throw it when they'd rather should return a `false` instead.

      Checklists

        Acceptance criteria

        Attachments

          Activity

            People

              mduerig Michael Duerig
              mduerig Michael Duerig
              Votes:
              0 Vote for this issue
              Watchers:
              1 Start watching this issue

              Dates

                Created:
                Updated:
                Resolved:

                Checklists

                  Task DoD