stale element fix for selected filter.

tests/page.js

@@ -47,16 +47,24 @@ module.exports = function Page(browser) {
 		return !idSelectors ? '//ul[@id="filters"]' : '//ul[contains(@class, "filters")]';
 	};
 
+	this.getFilterXpathByIndex = function (index) {
+		return this.getFiltersElementXpath() + '/li[' + index + ']/a';
+	};
+
+	this.getSelectedFilterXPathByIndex = function (index) {
+		return this.getFilterXpathByIndex(index) + '[contains(@class, "selected")]';
+	};
+
 	this.getFilterAllXpath = function () {
-		return this.getFiltersElementXpath() + '/li[1]/a';
+		return this.getFilterXpathByIndex(1);
 	};
 
 	this.getFilterActiveXpath = function () {
-		return this.getFiltersElementXpath() + '/li[2]/a';
+		return this.getFilterXpathByIndex(2);
 	};
 
 	this.getFilterCompletedXpath = function () {
-		return this.getFiltersElementXpath() + '/li[3]/a';
+		return this.getFilterXpathByIndex(3);
 	};
 
 	this.xPathForItemAtIndex = function (index) {
@@ -133,10 +141,6 @@ module.exports = function Page(browser) {
 		return this.findByXpath(this.xPathForItemAtIndex(index) + '//label');
 	};
 
-	this.getFilterElements = function () {
-		return this.tryFindByXpath(this.getFilterElementsXpath());
-	};
-
 	this.getItemLabels = function () {
 		var xpath = this.getTodoListXpath() + '/li//label';
 		return this.tryFindByXpath(xpath);

tests/testOperations.js

@@ -173,30 +173,10 @@ function TestOperations(page) {
 		});
 	};
 
-	function isSelected(cssClass) {
-		return cssClass.indexOf('selected') !== -1;
-	}
-
 	this.assertFilterAtIndexIsSelected = function (selectedIndex) {
-		page.getFilterElements().then(function (filterElements) {
-
-			// create an array of promises, each one holding a test
-			var tests = [];
-
-			// push a test into the array, avoiding the classic JS for loops + closures issue!
-			function pushTest(itemIndex) {
-				tests.push(filterElements[itemIndex].getAttribute('class').then(function (cssClass) {
-					assert(selectedIndex === itemIndex ? isSelected(cssClass) : !isSelected(cssClass),
-						'the filter / route at index ' + selectedIndex + ' should have been selected');
-				}));
-			}
-
-			for (var i = 0; i < 3; i++) {
-				pushTest(i);
-			}
-
-			// execute all the tests
-			return Q.all(tests);
+		page.findByXpath(page.getSelectedFilterXPathByIndex(selectedIndex + 1))
+		.then(function (elm) {
+			assert.notEqual(undefined, elm, 'the filter / route at index ' + selectedIndex + ' should have been selected');
 		});
 	};