resolve all x-spacing and no-spaced-x eslint violations

2 files changed, 88 insertions, 83 deletions

diff --git a/spec/javascripts/graphs/stat_graph_contributors_graph_spec.js b/spec/javascripts/graphs/stat_graph_contributors_graph_spec.js
index f79f771fd3b..d76fcc5206a 100644
--- a/spec/javascripts/graphs/stat_graph_contributors_graph_spec.js
+++ b/spec/javascripts/graphs/stat_graph_contributors_graph_spec.js
@@ -1,4 +1,4 @@
-/* eslint-disable quotes, object-curly-spacing, jasmine/no-suite-dupes, vars-on-top, no-var, spaced-comment, max-len */
+/* eslint-disable quotes, jasmine/no-suite-dupes, vars-on-top, no-var, max-len */
 /* global d3 */
 /* global ContributorsGraph */
 /* global ContributorsMasterGraph */
@@ -15,21 +15,21 @@ describe("ContributorsGraph", function () {
 
   describe("#set_y_domain", function () {
     it("sets the y_domain", function () {
-      ContributorsGraph.set_y_domain([{commits: 30}]);
+      ContributorsGraph.set_y_domain([{ commits: 30 }]);
       expect(ContributorsGraph.prototype.y_domain).toEqual([0, 30]);
     });
   });
 
   describe("#init_x_domain", function () {
     it("sets the initial x_domain", function () {
-      ContributorsGraph.init_x_domain([{date: "2013-01-31"}, {date: "2012-01-31"}]);
+      ContributorsGraph.init_x_domain([{ date: "2013-01-31" }, { date: "2012-01-31" }]);
       expect(ContributorsGraph.prototype.x_domain).toEqual(["2012-01-31", "2013-01-31"]);
     });
   });
 
   describe("#init_y_domain", function () {
     it("sets the initial y_domain", function () {
-      ContributorsGraph.init_y_domain([{commits: 30}]);
+      ContributorsGraph.init_y_domain([{ commits: 30 }]);
       expect(ContributorsGraph.prototype.y_domain).toEqual([0, 30]);
     });
   });
@@ -95,24 +95,24 @@ describe("ContributorsGraph", function () {
 
 describe("ContributorsMasterGraph", function () {
   // TODO: fix or remove
-  //describe("#process_dates", function () {
-    //it("gets and parses dates", function () {
-      //var graph = new ContributorsMasterGraph();
-      //var data = 'random data here';
-      //spyOn(graph, 'parse_dates');
-      //spyOn(graph, 'get_dates').andReturn("get");
-      //spyOn(ContributorsGraph,'set_dates').andCallThrough();
-      //graph.process_dates(data);
-      //expect(graph.parse_dates).toHaveBeenCalledWith(data);
-      //expect(graph.get_dates).toHaveBeenCalledWith(data);
-      //expect(ContributorsGraph.set_dates).toHaveBeenCalledWith("get");
-    //});
-  //});
+  // describe("#process_dates", function () {
+  //   it("gets and parses dates", function () {
+  //     var graph = new ContributorsMasterGraph();
+  //     var data = 'random data here';
+  //     spyOn(graph, 'parse_dates');
+  //     spyOn(graph, 'get_dates').andReturn("get");
+  //     spyOn(ContributorsGraph,'set_dates').andCallThrough();
+  //     graph.process_dates(data);
+  //     expect(graph.parse_dates).toHaveBeenCalledWith(data);
+  //     expect(graph.get_dates).toHaveBeenCalledWith(data);
+  //     expect(ContributorsGraph.set_dates).toHaveBeenCalledWith("get");
+  //   });
+  // });
 
   describe("#get_dates", function () {
     it("plucks the date field from data collection", function () {
       var graph = new ContributorsMasterGraph();
-      var data = [{date: "2013-01-01"}, {date: "2012-12-15"}];
+      var data = [{ date: "2013-01-01" }, { date: "2012-12-15" }];
       expect(graph.get_dates(data)).toEqual(["2013-01-01", "2012-12-15"]);
     });
   });
@@ -121,8 +121,8 @@ describe("ContributorsMasterGraph", function () {
     it("parses the dates", function () {
       var graph = new ContributorsMasterGraph();
       var parseDate = d3.time.format("%Y-%m-%d").parse;
-      var data = [{date: "2013-01-01"}, {date: "2012-12-15"}];
-      var correct = [{date: parseDate(data[0].date)}, {date: parseDate(data[1].date)}];
+      var data = [{ date: "2013-01-01" }, { date: "2012-12-15" }];
+      var correct = [{ date: parseDate(data[0].date) }, { date: parseDate(data[1].date) }];
       graph.parse_dates(data);
       expect(data).toEqual(correct);
     });
diff --git a/spec/javascripts/graphs/stat_graph_contributors_util_spec.js b/spec/javascripts/graphs/stat_graph_contributors_util_spec.js
index 30066788b0d..63f28dfb8ad 100644
--- a/spec/javascripts/graphs/stat_graph_contributors_util_spec.js
+++ b/spec/javascripts/graphs/stat_graph_contributors_util_spec.js
@@ -1,4 +1,4 @@
-/* eslint-disable quotes, no-var, camelcase, object-curly-spacing, object-property-newline, comma-dangle, comma-spacing, spaced-comment, max-len, key-spacing, vars-on-top, quote-props, no-multi-spaces */
+/* eslint-disable quotes, no-var, camelcase, object-property-newline, comma-dangle, max-len, vars-on-top, quote-props */
 /* global ContributorsStatGraphUtil */
 
 //= require graphs/stat_graph_contributors_util
@@ -7,24 +7,25 @@ describe("ContributorsStatGraphUtil", function () {
   describe("#parse_log", function () {
     it("returns a correctly parsed log", function () {
       var fake_log = [
-            {author_email: "karlo@email.com", author_name: "Karlo Soriano", date: "2013-05-09", additions: 471},
-            {author_email: "dzaporozhets@email.com", author_name: "Dmitriy Zaporozhets", date: "2013-05-08", additions: 6, deletions: 1},
-            {author_email: "dzaporozhets@email.com", author_name: "Dmitriy Zaporozhets", date: "2013-05-08", additions: 19, deletions: 3},
-            {author_email: "dzaporozhets@email.com", author_name: "Dmitriy Zaporozhets", date: "2013-05-08", additions: 29, deletions: 3}];
+        { author_email: "karlo@email.com", author_name: "Karlo Soriano", date: "2013-05-09", additions: 471 },
+        { author_email: "dzaporozhets@email.com", author_name: "Dmitriy Zaporozhets", date: "2013-05-08", additions: 6, deletions: 1 },
+        { author_email: "dzaporozhets@email.com", author_name: "Dmitriy Zaporozhets", date: "2013-05-08", additions: 19, deletions: 3 },
+        { author_email: "dzaporozhets@email.com", author_name: "Dmitriy Zaporozhets", date: "2013-05-08", additions: 29, deletions: 3 }
+      ];
 
       var correct_parsed_log = {
         total: [
-        {date: "2013-05-09", additions: 471, deletions: 0, commits: 1},
-        {date: "2013-05-08", additions: 54, deletions: 7, commits: 3}],
-        by_author:
-        [
+          { date: "2013-05-09", additions: 471, deletions: 0, commits: 1 },
+          { date: "2013-05-08", additions: 54, deletions: 7, commits: 3 }
+        ],
+        by_author: [
           {
             author_name: "Karlo Soriano", author_email: "karlo@email.com",
-            "2013-05-09": {date: "2013-05-09", additions: 471, deletions: 0, commits: 1}
+            "2013-05-09": { date: "2013-05-09", additions: 471, deletions: 0, commits: 1 }
           },
           {
-            author_name: "Dmitriy Zaporozhets",author_email: "dzaporozhets@email.com",
-            "2013-05-08": {date: "2013-05-08", additions: 54, deletions: 7, commits: 3}
+            author_name: "Dmitriy Zaporozhets", author_email: "dzaporozhets@email.com",
+            "2013-05-08": { date: "2013-05-08", additions: 54, deletions: 7, commits: 3 }
           }
         ]
       };
@@ -33,7 +34,7 @@ describe("ContributorsStatGraphUtil", function () {
   });
 
   describe("#store_data", function () {
-    var fake_entry = {author: "Karlo Soriano", date: "2013-05-09", additions: 471};
+    var fake_entry = { author: "Karlo Soriano", date: "2013-05-09", additions: 471 };
     var fake_total = {};
     var fake_by_author = {};
 
@@ -57,20 +58,20 @@ describe("ContributorsStatGraphUtil", function () {
   });
 
   // TODO: fix or remove
-  //describe("#store_commits", function () {
-    //var fake_total = "fake_total";
-    //var fake_by_author = "fake_by_author";
-
-    //it("calls #add twice with arguments fake_total and fake_by_author respectively", function () {
-      //spyOn(ContributorsStatGraphUtil, 'add');
-      //ContributorsStatGraphUtil.store_commits(fake_total, fake_by_author);
-      //expect(ContributorsStatGraphUtil.add.argsForCall).toEqual([["fake_total", "commits", 1], ["fake_by_author", "commits", 1]]);
-    //});
-  //});
+  // describe("#store_commits", function () {
+  //   var fake_total = "fake_total";
+  //   var fake_by_author = "fake_by_author";
+  //
+  //   it("calls #add twice with arguments fake_total and fake_by_author respectively", function () {
+  //     spyOn(ContributorsStatGraphUtil, 'add');
+  //     ContributorsStatGraphUtil.store_commits(fake_total, fake_by_author);
+  //     expect(ContributorsStatGraphUtil.add.argsForCall).toEqual([["fake_total", "commits", 1], ["fake_by_author", "commits", 1]]);
+  //   });
+  // });
 
   describe("#add", function () {
     it("adds 1 to current test_field in collection", function () {
-      var fake_collection = {test_field: 10};
+      var fake_collection = { test_field: 10 };
       ContributorsStatGraphUtil.add(fake_collection, "test_field", 1);
       expect(fake_collection.test_field).toEqual(11);
     });
@@ -83,28 +84,28 @@ describe("ContributorsStatGraphUtil", function () {
   });
 
   // TODO: fix or remove
-  //describe("#store_additions", function () {
-    //var fake_entry = {additions: 10};
-    //var fake_total= "fake_total";
-    //var fake_by_author = "fake_by_author";
-    //it("calls #add twice with arguments fake_total and fake_by_author respectively", function () {
-      //spyOn(ContributorsStatGraphUtil, 'add');
-      //ContributorsStatGraphUtil.store_additions(fake_entry, fake_total, fake_by_author);
-      //expect(ContributorsStatGraphUtil.add.argsForCall).toEqual([["fake_total", "additions", 10], ["fake_by_author", "additions", 10]]);
-    //});
-  //});
+  // describe("#store_additions", function () {
+  //   var fake_entry = {additions: 10};
+  //   var fake_total= "fake_total";
+  //   var fake_by_author = "fake_by_author";
+  //   it("calls #add twice with arguments fake_total and fake_by_author respectively", function () {
+  //     spyOn(ContributorsStatGraphUtil, 'add');
+  //     ContributorsStatGraphUtil.store_additions(fake_entry, fake_total, fake_by_author);
+  //     expect(ContributorsStatGraphUtil.add.argsForCall).toEqual([["fake_total", "additions", 10], ["fake_by_author", "additions", 10]]);
+  //   });
+  // });
 
   // TODO: fix or remove
-  //describe("#store_deletions", function () {
-    //var fake_entry = {deletions: 10};
-    //var fake_total= "fake_total";
-    //var fake_by_author = "fake_by_author";
-    //it("calls #add twice with arguments fake_total and fake_by_author respectively", function () {
-      //spyOn(ContributorsStatGraphUtil, 'add');
-      //ContributorsStatGraphUtil.store_deletions(fake_entry, fake_total, fake_by_author);
-      //expect(ContributorsStatGraphUtil.add.argsForCall).toEqual([["fake_total", "deletions", 10], ["fake_by_author", "deletions", 10]]);
-    //});
-  //});
+  // describe("#store_deletions", function () {
+  //   var fake_entry = {deletions: 10};
+  //   var fake_total= "fake_total";
+  //   var fake_by_author = "fake_by_author";
+  //   it("calls #add twice with arguments fake_total and fake_by_author respectively", function () {
+  //     spyOn(ContributorsStatGraphUtil, 'add');
+  //     ContributorsStatGraphUtil.store_deletions(fake_entry, fake_total, fake_by_author);
+  //     expect(ContributorsStatGraphUtil.add.argsForCall).toEqual([["fake_total", "deletions", 10], ["fake_by_author", "deletions", 10]]);
+  //   });
+  // });
 
   describe("#add_date", function () {
     it("adds a date field to the collection", function () {
@@ -130,32 +131,36 @@ describe("ContributorsStatGraphUtil", function () {
     it("returns the collection sorted via specified field", function () {
       var fake_parsed_log = {
         total: [
-          {date: "2013-05-09", additions: 471, deletions: 0, commits: 1},
-          {date: "2013-05-08", additions: 54, deletions: 7, commits: 3}
+          { date: "2013-05-09", additions: 471, deletions: 0, commits: 1 },
+          { date: "2013-05-08", additions: 54, deletions: 7, commits: 3 }
         ],
-        by_author:[
+        by_author: [
           {
             author: "Karlo Soriano",
-            "2013-05-09": {date: "2013-05-09", additions: 471, deletions: 0, commits: 1}
+            "2013-05-09": { date: "2013-05-09", additions: 471, deletions: 0, commits: 1 }
           },
           {
             author: "Dmitriy Zaporozhets",
-            "2013-05-08": {date: "2013-05-08", additions: 54, deletions: 7, commits: 3}
+            "2013-05-08": { date: "2013-05-08", additions: 54, deletions: 7, commits: 3 }
           }
         ]
       };
-      var correct_total_data = [{date: "2013-05-08", commits: 3},
-      {date: "2013-05-09", commits: 1}];
+      var correct_total_data = [
+        { date: "2013-05-08", commits: 3 },
+        { date: "2013-05-09", commits: 1 }
+      ];
       expect(ContributorsStatGraphUtil.get_total_data(fake_parsed_log, "commits")).toEqual(correct_total_data);
     });
   });
 
   describe("#pick_field", function () {
     it("returns the collection with only the specified field and date", function () {
-      var fake_parsed_log_total = [{date: "2013-05-09", additions: 471, deletions: 0, commits: 1},
-      {date: "2013-05-08", additions: 54, deletions: 7, commits: 3}];
+      var fake_parsed_log_total = [
+        { date: "2013-05-09", additions: 471, deletions: 0, commits: 1 },
+        { date: "2013-05-08", additions: 54, deletions: 7, commits: 3 }
+      ];
       ContributorsStatGraphUtil.pick_field(fake_parsed_log_total, "commits");
-      var correct_pick_field_data = [{date: "2013-05-09", commits: 1},{date: "2013-05-08", commits: 3}];
+      var correct_pick_field_data = [{ date: "2013-05-09", commits: 1 }, { date: "2013-05-08", commits: 3 }];
       expect(ContributorsStatGraphUtil.pick_field(fake_parsed_log_total, "commits")).toEqual(correct_pick_field_data);
     });
   });
@@ -164,23 +169,23 @@ describe("ContributorsStatGraphUtil", function () {
     it("returns the log by author sorted by specified field", function () {
       var fake_parsed_log = {
         total: [
-          {date: "2013-05-09", additions: 471, deletions: 0, commits: 1},
-          {date: "2013-05-08", additions: 54, deletions: 7, commits: 3}
+          { date: "2013-05-09", additions: 471, deletions: 0, commits: 1 },
+          { date: "2013-05-08", additions: 54, deletions: 7, commits: 3 }
         ],
         by_author: [
           {
             author_name: "Karlo Soriano", author_email: "karlo@email.com",
-            "2013-05-09": {date: "2013-05-09", additions: 471, deletions: 0, commits: 1}
+            "2013-05-09": { date: "2013-05-09", additions: 471, deletions: 0, commits: 1 }
           },
           {
             author_name: "Dmitriy Zaporozhets", author_email: "dzaporozhets@email.com",
-            "2013-05-08": {date: "2013-05-08", additions: 54, deletions: 7, commits: 3}
+            "2013-05-08": { date: "2013-05-08", additions: 54, deletions: 7, commits: 3 }
           }
         ]
       };
       var correct_author_data = [
-        {author_name:"Dmitriy Zaporozhets",author_email:"dzaporozhets@email.com",dates:{"2013-05-08":3},deletions:7,additions:54,"commits":3},
-        {author_name:"Karlo Soriano",author_email:"karlo@email.com",dates:{"2013-05-09":1},deletions:0,additions:471,commits:1}
+        { author_name: "Dmitriy Zaporozhets", author_email: "dzaporozhets@email.com", dates: { "2013-05-08": 3 }, deletions: 7, additions: 54, "commits": 3 },
+        { author_name: "Karlo Soriano", author_email: "karlo@email.com", dates: { "2013-05-09": 1 }, deletions: 0, additions: 471, commits: 1 }
       ];
       expect(ContributorsStatGraphUtil.get_author_data(fake_parsed_log, "commits")).toEqual(correct_author_data);
     });
@@ -188,10 +193,10 @@ describe("ContributorsStatGraphUtil", function () {
 
   describe("#parse_log_entry", function () {
     it("adds the corresponding info from the log entry to the author", function () {
-      var fake_log_entry =    { author_name: "Karlo Soriano", author_email: "karlo@email.com",
-        "2013-05-09": {date: "2013-05-09", additions: 471, deletions: 0, commits: 1}
+      var fake_log_entry = { author_name: "Karlo Soriano", author_email: "karlo@email.com",
+        "2013-05-09": { date: "2013-05-09", additions: 471, deletions: 0, commits: 1 }
       };
-      var correct_parsed_log = {author_name:"Karlo Soriano",author_email:"karlo@email.com",dates:{"2013-05-09":1},deletions:0,additions:471,commits:1};
+      var correct_parsed_log = { author_name: "Karlo Soriano", author_email: "karlo@email.com", dates: { "2013-05-09": 1 }, deletions: 0, additions: 471, commits: 1 };
       expect(ContributorsStatGraphUtil.parse_log_entry(fake_log_entry, 'commits', null)).toEqual(correct_parsed_log);
     });
   });