[SHINDIG-1771] Closure compile all feature JavaScript as part of JUnit execution to ensure no JavaScript errors at build time

• Type: Improvement
• Status: Resolved
• Priority: Major
• Resolution: Fixed
• Fix Version/s: 2.5.0-beta2
• Component/s: Java, Javascript
• Assignee: Stanton Sievers
• Reporter: Stanton Sievers
• Created: 09/May/12 14:53
• Updated: 30/May/12 12:44
• Resolved: 15/May/12 01:47

Description

At runtime, feature JavaScript is closure compiled at varying compilation levels depending on settings in shindig.properties and other debug flags. If there are compile errors, closure will return an error and this will eventually result in an error back to the client that requested the features.

A better approach would be to compile each feature at build time as part of the JUnit execution and report errors there. This would allow any JavaScript errors to be caught at build time instead of runtime.

To be clear, I'm not recommending a change to any of the runtime compilation behavior. That would stay exactly the same. This is simply extra testing we can do at build time to find errors.