Throw err when beforeEach.withApp is not called

[faridnsh]

The error that is thrown when lt.beforeEach.withApp is called is cryptic and new users of this module, may not understand what's going on.

[slnode]

Can one of the admins verify this patch? To accept patch and trigger a build add comment ".ok\W+to\W+test."

[bajtos]

@slnode ok to test

[bajtos]

In your code, this points to _describe. However, the request variable is set on the beforeEach context provided by Mocha.

You need to move this check into beforeEach few lines below.

[bajtos]

Also make sure that npm test passes. At the moment, it fails because of the problem pointed out in my comment above:

Error: App is not specified. Please use lt.beforeEach.withApp to specify the app.
    at Object._describe.whenCalledRemotely (/mnt/jenkins/workspace/loopback-testing.pr/a88e4f2b/lib/helpers.js:6:13080)
    at /mnt/jenkins/workspace/loopback-testing.pr/a88e4f2b/test/test.js:69:24

[faridnsh]

Damn, I can't believe I never wrote test for this. I guess I missed it in the rush... I'll add a test for this and fix the problem with it.

[faridnsh]

Actually I just realized it's not possible to test this cleanly(maybe that's why I didn't write it initially). I force pushed the fixed version. npm test works fine now.

[bajtos]

Actually I just realized it's not possible to test this cleanly(maybe that's why I didn't write it initially).

Yeah, I have realised the same, that's why I didn't asked you to write a test :)

I force pushed the fixed version. npm test works fine now.

Thanks.

[bajtos]

Landed and released as [email protected], thank you for the contribution!