validateIndentation: ignore empty module bodies

mikesherov · mikesherov · commit 3e517a8a13f8 · 2015-02-10T08:52:07.000-05:00
Fixes #1034
diff --git a/lib/rules/validate-indentation.js b/lib/rules/validate-indentation.js
@@ -365,7 +365,7 @@ module.exports.prototype = {
 
             // set number of indents for modules by detecting
             // whether the first statement is indented or not
-            if (moduleBody) {
+            if (moduleBody && moduleBody.body.length) {
                 _this._moduleIndents = moduleBody.body[0].loc.start.column > 0 ? 1 : 0;
             }
         }
diff --git a/test/rules/validate-indentation.js b/test/rules/validate-indentation.js
@@ -157,7 +157,8 @@ describe('rules/validate-indentation', function() {
 
         var cases = {
             'no indentation': 'a++;',
-            indentation: '    a++;'
+            indentation: '    a++;',
+            'empty body': ''
         };
 
         Object.keys(cases).forEach(function(title) {

Original file line number	Diff line number	Diff line change
`@@ -365,7 +365,7 @@ module.exports.prototype = {`
`365`	`365`
`366`	`366`	`// set number of indents for modules by detecting`
`367`	`367`	`// whether the first statement is indented or not`
`368`		`- if (moduleBody) {`
	`368`	`+ if (moduleBody && moduleBody.body.length) {`
`369`	`369`	`_this._moduleIndents = moduleBody.body[0].loc.start.column > 0 ? 1 : 0;`
`370`	`370`	`}`
`371`	`371`	`}`