Merge pull request #443 from stripe/ob-eslint-indent

Enable indent linting rule

Author: ob-stripe

.eslintrc.js

@@ -68,7 +68,7 @@ module.exports = {
         "id-blacklist": "error",
         "id-length": "off",
         "id-match": "error",
-        "indent": "off",
+        "indent": ["error", 2],
         "init-declarations": "off",
         "jsx-quotes": "error",
         "key-spacing": "error",

lib/Error.js

@@ -51,14 +51,14 @@ var StripeError = _Error.StripeError = _Error.extend({
  */
 StripeError.generate = function(rawStripeError) {
   switch (rawStripeError.type) {
-    case 'card_error':
-      return new _Error.StripeCardError(rawStripeError);
-    case 'invalid_request_error':
-      return new _Error.StripeInvalidRequestError(rawStripeError);
-    case 'api_error':
-      return new _Error.StripeAPIError(rawStripeError);
-    case 'idempotency_error':
-      return new _Error.StripeIdempotencyError(rawStripeError);
+  case 'card_error':
+    return new _Error.StripeCardError(rawStripeError);
+  case 'invalid_request_error':
+    return new _Error.StripeInvalidRequestError(rawStripeError);
+  case 'api_error':
+    return new _Error.StripeAPIError(rawStripeError);
+  case 'idempotency_error':
+    return new _Error.StripeIdempotencyError(rawStripeError);
   }
   return new _Error('Generic', 'Unknown Error');
 };

lib/StripeMethod.js

@@ -18,7 +18,7 @@ var OPTIONAL_REGEX = /^optional!/;
  */
 function stripeMethod(spec) {
   var commandPath = typeof spec.path == 'function' ? spec.path
-                  : utils.makeURLInterpolator(spec.path || '');
+    : utils.makeURLInterpolator(spec.path || '');
   var requestMethod = (spec.method || 'GET').toUpperCase();
   var urlParams = spec.urlParams || [];
   var encode = spec.encode || function(data) {return data;};

test/Webhook.spec.js

@@ -98,14 +98,14 @@ describe('Webhooks', function() {
 
     it('should return true when the header contains a valid signature and ' +
       'the timestamp is within the tolerance',
-      function() {
-        var header = generateHeaderString({
-          timestamp: (Date.now() / 1000),
-        });
-
-        expect(stripe.webhooks.signature.verifyHeader(EVENT_PAYLOAD_STRING, header, SECRET, 10)).to.equal(true);
+    function() {
+      var header = generateHeaderString({
+        timestamp: (Date.now() / 1000),
       });
 
+      expect(stripe.webhooks.signature.verifyHeader(EVENT_PAYLOAD_STRING, header, SECRET, 10)).to.equal(true);
+    });
+
     it('should return true when the header contains at least one valid signature', function() {
       var header = generateHeaderString({
         timestamp: (Date.now() / 1000),
@@ -118,13 +118,13 @@ describe('Webhooks', function() {
 
     it('should return true when the header contains a valid signature ' +
       'and the timestamp is off but no tolerance is provided',
-      function() {
-        var header = generateHeaderString({
-          timestamp: 12345,
-        });
-
-        expect(stripe.webhooks.signature.verifyHeader(EVENT_PAYLOAD_STRING, header, SECRET)).to.equal(true);
+    function() {
+      var header = generateHeaderString({
+        timestamp: 12345,
       });
+
+      expect(stripe.webhooks.signature.verifyHeader(EVENT_PAYLOAD_STRING, header, SECRET)).to.equal(true);
+    });
   });
 });
 

test/flows.spec.js

@@ -367,7 +367,7 @@ describe('Flows', function() {
               cleanup.deleteCustomer(cust.id);
               return cust;
             })
-        // Confirm it's expanded by checking that some prop (e.g. exp_year) exists:
+          // Confirm it's expanded by checking that some prop (e.g. exp_year) exists:
         ).to.eventually.have.nested.property('default_source.exp_year');
       });
     });

test/resources/ApplicationFeeRefunds.spec.js

@@ -9,8 +9,8 @@ var REFUND_TEST_ID = 'refundIdTest999';
 
 // Create new CustomerCard instance with pre-filled customerId:
 var appFeeRefund = new resources.ApplicationFeeRefunds(
-    stripe,
-    {feeId: APPFEE_TEST_ID}
+  stripe,
+  {feeId: APPFEE_TEST_ID}
 );
 
 // Use spy from existing resource:

test/resources/CustomerSubscriptions.spec.js

@@ -8,8 +8,8 @@ var CUSTOMER_TEST_ID = 'customerIdTest999';
 
 // Create new CustomerSubscription instance with pre-filled customerId:
 var customerSubscription = new resources.CustomerSubscriptions(
-    stripe,
-    {customerId: CUSTOMER_TEST_ID}
+  stripe,
+  {customerId: CUSTOMER_TEST_ID}
 );
 
 // Use spy from existing resource: