Details
-
Improvement
-
Resolution: Fixed
-
Neutral
-
None
-
None
-
-
Yes
-
Empty show more show less
-
TE 13
-
3
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