[SLING-3314] Remove test API from JavaScript wrappers

• Type: Task
• Status: Closed
• Priority: Major
• Resolution: Fixed
• Affects Version/s: Scripting JavaScript 2.0.12
• Fix Version/s: Scripting JavaScript 2.0.14
• Component/s: Scripting
• Assignee: Felix Meschberger
• Reporter: Felix Meschberger
• Created: 13/Jan/14 02:54
• Updated: 24/Jun/14 13:05
• Resolved: 14/Jan/14 02:41

Description

SLING-383 introduced an abstract method jsGet_javascriptWrapperClass() in the ScriptableBase class. This method is used by unit tests to verify the JavaScript host object is provided by the correct implementation class.

This test is problematic on different accounts:

• It creates public API exposing implementation details (the javascriptWrapperClass property)
• It creates an implementation detail dependency in the tests. The tests must validate the functionality not whether the provider is the right instance. For example if the classes would be moved to another package, the tests would fail.