search

honzabrecka / karma-reporter

A plugin for running clojurescript tests with Karma.
MIT License
24 stars 8 forks source link

Fix invalid aget use #17

Closed Deraen closed 6 years ago

Deraen commented 6 years ago

Looks like there is at least single place where Karma-reporter is using aget to access a object.

To test use :checked-arrays :warn.

https://github.com/honzabrecka/karma-reporter/blob/master/src/jx/reporter/karma.cljs#L21

WARNING: cljs.core/aget, arguments must be an array followed by numeric indices, got [js string] instead (consider goog.object/get for object access) at line 20 /home/juho/.cljs/.aot_cache/1.10.238/B8770A6/jx/reporter/karma.cljs

Fix would be to use goog.object/get instead.

honzabrecka commented 6 years ago

Thanks! Fixed.