From 145c0cea39c542beee4a3d98be6c1dd90c835afc Mon Sep 17 00:00:00 2001 From: Tim van der Meij Date: Thu, 27 Apr 2017 23:16:20 +0200 Subject: [PATCH] Remove unused Gulp target for testing To run the regression tests, developers use `gulp browsertest` and the bot uses `gulp bottest`. We're not passing the `noreftest` option anywhere in the code (probably because the `bottest` command takes care of this already), so we should remove this. --- gulpfile.js | 4 ---- make.js | 6 +----- 2 files changed, 1 insertion(+), 9 deletions(-) diff --git a/gulpfile.js b/gulpfile.js index a12d6aa7d..b47592c62 100644 --- a/gulpfile.js +++ b/gulpfile.js @@ -1044,10 +1044,6 @@ gulp.task('browsertest', function () { return createTestSource('browser'); }); -gulp.task('browsertest-noreftest', function () { - return createTestSource('browser (no reftest)'); -}); - gulp.task('unittest', function () { return createTestSource('unit'); }); diff --git a/make.js b/make.js index 1263476d2..498860459 100644 --- a/make.js +++ b/make.js @@ -186,11 +186,7 @@ target.bottest = function() { // make browsertest // target.browsertest = function(options) { - if (options && options.noreftest) { - execGulp('browsertest-noreftest'); - } else { - execGulp('browsertest'); - } + execGulp('browsertest'); }; //