search

hapijs / good

hapi process monitoring
Other
525 stars 160 forks source link

Test fixes. Closes #519 #537

Closed imrvelj closed 7 years ago

imrvelj commented 7 years ago

I updated tests as referenced in #519. The only thing to consider might be test/monitor.js@147 just because an assertion @139 requires it.

arb commented 7 years ago

Are these the only places we passed a sting as module? I feel like it's in there a lot of other places too?

imrvelj commented 7 years ago

I think that's all, but I'll double check in a moment.

imrvelj commented 7 years ago

Yup, that's all. Actually, there are some places where 'module' is defined as a string but all of them are defined as that because their corresponding tests require them to be (for example)

lock[bot] commented 4 years ago

This thread has been automatically locked due to inactivity. Please open a new issue for related bugs or questions following the new issue template instructions.