build: enable windows in travis (#42)

* build: enable windows in travis

* build: replace npm install --prefix with plain cd in travis.yml

* fix: handle termination of child_process for windows differently

* test: move spawn outside of promise

Author: danielcondemarin

.travis.yml

@@ -1,6 +1,6 @@
 sudo: false
 os:
-  # - windows
+  - windows
   - linux
   - osx
 language: node_js
@@ -9,7 +9,7 @@ node_js:
   - "11.10"
 install:
   - npm install
-  - npm install --prefix integration/app-with-serverless-offline
+  - cd integration/app-with-serverless-offline && npm install && cd ../../
 script:
   - npm run lint
   - npm test

integration/__tests__/local-deploy.test.js

@@ -1,3 +1,4 @@
+const execSync = require("child_process").execSync;
 const path = require("path");
 const serverlessOfflineStart = require("../../utils/test/serverlessOfflineStart");
 const httpGet = require("../../utils/test/httpGet");
@@ -13,7 +14,11 @@ describe("Local Deployment Tests (via serverless-offline)", () => {
   });
 
   afterAll(() => {
-    slsOffline.kill();
+    if (process.platform === "win32") {
+      execSync(`taskkill /pid ${slsOffline.pid} /f /t`);
+    } else {
+      slsOffline.kill();
+    }
   });
 
   it("should render the index page", () => {

utils/test/serverlessOfflineStart.js

@@ -4,9 +4,9 @@ const serverlessExec = require("./getServerlessExec");
 const bufferToStr = buffer => Buffer.from(buffer).toString("utf8");
 
 module.exports = () => {
-  return new Promise((resolve, reject) => {
-    const serverlessOffline = spawn(serverlessExec, ["offline"]);
+  const serverlessOffline = spawn(serverlessExec, ["offline"]);
 
+  return new Promise((resolve, reject) => {
     serverlessOffline.stdout.on("data", data => {
       const stdoutStr = bufferToStr(data);