Ticket #35557: 35557.3.diff

Gruntfile.js

@@ -1 +1 @@
 /* jshint node:true */
 module.exports = function(grunt) {
         var path = require('path'),
+                gitorsvn = require('git-or-svn'),
                 SOURCE_DIR = 'src/',
                 BUILD_DIR = 'build/',
                 autoprefixer = require('autoprefixer'),
@@ -654 +655 @@
                 grunt.task.run( '_' + this.nameArgs );
         } );
 
-        grunt.registerTask( 'precommit', 'Runs front-end dev/test tasks in preparation for a commit.', [
-                'postcss:core',
-                'imagemin:core',
-                'browserify',
-                'jshint:corejs',
-                'uglify:bookmarklet',
-                'qunit:compiled'
-        ] );
+        grunt.registerTask( 'precommit', 'Runs test and build tasks in preparation for a commit', function() {
+                var done = this.async();
 
+                // Figure out what tasks to run based on what files have been modified.
+                function enqueueTestingTasksForModifiedFiles( filesModified ) {
+                        var taskList = ['imagemin:core'];
+                        if ( /src.*\.js/.test( filesModified ) ) {
+                                grunt.log.write( 'JavaScript source files modified, will run JavaScript tests.');
+                                taskList = taskList.concat( ['browserify', 'jshint:corejs', 'uglify:bookmarklet', 'qunit:compiled'] );
+                        }
+                        if ( /src.*\.css/.test( filesModified ) ) {
+                                grunt.log.write( 'CSS source files modified, will run CSS tests.');
+                                taskList.push( 'postcss:core' );
+                        }
+                        if ( /src.*\.php/.test( filesModified ) ) {
+                                grunt.log.write( 'PHP source files modified, will run PHP tests.');
+                                taskList.push( 'phpunit' );
+                        }
+                        grunt.task.run( taskList );
+                        done();
+                }
+                gitorsvn( __dirname, function(gitorsvn) {
+                        if ( gitorsvn === 'svn' ) {
+                                grunt.util.spawn(
+                                        {
+                                                cmd: 'svn',
+                                                args: ['diff', '--summarize']
+                                        },
+                                        function(error, result, code) {
+                                                if ( code !== 0 ) {
+                                                        grunt.fail.warn( 'The `svn diff --summarize` command returned a non-zero exit code.', code );
+                                                }
+                                                enqueueTestingTasksForModifiedFiles( result.stdout );
+                                        }
+                                );
+                        } else if ( gitorsvn === 'git' ) {
+                                grunt.util.spawn(
+                                        {
+                                                cmd: 'git',
+                                                args: ['diff', '--name-only']
+                                        },
+                                        function(error, result, code) {
+                                                if ( code !== 0 ) {
+                                                        grunt.fail.warn( 'The `git diff --name-only` command returned a non-zero exit code.', code );
+                                                }
+                                                enqueueTestingTasksForModifiedFiles( result.stdout );
+                                        }
+                                );
+                        } else {
+                                grunt.log.write( 'This WordPress install is not under version control, no tests will be run.' );
+                        }
+                });
+        });
+
         grunt.registerTask( 'copy:all', [
                 'copy:files',
                 'copy:wp-admin-css-compat-rtl',

package.json

@@ -10 +10 @@
   "license": "GPL-2.0+",
   "devDependencies": {
     "autoprefixer": "~6.1.0",
+    "git-or-svn": "~0.1.0",
     "grunt": "~0.4.5",
     "grunt-browserify": "~4.0.1",
     "grunt-contrib-clean": "~0.6.0",