Merge pull request #1080 from webcompat/Issue/1079/1

Fixes #1079 - added functional tests for URL and search box update up…
webcompat · Jun 2, 2016 · f1c1620 · f1c1620
2 parents 13e31e2 + cbe8624
commit f1c1620
Showing 1 changed file with 28 additions and 2 deletions.
diff --git a/tests/functional/search-non-auth.js b/tests/functional/search-non-auth.js
@@ -52,10 +52,9 @@ define([
     },
 
     'Clicking on label search suggestion works': function() {
-      var params = '?q=dfjdkfjdkfjkdfjdkjf';
       return this.remote
         .setFindTimeout(intern.config.wc.pageLoadTimeout)
-        .get(require.toUrl(url('/issues') + params))
+        .get(require.toUrl(url('/issues')))
         .findByCssSelector('[data-remotename=browser-android]').click()
         .end()
         // click the first suggestion, which is "android"
@@ -66,6 +65,33 @@ define([
         .end();
     },
 
+    'Clicking on label search adds query parameter to the URL': function() {
+      return this.remote
+        .setFindTimeout(intern.config.wc.pageLoadTimeout)
+        .get(require.toUrl(url('/issues')))
+        .findByCssSelector('[data-remotename=browser-android]').click()
+        .end()
+        .getCurrentUrl()
+        .then(function(currUrl) {
+          assert.include(currUrl, 'q=label%3Abrowser-android', 'Url updated with label name');
+        })
+        .end();
+    },
+
+    'Clicking on label search updates the search input': function() {
+      return this.remote
+        .setFindTimeout(intern.config.wc.pageLoadTimeout)
+        .get(require.toUrl(url('/issues')))
+        .findByCssSelector('[data-remotename=browser-android]').click()
+        .end()
+        .sleep(2000)
+        .findById('js-SearchForm-input').getProperty('value')
+        .then(function(searchText) {
+          assert.include(searchText, 'label:browser-android', 'Url updated with label name');
+        })
+        .end();
+    },
+
     'Search input is visible': function() {
       return this.remote
         .get(require.toUrl(url('/issues')))