Adding jshint and karma.

codeBelt · codeBelt · commit 012bad7f18be · 2014-09-18T23:15:58.000-05:00
diff --git a/.jshintrc b/.jshintrc
@@ -0,0 +1,67 @@
+{
+  "maxerr": 50,
+  "bitwise": false,
+  "camelcase": true,
+  "curly": true,
+  "eqeqeq": true,
+  "forin": true,
+  "immed": true,
+  "indent": 4,
+  "latedef": true,
+  "newcap": true,
+  "noarg": true,
+  "noempty": true,
+  "nonew": true,
+  "plusplus": false,
+  "quotmark": "single",
+  "undef": true,
+  "unused": "vars",
+  "strict": true,
+  "trailing": true,
+  "maxparams": 7,
+  "maxdepth": 4,
+  "maxstatements": false,
+  "maxcomplexity": false,
+  "maxlen": 120,
+  "asi": false,
+  "boss": false,
+  "debug": false,
+  "eqnull": true,
+  "esnext": false,
+  "moz": false,
+  "evil": false,
+  "expr": false,
+  "funcscope": false,
+  "globalstrict": false,
+  "iterator": false,
+  "lastsemic": false,
+  "laxbreak": false,
+  "laxcomma": false,
+  "loopfunc": false,
+  "multistr": false,
+  "proto": false,
+  "scripturl": false,
+  "smarttabs": false,
+  "shadow": false,
+  "sub": true,
+  "supernew": false,
+  "validthis": false,
+  "browser": true,
+  "couch": false,
+  "devel": true,
+  "dojo": false,
+  "jquery": false,
+  "mootools": false,
+  "node": false,
+  "nonstandard": false,
+  "prototypejs": false,
+  "rhino": false,
+  "worker": true,
+  "wsh": false,
+  "yui": false,
+  "nomen": false,
+  "onevar": false,
+  "passfail": false,
+  "white": false,
+  "globals": {}
+}
diff --git a/Gruntfile.js b/Gruntfile.js
@@ -301,6 +301,40 @@ module.exports = function(grunt) {
             }
         },
 
+        // Verifies that script files conform to our standards.
+        jshint: {
+            options: {
+                jshintrc: '.jshintrc'
+            },
+            all: {
+                src: [
+                    'src/**/*.js'
+                ]
+            }
+        },
+
+        // Verifies that application files conform to the specification files
+        karma: {
+            options: {
+                singleRun: true,
+                configFile: 'karma.conf.js',
+                basePath: 'scripts/',
+                coverageReporter: {
+                    type: 'html',
+                    // path is relative to basePath
+                    // adjust to the best location for your project
+                    // and backend
+                    dir: 'build-reports/'
+                }
+            },
+            unit: {
+                reporters: ['progress']
+//            },
+//            coverage: {
+//                reporters: ['progress', 'coverage']
+            }
+        },
+
         /**
          * Watches files and will run task(s) when files are changed. It will also reload/refresh the browser.
          */
diff --git a/karma.conf.js b/karma.conf.js
@@ -0,0 +1,78 @@
+// Karma configuration
+module.exports = function(config) {
+    'use strict';
+
+    config.set({
+        // list of files to load in the browser
+        // NOTE: every time a vendor script is added to the project, 
+        // include a reference to that script here
+        files: [
+            'vendor/nerdery-function-bind/index.js',
+            'vendor/jquery/jquery.min.js', 
+            'scripts/shim.js',
+            'scripts/views/*.js', 
+            'scripts/*.js'
+        ],
+
+        // list of files to exclude
+        exclude: [
+
+        ],
+
+        // web server port
+        port: 9876,
+
+        // If browser does not capture in given timeout [ms], kill it
+        captureTimeout: 60000,
+
+        // enable / disable colors in the output (reporters and logs)
+        colors: true,
+
+        // enable / disable watching file and executing tests whenever any file changes
+        autoWatch: false,
+
+        // level of logging
+        // possible values:
+        // config.LOG_DISABLE || config.LOG_ERROR || config.LOG_WARN || config.LOG_INFO || config.LOG_DEBUG
+        logLevel: config.LOG_WARN,
+
+        // preprocess matching files before serving them to the browser
+        // available preprocessors: https://npmjs.org/browse/keyword/karma-preprocessor
+        preprocessors: {
+            'scripts/**/!(*.spec).js': 'coverage',
+        },
+
+        // Start these browsers, currently available:
+        // available browser launchers: https://npmjs.org/browse/keyword/karma-launcher
+        browsers: ['PhantomJS'],
+
+        // frameworks to use
+        // available frameworks: https://npmjs.org/browse/keyword/karma-adapter
+        frameworks: ['jasmine'],
+
+        // test results reporter to use
+        // available reporters: https://npmjs.org/browse/keyword/karma-reporter
+        reporters: ['progress', 'coverage'],
+
+        ///////////////////////////////////////////////////////////////////////////
+        // Note: the following settings are only used when running karma
+        // directly as `karma start`.
+        //
+        // When running as `grunt test` these are overridden in the Gruntfile.js.
+        // base path that will be used to resolve all patterns (eg. files, exclude)
+        ///////////////////////////////////////////////////////////////////////////
+        
+        // Continuous Integration mode
+        // if true, it capture browsers, run tests and exit
+        singleRun: false,
+
+        // base path, that will be used to resolve files and exclude
+        basePath: 'src/assets/',
+
+        // path is relative to basePath
+        coverageReporter: {
+            type: 'html',
+            dir: '../../build-reports/'
+        }
+    });
+};
diff --git a/package.json b/package.json
@@ -20,6 +20,8 @@
         "grunt-contrib-uglify": "0.2.7",
         "grunt-contrib-copy": "0.4.1",
         "grunt-contrib-clean": "0.5.0",
+        "grunt-contrib-jshint": "0.7.2",
+        "grunt-karma": "0.8.2",
         "grunt-banner": "0.2.0",
         "grunt-usemin": "2.0.2",
         "grunt-manifest": "0.4.0",
