-
Notifications
You must be signed in to change notification settings - Fork 167
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Redid the integration tests using serverless-offline #288
Conversation
test/integration/test.js
Outdated
@@ -139,7 +158,8 @@ runtimes.forEach(runtime => { | |||
.expect('Content-Type', /json/); | |||
}); | |||
|
|||
it('root', () => { | |||
// FIXME: Broken currently https://github.com/dougmoscrop/serverless-http/issues/270 | |||
it.skip('root', () => { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
See #270
package.json
Outdated
@@ -18,7 +18,7 @@ | |||
"pretest": "tsc --strict --skipLibCheck --noEmit test/typecheck.ts", | |||
"test": "nyc mocha", | |||
"posttest": "eslint lib test", | |||
"test:integration": "mocha test/integration/test.js", | |||
"test:integration": "mocha test/integration/test.js --exit", |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Not sure what's hanging around given the process kills, but doesn't work without this.
Interesting, I'm a fan of using the actual AWS service in case there's drift between serverless-offline, but I also see the value in being able to use it - do you think we could support both? |
Added. I've also plugged the offline integration into the new Github actions work. |
@palfrey do you mind rebasing? |
f6ec22a
to
5ed776b
Compare
This all got merged as part of #286 |
As part of looking at #280, I couldn't figure out how to get the integration tests running without actually deploying things to AWS. This PR redoes them with
serverless-offline
.