Merge pull request #71 from plumelo/feature/NEOV-282-fix-freeze

Fixes freeze caused by _selectDebouncer not being cancelled sometimes

cosmoz-data-nav.js

@@ -706,7 +706,7 @@
 			if (isNaN(select)) {
 				return;
 			}
-			this._selectDebouncer = enqueueDebouncer(Debouncer.debounce(this._selectDebouncer,
+			enqueueDebouncer(this._selectDebouncer = Debouncer.debounce(this._selectDebouncer,
 				Polymer.Async.timeOut.after(15),
 				() => {
 					this.animating = true;

package.json

@@ -7,10 +7,10 @@
 		"test": "test"
 	},
 	"devDependencies": {
-		"eslint": "^5.12.0",
-		"eslint-plugin-html": "^5.0.0",
-		"eslint-plugin-mocha": "^5.2.1",
-		"polymer-cli": "^1.8.0",
+		"eslint": "^5.14.0",
+		"eslint-plugin-html": "^5.0.3",
+		"eslint-plugin-mocha": "^5.3.0",
+		"polymer-cli": "^1.9.6",
 		"web-component-tester-istanbul": "Neovici/web-component-tester-istanbul"
 	},
 	"scripts": {

test/basic.html

@@ -316,6 +316,34 @@
 				});
 			});
 
+			suite('navigation', () => {
+				let nav;
+
+				setup(done => {
+					nav = fixture('defaults');
+					nav._templatesObserver.flush();
+					nav.items = Array(10).fill('').map((e, i) => i.toString());
+					flush(done);
+				});
+
+				test('selects next item', () => {
+					nav.setItemById('0', { id: 0 });
+					nav.setItemById('1', { id: 1 });
+					nav._renderQueue();
+					nav._renderQueue();
+					const nextBtn = nav._getElement(0)
+						.querySelector('*')
+						.root.
+						querySelector('[cosmoz-data-nav-select="+1"]');
+					nextBtn.click();
+
+					assert.isOk(nav._selectDebouncer);
+					Polymer.flush();
+					assert.isTrue(nav.animating);
+					assert.equal(nav.selected, 1);
+				});
+			});
+
 			suite('elements buffer', () => {
 				let nav;