Sheriff-o-Matic: Convert some more ui tests to mocha

Tools/GardeningServer/karma.conf.js

 // Karma configuration
 // Generated on Mon Jul 21 2014 15:27:46 GMT-0700 (PDT)
 
 module.exports = function(config) {
   config.set({
 
     // base path that will be used to resolve all patterns (eg. files, exclude)
     basePath: '.',
 
     // frameworks to use
     // available frameworks: https://npmjs.org/browse/keyword/karma-adapter
     frameworks: ['mocha'],
 
     // mocha setup
     client: {
       mocha: {
         ui: 'bdd',
+        checkLeaks: true,

[ojan, 2014/08/13 00:30:28]

just curious, what does this do?

[michaelpg, 2014/08/13 00:41:09]

Checks for global variables being set (window.net is whitelisted below).

This is already being done in the browser unit tests but I noticed (when I
accidentally introduced a global in a test) that karma wasn't catching it.

         globals: ['net']
       }
     },
 
     // list of files / patterns to load in the browser
     files: [
       'bower_components/platform/platform.js',
       'test/karma-loader.html',
       'bower_components/chai/chai.js',
       'bower_components/sugar/release/sugar-full.development.js',
       {pattern: 'bower_components/**/*.{js,html,css,map}', included: false},
       'node_modules/mocha/mocha.js',
       'polymer-load-warning.html',
+      {pattern: 'base/*.html', included: false},
+      {pattern: 'lib/*.html', included: false},
       {pattern: 'model/*.html', included: false},
-      {pattern: 'lib/*.html', included: false},
       'lib/test/*.html',
       'model/test/*.html',
       'scripts/*.js',
       {pattern: 'ui/*.html', included: false},
       'ui/test/*',
     ],
 
     // list of files to exclude
     exclude: [
       'model/*tests.html',
@@ skipping 31 matching lines @@
     // Continuous Integration mode
     // if true, Karma captures browsers, runs the tests and exits
     singleRun: false,
 
     plugins: [
       'karma-mocha',
       'karma-chrome-launcher'
     ]
   });
 };