Make JSHInt happy

Author: crissdev

spec/issues.js

@@ -1,3 +1,4 @@
+(function() {
 'use strict';
 /*global ko, QUnit*/
 
@@ -241,12 +242,13 @@ QUnit.test('Issue #33', function(assert) {
                 return ko.utils.unwrapObservable(data.id);
             },
             create: function(options) {
-                var o = (new (function() {
+                function SimpleObject() {
                     this._remove = function() {
                         options.parent.items.mappedRemove(options.data);
                     };
                     ko.mapping.fromJS(options.data, {}, this);
-                })());
+                }
+                var o = new SimpleObject();
                 return o;
             }
         }
@@ -330,3 +332,4 @@ QUnit.test('Issue #203', function(assert) {
 
     assert.deepEqual(ko.mapping.toJS(viewModel), {connectionId: 1, type: 'thirdType', thirdTypeProperty: 'thirdTypeProperty'});
 });
+})();

spec/mappingBehaviors.js

@@ -1,3 +1,4 @@
+(function() {
 'use strict';
 /*global ko, QUnit*/
 
@@ -1009,14 +1010,14 @@ QUnit.test('ko.mapping.fromJS should be able to map empty object structures', fu
 QUnit.test('ko.mapping.fromJS should send create callbacks when atomic items are constructed', function (assert) {
 	var atomicValues = ["hello", 123, true, null, undefined];
 	var callbacksReceived = 0;
-	for (var i = 0; i < atomicValues.length; i++) {
-		var result = ko.mapping.fromJS(atomicValues[i], {
-			create: function (item) {
-				callbacksReceived++;
-				return item;
-			}
-		});
-	}
+    atomicValues.forEach(function(value) {
+        var result = ko.mapping.fromJS(value, {
+            create: function (item) {
+                callbacksReceived++;
+                return item;
+            }
+        });
+    });
     assert.equal(callbacksReceived, 5);
 });
 
@@ -1836,3 +1837,4 @@ QUnit.test('ko.mapping.toJS explicit declared none observable members should be
 
     assert.equal(js.b, data.b);
 });
+})();