Skip to content

Commit

Permalink
Fixing TypeScript warnings, upgrade to Travis Trusty.
Browse files Browse the repository at this point in the history
  • Loading branch information
John Vilk committed Feb 8, 2018
1 parent bef1dc0 commit 7ecbab5
Show file tree
Hide file tree
Showing 4 changed files with 18 additions and 14 deletions.
21 changes: 11 additions & 10 deletions .travis.yml
Original file line number Diff line number Diff line change
@@ -1,23 +1,24 @@
dist: trusty
sudo: false
language: node_js
node_js:
- "4"

sudo: false
- "8"

cache:
yarn: true
directories:
- node_modules

before_install:
- "node --version"
- "npm --version"
- "nvm --version"
- "firefox --version"
- node --version
- yarn --version
- nvm --version
- firefox --version

before_script:
- export DISPLAY=:99.0
- sh -e /etc/init.d/xvfb start

install:
- yarn install

script:
- "npm test"
- yarn test
2 changes: 2 additions & 0 deletions test/tests/fs/OverlayFS/delete-log-test.ts
Original file line number Diff line number Diff line change
Expand Up @@ -8,6 +8,8 @@ const logPath = '/.deletedFiles.log';
declare var __numWaiting: number;

export default function() {
// HACK around TypeScript bug.
if (__numWaiting) {}
var rootFS = (<OverlayFS> fs.getRootFS()).unwrap(),
fses = rootFS.getOverlayedFileSystems(),
// XXX: Make these proper API calls.
Expand Down
6 changes: 3 additions & 3 deletions test/tests/fs/all/node-fs-write-file-sync.ts
Original file line number Diff line number Diff line change
Expand Up @@ -4,7 +4,7 @@ import assert from '../../../harness/wrapped-assert';
import common from '../../../harness/common';

export default function() {
var openCount = 0;
//var openCount = 0;
var mode: number;
var content: string;
var rootFS = fs.getRootFS();
Expand All @@ -22,12 +22,12 @@ export default function() {
}

function openSync() {
openCount++;
//openCount++;
return (<any>rootFS)._openSync.apply(rootFS, arguments);
}

function closeSync() {
openCount--;
//openCount--;
return (<any>fs)._closeSync.apply(fs, arguments);
}

Expand Down
3 changes: 2 additions & 1 deletion test/tests/general/check-options-test.ts
Original file line number Diff line number Diff line change
Expand Up @@ -46,7 +46,8 @@ function errorMessageAssert(expectedMsgs: string[], unexpectedMsgs: string[] = [
}

export default function() {

// HACK around TypeScript bug.
if (__numWaiting) {}
const emptyOptionsFS = getFileSystem({});
checkOptions(emptyOptionsFS, {}, noErrorAssert);
// Tolerates unrecognized options.
Expand Down

0 comments on commit 7ecbab5

Please sign in to comment.