From e8bd0cf56b51bff2f204e88fa1ad2419c663b097 Mon Sep 17 00:00:00 2001 From: Dominic Kramer Date: Wed, 2 Aug 2017 07:36:33 -0700 Subject: [PATCH] Enable checking test linting/formatting errors (#322) --- gulpfile.js | 6 ++---- src/debuggee.ts | 2 +- 2 files changed, 3 insertions(+), 5 deletions(-) diff --git a/gulpfile.js b/gulpfile.js index 09ba286f..2168bbcd 100644 --- a/gulpfile.js +++ b/gulpfile.js @@ -155,8 +155,6 @@ gulp.task('watch', () => { return gulp.watch(sources, ['test.compile']); }); -// TODO: Enable linting and checking format after the conversion to -// Typescript is complete. -gulp.task('test', ['test.run']);//, 'test.check-format', 'test.check-lint']); -gulp.task('test.coverage', ['test.coverage.run']);//, 'test.check-format', 'test.check-lint']); +gulp.task('test', ['test.run', 'test.check-format', 'test.check-lint']); +gulp.task('test.coverage', ['test.coverage.run', 'test.check-format', 'test.check-lint']); gulp.task('default', ['compile', 'test.unit.compile', 'test.system.compile']); diff --git a/src/debuggee.ts b/src/debuggee.ts index 58004d05..f8dcae8f 100644 --- a/src/debuggee.ts +++ b/src/debuggee.ts @@ -84,7 +84,7 @@ export class Debuggee { // TODO: Determine if `statusMessage` should be optional or be required // and be explicitly set to `null`. - properties = properties || { statusMessage: null }; + properties = properties || {statusMessage: null}; if (!_.isString(properties.project)) { throw new Error('properties.project must be a string');