search

A / superagent-mocker

Pretty simple in-browser mocks for CRUD and REST API
132 stars 32 forks source link

Fix mergeObjects #15

Closed evanboho closed 9 years ago

evanboho commented 9 years ago

For some reason mergeObjects breaks in our environment but works in superagent-mocker tests.

In our case, the arguments object does not iterate with for(var in arguments). Additionally, Object.keys(arguments) returns an empty array. This is in line with what I would expect from the arguments object.

I haven't been able to figure out why this does not break in superagent-mocker tests, but this fix works in both situations.

A commented 9 years ago

thank you! :D I'm going to release new version for few days.

A commented 9 years ago

@evanboho published with 0.2.0

evanboho commented 9 years ago

:+1: And thank you!