Skip to content

Commit 4f6041e

Browse files
authored
Remove redundant search tests
We don’t really need to test whether there were more than 0 results _and_ more than 1 result.
1 parent dad5ee2 commit 4f6041e

File tree

1 file changed

+0
-7
lines changed

1 file changed

+0
-7
lines changed

test/search.js

Lines changed: 0 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -27,7 +27,6 @@ QUnit.module('search control');
2727
QUnit.test('Search for “about”', function(assert) {
2828
var done = assert.async();
2929
search.searchEngineSearch('about').then(function(results) {
30-
assert.equal(results.length > 0, true, 'got results');
3130
assert.equal(results.length > 1, true, 'got more than 1 result');
3231
assert.equal(indexOfPageInResults('about', results), 0, 'first result is the About page');
3332
done();
@@ -37,7 +36,6 @@ QUnit.test('Search for “about”', function(assert) {
3736
QUnit.test('Search for “can-component”', function(assert) {
3837
var done = assert.async();
3938
search.searchEngineSearch('can-component').then(function(results) {
40-
assert.equal(results.length > 0, true, 'got results');
4139
assert.equal(results.length > 1, true, 'got more than 1 result');
4240
assert.equal(indexOfPageInResults('can-component', results), 0, 'first result is the can-component page');
4341
done();
@@ -47,7 +45,6 @@ QUnit.test('Search for “can-component”', function(assert) {
4745
QUnit.test('Search for “can-connect”', function(assert) {
4846
var done = assert.async();
4947
search.searchEngineSearch('can-connect').then(function(results) {
50-
assert.equal(results.length > 0, true, 'got results');
5148
assert.equal(results.length > 1, true, 'got more than 1 result');
5249
assert.equal(indexOfPageInResults('can-connect', results), 0, 'first result is the can-connect page');
5350
done();
@@ -57,7 +54,6 @@ QUnit.test('Search for “can-connect”', function(assert) {
5754
QUnit.test('Search for “helpers/', function(assert) {
5855
var done = assert.async();
5956
search.searchEngineSearch('helpers/').then(function(results) {
60-
assert.equal(results.length > 0, true, 'got results');
6157
assert.equal(results.length > 1, true, 'got more than 1 result');
6258
done();
6359
});
@@ -66,7 +62,6 @@ QUnit.test('Search for “helpers/', function(assert) {
6662
QUnit.test('Search for “Live Binding”', function(assert) {
6763
var done = assert.async();
6864
search.searchEngineSearch('Live Binding').then(function(results) {
69-
assert.equal(results.length > 0, true, 'got results');
7065
assert.equal(results.length > 1, true, 'got more than 1 result');
7166
assert.equal(indexOfPageInResults('can-stache.Binding', results) < 2, true, 'first result is the can-stache Live Binding page');
7267
done();
@@ -85,7 +80,6 @@ QUnit.test('Search for “Play”', function(assert) {
8580
QUnit.test('Search for “stache”', function(assert) {
8681
var done = assert.async();
8782
search.searchEngineSearch('stache').then(function(results) {
88-
assert.equal(results.length > 0, true, 'got results');
8983
assert.equal(results.length > 1, true, 'got more than 1 result');
9084
assert.equal(indexOfPageInResults('can-stache', results), 0, 'first result is the can-stache page');
9185
done();
@@ -104,7 +98,6 @@ QUnit.test('Search for “%special”', function(assert) {
10498
QUnit.test('Search for “define/map”', function(assert) {
10599
var done = assert.async();
106100
search.searchEngineSearch('define/map').then(function(results) {
107-
assert.equal(results.length > 0, true, 'got results');
108101
assert.equal(results.length > 1, true, 'got more than 1 result');
109102
assert.equal(indexOfPageInResults('can-define/map/map', results), 0, 'first result is the can-define/map/map page');
110103
done();

0 commit comments

Comments
 (0)