hulu / roca

A command-line tool for running brightscript tests
Apache License 2.0
25 stars 19 forks source link

fix(reporter): don't use case sensitive keys #102

Closed lkipke closed 3 years ago

lkipke commented 3 years ago

Summary

brs recently introduced better handling of associative array keys, and case-sensitive keys are now the default when printing assoc arrays. However, this broke our stackFrames implementation because we used camel case. This results in a runtime error that prints out a Javascript stack trace to the user whenever they have a failed test case.

An easy, backwards-compatible fix is to use snake case instead!