diff --git a/src/noUnusedCssRule.ts b/src/noUnusedCssRule.ts index d90a6c72b..ed4200162 100644 --- a/src/noUnusedCssRule.ts +++ b/src/noUnusedCssRule.ts @@ -169,7 +169,7 @@ class UnusedCssVisitor extends BasicCssAstVisitor { } = ast; // length + 1 because we want to drop the '}' const length = endOffset - startOffset + 1; - this.addFailureAt(startOffset, length, 'Unused styles', this.createReplacement(startOffset, length, '')); + this.addFailureAt(startOffset, length, 'Unused styles', Lint.Replacement.deleteText(startOffset - 1, length + 1)); } } catch (e) { logger.error(e); diff --git a/test/noUnusedCssRule.spec.ts b/test/noUnusedCssRule.spec.ts index 5f28fbdbb..556629242 100644 --- a/test/noUnusedCssRule.spec.ts +++ b/test/noUnusedCssRule.spec.ts @@ -868,8 +868,8 @@ describe('no-unused-css', () => { )!; const replacement = failures[0].getFix() as Replacement; expect(replacement.text).to.eq(''); - expect(replacement.start).to.eq(199); - expect(replacement.end).to.eq(246); + expect(replacement.start).to.eq(14); + expect(replacement.end).to.eq(62); }); it('should work with SASS', () => { @@ -919,8 +919,8 @@ describe('no-unused-css', () => { Config.transformStyle = (code: string) => ({ code, map: null }); const replacement = failures[0].getFix() as Replacement; expect(replacement.text).to.eq(''); - expect(replacement.start).to.eq(168); - expect(replacement.end).to.eq(271); // should be 276 + expect(replacement.start).to.eq(-1); + expect(replacement.end).to.eq(29); }); });