Add appveyor test

Add appveyor node test

Clean up tests

Attempt to get the correct python

Another attempt to get paths right

Another attempt to get paths right

try to be more explicit to get correct python

Force explicit tornado and debug prints

Add python itself to the path

Fix path syntax

appveyor.yml

@@ -28,8 +28,12 @@ install:
 
   # Install node:
   - ps: Install-Product node $env:nodejs_version
-  # Ensure python scripts are from right version:
-  - 'SET "PATH=%PYTHON%\\Scripts;%PATH%"'
+  # Install newer Firefox
+  - choco install firefox
+  # Make sure our packages are up to date.
+  - npm update
+  # Ensure python and python scripts are from right version:
+  - 'SET "PATH=%PYTHON%;%PYTHON%\\Scripts;%PATH%"'
   # Install our package:
   - 'pip install --upgrade -e ".[test]"'
 
@@ -38,3 +42,4 @@ build: off
 # to run your custom scripts instead of automatic tests
 test_script:
   - 'py.test'
+  - 'npm test'

test/src/terminal/terminal.spec.ts

@@ -77,8 +77,7 @@ describe('terminal/index', () => {
       if (Platform.IS_WIN) {
         session = null;
         expect(() => { TerminalSession.startNew(); }).to.throwError();
-        done();
-        return;
+        return done();
       }
       TerminalSession.startNew().then(s => {
         session = s;
@@ -109,8 +108,7 @@ describe('terminal/index', () => {
 
       it('should set the title when ready', (done) => {
         if (Platform.IS_WIN) {
-          done();
-          return;
+          return done();
         }
         widget.session = session;
         expect(widget.session).to.be(session);
@@ -188,8 +186,8 @@ describe('terminal/index', () => {
 
       it('should refresh the widget', (done) => {
         if (Platform.IS_WIN) {
-          done();
-          return;
+          expect(widget.refresh()).to.be.a(Promise);
+          return done();
         }
         widget.session = session;
         widget.refresh().then(done, done);