Merge lp:~elopio/u1-test-utils/smart_scopes_tests into lp:~canonical-isd-hackers/u1-test-utils/test-in-dash-payments

[Leo Arias]

[lp:~elopio/u1-test-utils/smart_scopes_tests]

66. By Leo Arias on 2013-06-23

Preview the result.

67. By Leo Arias on 2013-06-23

Click the View button.

68. By Leo Arias on 2013-06-23

Check the URL on the browser.

[Vincent Ladeuil (vila)]

26 +def _wait_for_result_settle(test):

Why don't you use Eventually here ?

It seems unlikely that the code will loop forever but it can be exited with 0 rows. Why not use a specific number of rows and an overall timeout (which Eventually implements) instead ?

review: Needs Information

[Natalia Bidart (nataliabidart)]

Setting MP as Rejected because is more than 500 days old, and did not have any activity recorded. Change status again if this MP is still current, and please ping someone for review and landing.
Thanks!

Unmerged revisions

68. By Leo Arias on 2013-06-23

Check the URL on the browser.

67. By Leo Arias on 2013-06-23

Click the View button.

66. By Leo Arias on 2013-06-23

Preview the result.

65. By Leo Arias on 2013-06-23

Added the first part of the first scope search test.

=== modified file 'tests/dash.py'
--- tests/dash.py	2013-06-19 20:35:04 +0000
+++ tests/dash.py	2013-06-23 07:46:25 +0000
@@ -1,3 +1,7 @@
+import time
+
+import unity.emulators.dash 
+from autopilot import input
 from autopilot.matchers import Eventually
 from testtools.matchers import (
     Equals,
@@ -6,6 +10,36 @@
 )
 
 
+def search_home_scope(test, query):
+    test.addCleanup(test.unity.dash.ensure_hidden)
+    test.unity.dash.ensure_visible()
+    test.addCleanup(test.unity.dash.clear_search)
+    test.keyboard.type(query)
+    _wait_for_result_settle(test)
+
+
+# Copied from unity tests. TODO move it to the unity emulator.
+# --elopio -2013-06-22
+def _wait_for_result_settle(test):
+    """wait for row count to settle"""
+    old_row_count = -1
+    new_row_count = test.unity.dash.get_num_rows()
+    while(old_row_count != new_row_count):
+        time.sleep(1)
+        old_row_count = new_row_count
+        new_row_count = test.unity.dash.get_num_rows()
+
+
+def get_result_from_category_by_name(test, name, category_name):
+    current_scope = test.unity.dash.get_current_scope()
+    category = current_scope.get_category_by_name(category_name)
+    test.assertTrue(category.is_visible)
+    for result in category.get_results():
+        if result.name == name:
+            return result
+    return None
+
+
 def search_music_scope(test, query):
     test.addCleanup(test.unity.dash.ensure_hidden)
     test.unity.dash.reveal_music_scope()
@@ -37,6 +71,7 @@
 def preview_result(test, result):
     current_scope = test.unity.dash.get_current_scope()
     test.assertThat(current_scope.get_num_visible_categories(), GreaterThan(0))
+    test.addCleanup(ensure_preview_closed, test)
     result.preview()
     _wait_for_preview_animation(test)
 
@@ -66,7 +101,21 @@
         Eventually(NotEquals(None)))
     container = test.unity.dash.view.get_preview_container()
     test.assertThat(container.waiting_preview, Eventually(Equals(False)))
-    return container.current_preview    
+    return container.current_preview
+
+
+def execute_preview_action_by_label(test, preview, label):
+    # XXX Most of this code comes from the unity Preview emulator, that clicks
+    # an action by its id. On the smart scopes, the id is the URL, not nice.
+    # Ask David Calle -- elopio - 2013-06-23
+    actions = preview.get_children_by_type(unity.emulators.dash.ActionButton)
+    for action in actions:
+        if action.label == label:
+            tx = action.x + (action.width / 2)
+            ty = action.y + (action.height / 2)
+            m = input.Mouse.create()
+            m.move(tx, ty)
+            m.click()
 
 
 class Window(object):
@@ -86,7 +135,6 @@
     def get_preview(self, test, good):
         search_music_scope(test, good.search)        
         result = self.get_result(test)
-        test.addCleanup(ensure_preview_closed, test)
         preview_result(test, result)
         # We have the preview now
         return Preview(self.user, good)

=== modified file 'tests/schema.py'
--- tests/schema.py	2013-05-23 23:27:05 +0000
+++ tests/schema.py	2013-06-23 07:46:25 +0000
@@ -18,5 +18,9 @@
     class musicsearch(schema.Section):
         musicsearch_server_url = schema.StringOption()
 
+    class smartscopes(schema.Section):
+        smartscopes_server_url = schema.StringOption()
+        smartscopes_geo_store = schema.StringOption()
+
 
 schema = InDashPaymentsSchema

=== added file 'tests/test_smart_scopes.py'
--- tests/test_smart_scopes.py	1970-01-01 00:00:00 +0000
+++ tests/test_smart_scopes.py	2013-06-23 07:46:25 +0000
@@ -0,0 +1,37 @@
+import os
+import subprocess
+
+# FIXME do not use django config to get the server settings.
+# -- elopio - 2013-05-15
+from django import conf
+
+import tests
+from tests import browsers, dash
+
+
+class SmartScopesSearchTestCase(tests.UnitySSTestcase):
+
+    def setUp(self):
+        super(SmartScopesSearchTestCase, self).setUp()
+        self._run_home_scope()
+
+    def _run_home_scope(self):
+        os.environ['SMART_SCOPES_SERVER"'] = (
+            conf.settings.SMARTSCOPES_SERVER_URL)
+        os.environ['SMART_SCOPES_GEO_STORE'] = (
+            conf.settings.SMARTSCOPES_GEO_STORE)
+        os.system('pkill unity-scope-home*')
+        subprocess.Popen(
+            '/usr/lib/x86_64-linux-gnu/unity-scope-home/unity-scope-home')
+
+    def test_search_wikipedia(self):
+        dash.search_home_scope(self, 'wikipedia:Test')
+        result = dash.get_result_from_category_by_name(
+            self, 'Test', 'Reference')
+        dash.preview_result(self, result)
+        preview = dash.get_current_preview(self)
+        dash.execute_preview_action_by_label(self, preview, 'View')
+        browser = browsers.Browser(self)
+        # We've been redirected to the web
+        browser.switch_to_new_window(self)
+        self.assertEqual(browser.url, 'http://en.wikipedia.org/wiki/Test')