From 93c187de2a87f146bfdc9325395ef779740a1fcc Mon Sep 17 00:00:00 2001 From: Peter Bacon Darwin Date: Tue, 10 Nov 2015 11:20:13 +0000 Subject: [PATCH] api-builder: trim blank lines from the end of a text block too --- .../angular.io-package/rendering/trimBlankLines.js | 3 +++ .../angular.io-package/rendering/trimBlankLines.spec.js | 6 +++--- 2 files changed, 6 insertions(+), 3 deletions(-) diff --git a/tools/api-builder/angular.io-package/rendering/trimBlankLines.js b/tools/api-builder/angular.io-package/rendering/trimBlankLines.js index 510c7617ac..cfa0026806 100644 --- a/tools/api-builder/angular.io-package/rendering/trimBlankLines.js +++ b/tools/api-builder/angular.io-package/rendering/trimBlankLines.js @@ -6,6 +6,9 @@ module.exports = function() { while(lines.length && (lines[0].trim() === '')) { lines.shift(); } + while(lines.length && (lines[lines.length-1].trim() === '')) { + lines.pop(); + } return lines.join('\n'); } }; diff --git a/tools/api-builder/angular.io-package/rendering/trimBlankLines.spec.js b/tools/api-builder/angular.io-package/rendering/trimBlankLines.spec.js index 762fc9fd2e..7dde9cf490 100644 --- a/tools/api-builder/angular.io-package/rendering/trimBlankLines.spec.js +++ b/tools/api-builder/angular.io-package/rendering/trimBlankLines.spec.js @@ -11,8 +11,8 @@ describe('trimBlankLines filter', function() { expect(filter.name).toEqual('trimBlankLines'); }); - it('should remove all empty lines from the start of the string', function() { - expect(filter.process('\n\n\nsome text\n\nmore text\n\n')) - .toEqual('some text\n\nmore text\n\n'); + it('should remove empty lines from the start and end of the string', function() { + expect(filter.process('\n \n\nsome text\n \nmore text\n \n')) + .toEqual('some text\n \nmore text'); }); }); \ No newline at end of file