THRIFT-3627 fix missing basic code style consistency of JavaScript.
Client: js

This closes #858
This closes #1243
diff --git a/lib/js/test/deep-constructor.test.js b/lib/js/test/deep-constructor.test.js
index 9a19809..336fc15 100644
--- a/lib/js/test/deep-constructor.test.js
+++ b/lib/js/test/deep-constructor.test.js
@@ -1,16 +1,16 @@
 function serialize(data) {
-  var transport = new Thrift.Transport("/service");
-  var protocol  = new Thrift.Protocol(transport);
-  protocol.writeMessageBegin("", 0, 0);
+  var transport = new Thrift.Transport('/service');
+  var protocol = new Thrift.Protocol(transport);
+  protocol.writeMessageBegin('', 0, 0);
   data.write(protocol);
   protocol.writeMessageEnd();
   return transport.send_buf;
 }
 
 function deserialize(serialized, type) {
-  var transport = new Thrift.Transport("/service");
+  var transport = new Thrift.Transport('/service');
   transport.setRecvBuffer(serialized);
-  var protocol  = new Thrift.Protocol(transport);
+  var protocol = new Thrift.Protocol(transport);
   protocol.readMessageBegin();
   var data = new type();
   data.read(protocol);
@@ -27,12 +27,12 @@
 
     struct_list_field: [
       new Simple({value: 'b'}),
-      new Simple({value: 'c'}),
+      new Simple({value: 'c'})
     ],
 
     struct_set_field: [
       new Simple({value: 'd'}),
-      new Simple({value: 'e'}),
+      new Simple({value: 'e'})
     ],
 
     struct_map_field: {
@@ -75,12 +75,12 @@
 
     struct_list_field: [
       {value: 'b'},
-      {value: 'c'},
+      {value: 'c'}
     ],
 
     struct_set_field: [
       {value: 'd'},
-      {value: 'e'},
+      {value: 'e'}
     ],
 
     struct_map_field: {
@@ -129,7 +129,7 @@
 
 var cases = {
 
-  "Serialize/deserialize simple struct should return equal object": function(assert){
+  'Serialize/deserialize simple struct should return equal object': function(assert) {
     var tObj = new Simple({value: 'a'});
     var received = deserialize(serialize(tObj), Simple);
     assert.ok(tObj !== received);
@@ -137,21 +137,21 @@
   },
 
 
-  "Serialize/deserialize should return equal object": function(assert){
+  'Serialize/deserialize should return equal object': function(assert) {
     var tObj = createThriftObj();
     var received = deserialize(serialize(tObj), Complex);
     assert.ok(tObj !== received);
     assert.deepEqual(received, tObj);
   },
 
-  "Nested structs and containers initialized from plain js objects should serialize same as if initialized from thrift objects": function(assert) {
+  'Nested structs and containers initialized from plain js objects should serialize same as if initialized from thrift objects': function(assert) {
     var tObj1 = createThriftObj();
     var tObj2 = new Complex(createJsObj());
     assertValues(tObj2, assert);
     assert.equal(serialize(tObj2), serialize(tObj1));
   },
 
-  "Modifications to args object should not affect constructed Thrift object": function (assert) {
+  'Modifications to args object should not affect constructed Thrift object': function(assert) {
 
     var args = createJsObj();
     assertValues(args, assert);
@@ -174,7 +174,7 @@
     assertValues(tObj, assert);
   },
 
-  "nulls are ok": function(assert) {
+  'nulls are ok': function(assert) {
     var tObj = new Complex({
       struct_field: null,
       struct_list_field: null,