Expose oldName and newName to file list templates
This commit is contained in:
parent
0697ac6d55
commit
f68f930dd2
2 changed files with 49 additions and 2 deletions
|
|
@ -29,6 +29,8 @@
|
|||
|
||||
return lineTemplate.render({
|
||||
fileHtmlId: printerUtils.getHtmlId(file),
|
||||
oldName: file.oldName,
|
||||
newName: file.newName,
|
||||
fileName: printerUtils.getDiffName(file),
|
||||
deletedLines: '-' + file.deletedLines,
|
||||
addedLines: '+' + file.addedLines
|
||||
|
|
|
|||
|
|
@ -1,9 +1,53 @@
|
|||
var assert = require('assert');
|
||||
|
||||
var fileListPrinter = new (require('../src/file-list-printer.js').FileListPrinter)();
|
||||
var FileListPrinter = require('../src/file-list-printer.js').FileListPrinter;
|
||||
|
||||
describe('FileListPrinter', function() {
|
||||
describe('generateFileList', function() {
|
||||
it('should expose old and new files to templates', function() {
|
||||
var files = [{
|
||||
addedlines: 12,
|
||||
deletedlines: 41,
|
||||
language: 'js',
|
||||
oldName: 'my/file/name.js',
|
||||
newName: 'my/file/name.js'
|
||||
}, {
|
||||
addedLines: 12,
|
||||
deletedLines: 41,
|
||||
language: 'js',
|
||||
oldName: 'my/file/name1.js',
|
||||
newName: 'my/file/name2.js'
|
||||
}, {
|
||||
addedLines: 12,
|
||||
deletedLines: 0,
|
||||
language: 'js',
|
||||
oldName: 'dev/null',
|
||||
newName: 'my/file/name.js',
|
||||
isNew: true
|
||||
}, {
|
||||
addedLines: 0,
|
||||
deletedLines: 41,
|
||||
language: 'js',
|
||||
oldName: 'my/file/name.js',
|
||||
newName: 'dev/null',
|
||||
isDeleted: true
|
||||
}];
|
||||
|
||||
var fileListPrinter = new FileListPrinter({
|
||||
rawTemplates: {
|
||||
'file-summary-wrapper': '{{{files}}}',
|
||||
'file-summary-line': '{{oldName}}, {{newName}}, {{fileName}}'
|
||||
}
|
||||
});
|
||||
|
||||
var fileHtml = fileListPrinter.generateFileList(files);
|
||||
var expected = 'my/file/name.js, my/file/name.js, my/file/name.js\n' +
|
||||
'my/file/name1.js, my/file/name2.js, my/file/{name1.js → name2.js}\n' +
|
||||
'dev/null, my/file/name.js, my/file/name.js\n' +
|
||||
'my/file/name.js, dev/null, my/file/name.js';
|
||||
|
||||
assert.equal(expected, fileHtml);
|
||||
});
|
||||
|
||||
it('should work for all kinds of files', function() {
|
||||
var files = [{
|
||||
addedLines: 12,
|
||||
|
|
@ -33,6 +77,7 @@ describe('FileListPrinter', function() {
|
|||
isDeleted: true
|
||||
}];
|
||||
|
||||
var fileListPrinter = new FileListPrinter();
|
||||
var fileHtml = fileListPrinter.generateFileList(files);
|
||||
|
||||
var expected =
|
||||
|
|
|
|||
Loading…
Reference in a new issue