Beminfo 1352 #40
Beminfo 1352 #40
Conversation
…asks/libraries-db.js task module
|
Пожалуйста посмотрите этот pr. |
| describe('_getRootLibNodes', function () { | ||
| it('should return empty list of root library nodes', function () { | ||
| return task._getRootLibNodes(target).then(function (result) { | ||
| result.should.be.instanceOf(Array).and.have.length(0); |
There was a problem hiding this comment.
Может лучше result.should.be.instanceOf(Array).and.be.empty(); ?
There was a problem hiding this comment.
А вот не работает: Property 'empty' of object # is not a function
There was a problem hiding this comment.
на самом деле, кажется что не так важно как написать эту проверку
|
@tormozz48 нужна встреча |
| var local = JSON.parse(JSON.stringify(remote)), | ||
| result = task._compareRegistryFiles(target, local, remote); | ||
|
|
||
| result.should.be.instanceOf(Object); |
| result.modified.should.have.length(0); | ||
| result.removed.should.have.length(0); | ||
| result.added.should.have.length(3); | ||
| should.deepEqual(result.added, [ |
There was a problem hiding this comment.
Может ли быть такое, что в результате версии пойдут не в этом порядке? Если да - то может проверить их на вхождение в массив по отдельности?
Например:
should(result.added).containEql({ lib: 'bem-core', version: 'v2' })
.and.containEql({ lib: 'bem-core', version: 'v2.5.1' })
.and.containEql({ lib: 'bem-core', version: 'v2.6.0' });
|
Я бы для всех тестов _compareRegistryFiles переписал как-то описания на |
| it('should rejected with error if library version data file does not exists in MDS', function () { | ||
| return task._saveLibraryVersionFile({ lib: 'bem-core', version: 'v2' }, target) | ||
| .fail(function (error) { | ||
| error.message.should.equal('Bla'); |
No description provided.