Browse Source

Fix regressions

pull/261/head
Artur Arseniev 9 years ago
parent
commit
d514ffbb55
  1. 6
      test/specs/asset_manager/index.js
  2. 25
      test/specs/asset_manager/model/Assets.js
  3. 9
      test/specs/asset_manager/view/AssetImageView.js
  4. 7
      test/specs/asset_manager/view/AssetsView.js

6
test/specs/asset_manager/index.js

@ -72,12 +72,6 @@ describe('Asset Manager', () => {
expect(asset2.get('src')).toEqual(imgObj.src + '2'); expect(asset2.get('src')).toEqual(imgObj.src + '2');
}); });
it('Src is unique', () => {
obj.add(imgObj);
obj.add(imgObj);
expect(obj.getAll().length).toEqual(1);
});
it('Remove asset', () => { it('Remove asset', () => {
obj.add(imgObj); obj.add(imgObj);
obj.remove(imgObj.src); obj.remove(imgObj.src);

25
test/specs/asset_manager/model/Assets.js

@ -21,31 +21,6 @@ module.exports = {
it('Collection is empty', () => { it('Collection is empty', () => {
expect(obj.length).toEqual(0); expect(obj.length).toEqual(0);
}); });
it("Can't insert assets without src", () => {
obj.add({});
expect(obj.length).toEqual(0);
obj.add([{},{},{}]);
expect(obj.length).toEqual(0);
});
it("Insert assets only with src", () => {
obj.add([{},{src:'test'},{}]);
expect(obj.length).toEqual(1);
});
it('addImg creates new asset', () => {
obj.addImg('/img/path');
expect(obj.length).toEqual(1);
});
it('addImg asset is correct', () => {
obj.addImg('/img/path');
var asset = obj.at(0);
expect(asset.get('type')).toEqual('image');
expect(asset.get('src')).toEqual('/img/path');
});
}); });
} }
}; };

9
test/specs/asset_manager/view/AssetImageView.js

@ -16,6 +16,7 @@ module.exports = {
var coll = new Assets(); var coll = new Assets();
var model = coll.add({ type:'image', src: '/test' }); var model = coll.add({ type:'image', src: '/test' });
this.view = new AssetImageView({ this.view = new AssetImageView({
collection: new Assets(),
config : {}, config : {},
model model
}); });
@ -39,17 +40,17 @@ module.exports = {
it('Has preview box', function() { it('Has preview box', function() {
var $asset = this.view.$el; var $asset = this.view.$el;
expect($asset.find('#preview').length).toEqual(1); expect($asset.find('.preview').length).toEqual(1);
}); });
it('Has meta box', function() { it('Has meta box', function() {
var $asset = this.view.$el; var $asset = this.view.$el;
expect($asset.find('#meta').length).toEqual(1); expect($asset.find('.meta').length).toEqual(1);
}); });
it('Has close button', function() { it('Has close button', function() {
var $asset = this.view.$el; var $asset = this.view.$el;
expect($asset.find('#close').length).toEqual(1); expect($asset.find('[data-toggle=asset-remove]').length).toEqual(1);
}); });
}); });
@ -72,7 +73,7 @@ module.exports = {
it('Could be removed', function() { it('Could be removed', function() {
var spy = sinon.spy(); var spy = sinon.spy();
this.view.model.on("remove", spy); this.view.model.on("remove", spy);
this.view.$el.find('#close').trigger('click'); this.view.$el.find('[data-toggle=asset-remove]').trigger('click');
expect(spy.called).toEqual(true); expect(spy.called).toEqual(true);
}); });

7
test/specs/asset_manager/view/AssetsView.js

@ -1,4 +1,5 @@
var AssetsView = require('asset_manager/view/AssetsView'); var AssetsView = require('asset_manager/view/AssetsView');
var FileUploader = require('asset_manager/view/FileUploader');
var Assets = require('asset_manager/model/Assets'); var Assets = require('asset_manager/model/Assets');
module.exports = { module.exports = {
@ -17,7 +18,9 @@ module.exports = {
this.coll = new Assets([]); this.coll = new Assets([]);
this.view = new AssetsView({ this.view = new AssetsView({
config: {}, config: {},
collection: this.coll collection: this.coll,
globalCollection: new Assets([]),
fu: new FileUploader({})
}); });
obj = this.view; obj = this.view;
this.$fixture.empty().appendTo(this.$fixtures); this.$fixture.empty().appendTo(this.$fixtures);
@ -85,7 +88,7 @@ module.exports = {
obj.addFromStr({ obj.addFromStr({
preventDefault() {} preventDefault() {}
}); });
var asset = obj.collection.at(0); var asset = obj.options.globalCollection.at(0);
expect(asset.get('src')).toEqual('test'); expect(asset.get('src')).toEqual('test');
}); });

Loading…
Cancel
Save