Merge db4c271 into 80e8e41

jscs-dev · Mar 6, 2016 · 304d855 · 304d855
2 parents 80e8e41 + db4c271
commit 304d855
Show file tree

Hide file tree

Showing 3 changed files with 7 additions and 4 deletions.
diff --git a/lib/rules/require-capitalized-comments.js b/lib/rules/require-capitalized-comments.js
@@ -106,6 +106,7 @@
  */
 
 var assert = require('assert');
+var cst = require('cst');
 
 var isPragma = require('../utils').isPragma;
 var letterPattern = require('../../patterns/L');
@@ -279,6 +280,9 @@ module.exports.prototype = {
         var comment = error.additional;
         var first = this._getFirstChar(comment);
 
-        comment.value = comment.value.replace(first, first.toUpperCase());
+        var newValue = comment.value.replace(first, first.toUpperCase());
+        var newComment = new cst.Token(comment.type, newValue);
+
+        comment.parentElement.replaceChild(newComment, comment);
     }
 };
diff --git a/test/specs/rules/disallow-comma-before-line-break.js b/test/specs/rules/disallow-comma-before-line-break.js
@@ -12,7 +12,7 @@ describe('rules/disallow-comma-before-line-break', function() {
         checker.configure(rules);
     });
 
-    describe.skip('autofix', function() {
+    describe('autofix', function() {
         reportAndFix({
             name: 'illegal comma placement in multiline var declaration',
             rules: rules,

diff --git a/test/specs/rules/require-capitalized-comments.js b/test/specs/rules/require-capitalized-comments.js
@@ -317,8 +317,7 @@ describe('rules/require-capitalized-comments', function() {
         });
     });
 
-    // TODO: 3.0
-    describe.skip('fix', function() {
+    describe('fix', function() {
         reportAndFix({
             name: 'simple case',
             rules: {