Fix jscs code style violations and run jscs in the test operation

FagnerMartinsBrack · FagnerMartinsBrack · commit 7c3a8bd7a05d · 2015-06-07T21:52:48.000-03:00
I added jscs to run outside the regular build flow and make
contributions easier, but I always forget to run it and several code
style violations pass through.
diff --git a/Gruntfile.js b/Gruntfile.js
@@ -138,7 +138,7 @@ module.exports = function (grunt) {
 					pollInterval: 10000,
 					statusCheckAttempts: 90,
 					throttled: 3,
-					browsers: (function() {
+					browsers: (function () {
 						var browsers = {
 							'iOS': [{
 								browserName: 'iphone',
@@ -216,7 +216,7 @@ module.exports = function (grunt) {
 						};
 
 						var matrix = [];
-						for ( var os in browsers ) {
+						for (var os in browsers) {
 							matrix = matrix.concat(browsers[os]);
 						}
 						return matrix;
@@ -234,7 +234,7 @@ module.exports = function (grunt) {
 	}
 
 	grunt.registerTask('saucelabs', ['connect:build-sauce', 'saucelabs-qunit']);
-	grunt.registerTask('test', ['jshint', 'connect:build-qunit', 'qunit', 'nodeunit']);
+	grunt.registerTask('test', ['jshint', 'jscs', 'connect:build-qunit', 'qunit', 'nodeunit']);
 
 	grunt.registerTask('dev', ['test', 'uglify', 'compare_size']);
 	grunt.registerTask('ci', ['test', 'saucelabs']);
diff --git a/test/tests.js b/test/tests.js
@@ -165,10 +165,10 @@ QUnit.test('API for changing defaults', function (assert) {
 
 	Cookies.defaults.path = '/foo';
 	assert.ok(Cookies.set('c', 'v').match(/path=\/foo/), 'should use attributes from defaults');
-	Cookies.remove( 'c', { path: '/foo' });
+	Cookies.remove('c', { path: '/foo' });
 
 	assert.ok(Cookies.set('c', 'v', { path: '/bar' }).match(/path=\/bar/), 'attributes argument has precedence');
-	Cookies.remove( 'c', { path: '/bar' });
+	Cookies.remove('c', { path: '/bar' });
 
 	delete Cookies.defaults.path;
 	assert.ok(Cookies.set('c', 'v').match(/path=\//), 'should roll back to the default path');
diff --git a/test/utils.js b/test/utils.js
@@ -1,6 +1,6 @@
 // https://github.com/axemclion/grunt-saucelabs#test-result-details-with-qunit
 (function () {
-	"use strict";
+	'use strict';
 
 	var log = [];
 
@@ -54,13 +54,13 @@
 		element[ method ](eventName, fn);
 	};
 
-	window.using = function( assert ) {
+	window.using = function (assert) {
 		function getQuery(key) {
 			var queries = location.href.split('?')[1];
 			if (!queries) {
 				return;
 			}
-			var pairs = queries.split( /&|=/ );
+			var pairs = queries.split(/&|=/);
 			var indexBaseURL = pairs.indexOf(key);
 			var result = pairs[indexBaseURL + 1];
 			if (result) {
@@ -69,7 +69,7 @@
 		}
 		function setCookie(name, value) {
 			return {
-				then: function(callback) {
+				then: function (callback) {
 					var iframe = document.getElementById('request_target');
 					var serverURL = getQuery('integration_baseurl');
 					Cookies.set(name, value);
@@ -87,7 +87,7 @@
 							var iframeDocument = iframe.contentWindow.document;
 							var root = iframeDocument.documentElement;
 							var content = root.textContent;
-							if ( !content ) {
+							if (!content) {
 								ok(false, [
 									'"' + requestURL + '"',
 									'content should not be empty'
