Fix contexts in api-mock

localmed / api-mock

Creates a mock server based on an API Blueprint

MIT License

496 stars 58 forks source link

Closed enykeev closed 10 years ago

enykeev commented 10 years ago

There is several issues within api-mock related to the context functions executed in:

first in ApiMock constructor where we trying to access configuration variable we haven't defined (instead we should use @configuration)
second one is in protagonist.parse callback where our ApiMock context is replaced by Protagonist one so we need to use "fat" arrow instead of "thin" to keep the original context

enykeev commented 10 years ago

I would also ask you to update npm package since it's outdated and lacks CORS fix.

ecordell commented 10 years ago

I merged this in, but I'm going to wait to release a new version until I've at least manually verified that everything is working.