Code cleanup and test fixes

Cleanup some code from PR #149 and fix the broken unit tests and
hopefully make it so that failing unit tests will fail the travis build
again

Author: lukepfeiffer10

README.md

@@ -67,7 +67,7 @@ Inside of `ui-options`, you can customize these five properties:
 * `eventsToHandle` - default: `['input', 'keyup', 'click', 'focus']`
 * `addDefaultPlaceholder` - default: `true`
 * `escChar` - default: `'\\'`
-* `unsafe` - default: `false`
+* `allowInvalidValue` - default: `false`
 
 When customizing `eventsToHandle`, `clearOnBlur`, or `addDefaultPlaceholder`, the value you supply will replace the default. To customize `eventsToHandle`, be sure to replace the entire array.
 
@@ -77,7 +77,7 @@ When setting `clearOnBlurPlaceholder` to `true`, it will show the placeholder te
 
 If the `escChar` (\\ by default) is encountered in a mask, the next character will be treated as a literal and not a mask definition key.  To disable the `escChar` feature completely, set `escChar` to `null`.
 
-When `unsafe` set to true, apply value to `$modelValue` even if it isn't valid. By default, if you write invalid value, model stay `undefined`.
+When `allowInvalidValue` is set to true, apply value to `$modelValue` even if it isn't valid. By default, if you write an invalid value, the model will stay `undefined`.
 
 #### Global customization
 In addition to customizing behaviors for a specific element, you can also customize the behaviors globally. To do this, simply use the `uiMaskConfig` provider in your app configuration. Example:

gulpfile.js

@@ -121,7 +121,7 @@ gulp.task('karma-sauce', ['build', 'start-sauce-connect'], function(callback) {
       karma: Server,
       customLaunchers: customLaunchers
   }, function(code) {
-      callback();
+      callback(code);
   });
 });
 

src/mask.js

@@ -12,7 +12,8 @@ angular.module('ui.mask', [])
             clearOnBlurPlaceholder: false,
             escChar: '\\',
             eventsToHandle: ['input', 'keyup', 'click', 'focus'],
-            addDefaultPlaceholder: true
+            addDefaultPlaceholder: true,
+            allowInvalidValue: false
         })
         .provider('uiMask.Config', function() {
             var options = {};
@@ -58,7 +59,7 @@ angular.module('ui.mask', [])
                     require: 'ngModel',
                     restrict: 'A',
                     compile: function uiMaskCompilingFunction() {
-                        var options = maskConfig;
+                        var options = angular.copy(maskConfig);
 
                         return function uiMaskLinkingFunction(scope, iElement, iAttrs, controller) {
                             var maskProcessed = false, eventsBound = false,
@@ -161,12 +162,8 @@ angular.module('ui.mask', [])
                                 controller.$viewValue = value.length ? maskValue(value) : '';
                                 controller.$setValidity('mask', isValid);
 
-                                if (modelViewValue) value = controller.$viewValue;
-                                if (!value.length) return undefined;
                                 if (isValid || linkOptions.allowInvalidValue) {
-                                    return value;
-                                } else {
-                                    return undefined;
+                                    return modelViewValue ? controller.$viewValue : value;
                                 }
                             }