From aaf97212973eb6db9843c01eb8c2d2ebb643cce0 Mon Sep 17 00:00:00 2001
From: Artur Arseniev <artur.catch@hotmail.it>
Date: Mon, 23 Dec 2019 19:27:54 +0100
Subject: [PATCH] Update Selectors states

---
 src/selector_manager/view/ClassTagsView.js    |  2 +-
 test/specs/selector_manager/index.js          |  1 +
 .../selector_manager/view/ClassTagsView.js    | 26 +++++++++----------
 3 files changed, 15 insertions(+), 14 deletions(-)

diff --git a/src/selector_manager/view/ClassTagsView.js b/src/selector_manager/view/ClassTagsView.js
index 93b5a545d..720842f3e 100644
--- a/src/selector_manager/view/ClassTagsView.js
+++ b/src/selector_manager/view/ClassTagsView.js
@@ -16,7 +16,7 @@ export default Backbone.View.extend({
     <div id="${pfx}up">
       <div id="${pfx}label">${labelHead}</div>
       <div id="${pfx}status-c">
-        <span id="${pfx}input-c">
+        <span id="${pfx}input-c" data-states-c>
           <div class="${ppfx}field ${ppfx}select">
             <span id="${ppfx}input-holder">
               <select id="${pfx}states" data-states>
diff --git a/test/specs/selector_manager/index.js b/test/specs/selector_manager/index.js
index 80c1b2f0e..c96c69a91 100644
--- a/test/specs/selector_manager/index.js
+++ b/test/specs/selector_manager/index.js
@@ -26,6 +26,7 @@ describe('SelectorManager', () => {
 
     test('Able to add default selectors', () => {
       var cm = new SelectorManager().init({
+        em,
         selectors: ['test1', 'test2', 'test3']
       });
       expect(cm.getAll().length).toEqual(3);
diff --git a/test/specs/selector_manager/view/ClassTagsView.js b/test/specs/selector_manager/view/ClassTagsView.js
index e3650ebf9..776d968b3 100644
--- a/test/specs/selector_manager/view/ClassTagsView.js
+++ b/test/specs/selector_manager/view/ClassTagsView.js
@@ -17,6 +17,7 @@ describe('ClassTagsView', () => {
   var coll;
   var target;
   var em;
+  let compTest;
 
   beforeAll(() => {
     document.body.innerHTML = '<div id="fixtures"></div>';
@@ -30,9 +31,10 @@ describe('ClassTagsView', () => {
 
   beforeEach(() => {
     target = new Editor();
+    em = target;
     coll = new Selectors();
     view = new ClassTagsView({
-      config: { em: target },
+      config: { em },
       collection: coll
     });
 
@@ -42,16 +44,16 @@ describe('ClassTagsView', () => {
       }
     };
 
-    testContext.compTargetStub = new Component();
+    compTest = new Component();
+    testContext.compTargetStub = compTest;
 
     fixtures.innerHTML = '';
     fixture.empty().appendTo(fixtures);
     fixture.append(view.render().el);
     testContext.btnAdd = view.$addBtn;
-    testContext.input = view.$el.find('input#' + view.newInputId);
-    testContext.$tags = fixture.find('#tags-c');
-    testContext.$states = fixture.find('#states');
-    testContext.$statesC = fixture.find('#input-c');
+    testContext.input = view.$el.find('[data-input]');
+    testContext.$tags = fixture.find('[data-selectors]');
+    testContext.$statesC = fixture.find('[data-states-c]');
   });
 
   afterEach(() => {
@@ -81,7 +83,7 @@ describe('ClassTagsView', () => {
   test('Stop tag creation', () => {
     testContext.btnAdd.trigger('click');
     testContext.input.val('test');
-    testContext.input.trigger('blur');
+    testContext.input.trigger('focusout');
     expect(testContext.btnAdd.css('display')).not.toEqual('none');
     expect(testContext.input.css('display')).toEqual('none');
     expect(testContext.input.val()).toEqual(null);
@@ -129,9 +131,8 @@ describe('ClassTagsView', () => {
   });
 
   test('Accept new tags', () => {
-    view.compTarget = testContext.compTargetStub;
+    em.setSelected(compTest);
     view.addNewTag('test');
-    view.compTarget = testContext.compTargetStub;
     view.addNewTag('test2');
     expect(testContext.$tags.children().length).toEqual(2);
   });
@@ -155,19 +156,18 @@ describe('ClassTagsView', () => {
   test('States are visible in case of more tags inside', () => {
     coll.add({ label: 'test' });
     view.updateStateVis();
-    expect(testContext.$statesC.css('display')).toEqual('block');
+    expect(testContext.$statesC.css('display')).toEqual('');
   });
 
   test('Update state visibility on new tag', () => {
     sinon.stub(view, 'updateStateVis');
-    sinon.stub(target, 'get').returns(testContext.targetStub);
-    view.compTarget = testContext.compTargetStub;
+    em.setSelected(compTest);
     view.addNewTag('test');
     expect(view.updateStateVis.called).toEqual(true);
   });
 
   test('Update state visibility on removing of the tag', () => {
-    view.compTarget = testContext.compTargetStub;
+    em.setSelected(compTest);
     view.addNewTag('test');
     sinon.stub(view, 'updateStateVis');
     coll.remove(coll.at(0));