* Changed updateExpression to report itself as violation instead of its argument
* Update getBindingIdentifiers to work with forXStatement and return proper node as violation
* Updated unaryExpression violation to be consistent with changes.
* Figuring out where to place throw statement
* Restored path argument destructuring
* New approach using comma expressions for assignments
* Moved throwNode into body of forXstatements
* Refactored with helper function and ensureBlock, additional tests for update exprs
* Added exec tests for all violations
* Hoisted helper function for comma expression outside of visitor
