Merge lp:~elopio/u1-test-utils/page-objects into lp:u1-test-utils

[Leo Arias]

Refactor pages to use page objects.

We have been playing with the u1servers acceptance test, and experimenting with a pure page object pattern, nicely explained in [1].
This branch is a prototype for SSO helpers, to gather feedback and opinions.

The functions in helpers.py show in a great way the cool ideas of this pattern:
 * The methods are the actions a user can do with each page.
 * The methods return other pages. This shows in a clear way the site workflow and gives immediate access to the methods available in the new page.
 * Actions with different results are modeled in different methods. This helps avoiding conditional logic in the tests and make them more expressive.

[1] http://code.google.com/p/selenium/wiki/PageObjects

[lp:~elopio/u1-test-utils/page-objects]

21. By Leo Arias on 2013-01-22

Removed the site not recognized default values.

[Matias Bordese (matiasb)]

Liking the pages approach.

A few questions/comments:

- return_page should be a Page instance? related, a Page class is expected here (l.103)?

- the return_page argument seems to be just for checks? (verifying the user is correctly redirected after an action to that page); at first I thought it was a way to tell the action where to redirect me after it. I would expect getting a page instance as result and maybe the caller check the got object in an instance of the right page?

- shouldn't l.149 and l.332 be comments instead?

[lp:~elopio/u1-test-utils/page-objects]

22. By Leo Arias on 2013-01-22

Now we are passing instantiated return pages.

[Leo Arias (elopio)]

> Liking the pages approach.
>
> A few questions/comments:
>
> - return_page should be a Page instance? related, a Page class is expected
> here (l.103)?

Yes. We are passing instantiated page objects. At first I tried passing just the classes so the verification could be done when we instantiated them, but it seemed harder to understand. l.103 is a typo left from that experiment, I fixed it.

> - the return_page argument seems to be just for checks? (verifying the user is
> correctly redirected after an action to that page); at first I thought it was
> a way to tell the action where to redirect me after it. I would expect getting
> a page instance as result and maybe the caller check the got object in an
> instance of the right page?

It's a way to make sure we are on the right page, and a way to make it clearer for the caller where I will be after the call.
A page instance as a result is what I like too, but I thought the check could be done on the action that leads to that page instead of leaving the job to every caller. I added to the __init__ a call to assert_page_is_open, and I added it too to all methods that don't instantiate the return page, so there's no need for the caller to do an extra check. Maybe I'm not getting right what you are saying, if you can give me a little code snippet it will be easier.

> - shouldn't l.149 and l.332 be comments instead?

I think not. Why? They are docstrings, and according to pep257 they should use triple double quotes. It really doesn't say anything specific to class attributes, but that's what I understood. I might be wrong.

[Matias Bordese (matiasb)]

> It's a way to make sure we are on the right page, and a way to make it clearer
> for the caller where I will be after the call.

In that case I would make it clearer in the method description that you are not redirected there but that's where you expect to be after the call (instead of 'The page to return on a successful log in.').

> A page instance as a result is what I like too, but I thought the check could
> be done on the action that leads to that page instead of leaving the job to
> every caller.

Well, right now the caller should know where it would be redirected and pass the respective page instance to the action. If that wasn't required it would make more sense to me to the action internally check the page is ok, but it sounds like the caller is the one who knows where it should be redirected.

> I added to the __init__ a call to assert_page_is_open, and I
> added it too to all methods that don't instantiate the return page, so there's
> no need for the caller to do an extra check. Maybe I'm not getting right what
> you are saying, if you can give me a little code snippet it will be easier.

I was thinking on something like (pseudo code):

page = sign_in(user)
self.assert_is_page(page, ExpectedPage) # isinstance?

instead of:

expected_page = ExpectedPage()
page = sign_in(user, return_page=expected_page)

But this is just an opinion, maybe I'm missing something/context.

> I think not. Why? They are docstrings, and according to pep257 they should use
> triple double quotes. It really doesn't say anything specific to class
> attributes, but that's what I understood. I might be wrong.

They look as comments about the attributes and not a class docstring, I think.

This for example (l.331-2):
    title = "{0}'s details"
    """The title of the page. The parameter is the user name."""
I think should be:
    # The title of the page. The parameter is the user name.
    title = "{0}'s details"

[Leo Arias (elopio)]

> > It's a way to make sure we are on the right page, and a way to make it
> clearer
> > for the caller where I will be after the call.
>
> In that case I would make it clearer in the method description that you are
> not redirected there but that's where you expect to be after the call (instead
> of 'The page to return on a successful log in.').

You are right, I'll try to improve that.

> > A page instance as a result is what I like too, but I thought the check
> could
> > be done on the action that leads to that page instead of leaving the job to
> > every caller.
>
> Well, right now the caller should know where it would be redirected and pass
> the respective page instance to the action. If that wasn't required it would
> make more sense to me to the action internally check the page is ok, but it
> sounds like the caller is the one who knows where it should be redirected.
>
> > I added to the __init__ a call to assert_page_is_open, and I
> > added it too to all methods that don't instantiate the return page, so
> there's
> > no need for the caller to do an extra check. Maybe I'm not getting right
> what
> > you are saying, if you can give me a little code snippet it will be easier.
>
> I was thinking on something like (pseudo code):
>
> page = sign_in(user)
> self.assert_is_page(page, ExpectedPage) # isinstance?

The problem here is that sign_in doesn't know what's the expected page. If it's called from sso, it would be the account page. If it's called from pay, it would be the payment history page. And from u1, it would be the dashboard.

But what you say is actually an option. It could be something like:

anonymous_home_page = AnonymousHomePage()
ubuntu_single_sign_on = anonymous_home_page.go_to_sign_in()
ubuntu_single_sign_on.sign_in(user)
# Here is a workflow glitch
user_home_page = UserHomePage()
user_home_page.do_the_test()

I think I would prefer:

anonymous_home_page = AnonymousHomePage()
ubuntu_single_sign_on = anonymous_home_page.go_to_sign_in()
user_home_page = ubuntu_single_sign_on.sign_in(user, UserHomePage())
user_home_page.do_the_test()

But I'm definitely not sure about which is best. And maybe both can be improved.

> They look as comments about the attributes and not a class docstring, I think.
>
> This for example (l.331-2):
> title = "{0}'s details"
> """The title of the page. The parameter is the user name."""
> I think should be:
> # The title of the page. The parameter is the user name.
> title = "{0}'s details"

Yes. It was Vincent who told me that the comment was expected after the variable. Any insights Vincent?

[Natalia Bidart (nataliabidart)]

Regarding the comments for attributes, as you can see in the pep-257 intro:

'A docstring is a string literal that occurs as the first statement in a module, function, class, or method definition. Such a docstring becomes the __doc__ special attribute of that object.'

So, class or instance variables do not have docstrings, ergo the clarification added in the source (which comes handy to the one inspecting the class) should be a comment, if not relevant to go into the class' docstring. I personally prefer adding those details to the class' docstring, so this particular case:

329 +class YourAccount(Page):
330 +
331 + title = "{0}'s details"
332 + """The title of the page. The parameter is the user name."""

would result in:

329 +class YourAccount(Page):
3xx + """A page showing the details for the currently logged in user.
3xx +
3xx + This class defines a class attribute 'title' that represents the title of
3xx + the page, and is built using the user name.
3xx +
3xx + """
3xx +
332 + title = "{0}'s details"

And since we're talking about pep-257, the docstring for make_all_information_available_to_website (and every other docstring that has the first line longer than a single 80 column line) is not pep-257 compliant.

Any chance you fix that?

[Leo Arias (elopio)]

> Any chance you fix that?

Of course, sounds good.

I've just read the pep again, and it says: "The docstring for a class should summarize its behavior and list the public methods and instance variables."

[lp:~elopio/u1-test-utils/page-objects]

23. By Leo Arias on 2013-01-22

Fixed comments style.

[Leo Arias (elopio)]

> I think I would prefer:
>
> anonymous_home_page = AnonymousHomePage()
> ubuntu_single_sign_on = anonymous_home_page.go_to_sign_in()
> user_home_page = ubuntu_single_sign_on.sign_in(user, UserHomePage())
> user_home_page.do_the_test()

Actually, there's a problem here because we shouldn't instantiate pages that are not visible. So, instead of this I think I would go back to my first approach:

anonymous_home_page = AnonymousHomePage()
ubuntu_single_sign_on = anonymous_home_page.go_to_sign_in()
# UserHomePage is the name of the class, and it will be instantiated by sign_in just before returning it.
user_home_page = ubuntu_single_sign_on.sign_in(user, UserHomePage)
user_home_page.do_the_test()

Or, the other option that's not sending any return page to SSO, and leave it all to the caller.

[lp:~elopio/u1-test-utils/page-objects]

24. By Leo Arias on 2013-01-22

Improved the comment on return_page.

25. By Leo Arias on 2013-01-22

pages.__init__.py is now not necessary.

26. By Leo Arias on 2013-01-24

Remove the return pages, and leave them to the caller.

[Natalia Bidart (nataliabidart)]

Branch looks good! Approving with some comments:

* I personally would not recommend changing the order of the param to existing API (so I would not swap is_site_recognized with user, I would leave user first). But I also understand this lib is not heavily used, so you can leave as is. What I do think we need to leave in place is a default value for is_site_recognized.

* I advice against having method that return different stuff depending on the branch being used. Any chance that
create_new_account returns something more unified (I'm not sure what type does site_not_recognized.yes_sign_me_in() has)? In any case, if you feel like this is the right approach, feel free to leave as is.

* Same two comments for sign_in.

review: Approve

[lp:~elopio/u1-test-utils/page-objects]

27. By Leo Arias on 2013-01-29

Yes sign me in returns no value, and inverted the order of the parameters.

=== modified file 'u1testutils/sst/sso/helpers.py'
--- u1testutils/sst/sso/helpers.py	2013-01-16 02:55:27 +0000
+++ u1testutils/sst/sso/helpers.py	2013-01-29 14:54:22 +0000
@@ -14,17 +14,11 @@
 # You should have received a copy of the GNU General Public License along
 # with this program.  If not, see <http://www.gnu.org/licenses/>.
 
-from u1testutils.sst.sso.pages import (
-    create_account,
-    log_in,
-    registration_mail_sent,
-    site_not_recognised,
-    subheader,
-)
+from u1testutils.sst.sso import pages
 from u1testutils.sst.sso.utils import mail
 
 
-def create_new_account(user, not_recognized=True):
+def create_new_account(user, is_site_recognized=True):
     """Create a new account on Ubuntu Single Sign On.
 
     The browser must be on the Log in page.
@@ -34,22 +28,27 @@
     Keyword arguments:
     user -- The user object with the information for the new account.
         It must have the full_name, email and password attributes.
-    not_recognized -- If True, the user will be logged in even if the site
-        is not recognized, and all the user information will be made available
-        to the site. Default is True.
+    is_site_recognized -- Boolean indicating if the site that requested the
+        log in is recognized. If it is not recognized, all the user available
+        information will be send to it. Default is True.
 
     """
-    log_in.assert_page()
-    log_in.click_create_new_account()
-    create_account.create_ubuntu_sso_account(user)
+    log_in = pages.LogIn()
+    create_account = log_in.go_to_create_new_account()
+    registration_mail_sent = create_account.create_ubuntu_sso_account(user)
     confirmation_code = mail.get_verification_code_for_address(user.email)
-    registration_mail_sent.confirm_email(confirmation_code)
-    if not_recognized and site_not_recognised.is_site_not_recognized():
-        site_not_recognised.make_all_information_available_to_website()
-        site_not_recognised.yes_sign_me_in()
-
-
-def sign_in(user=None, not_recognized=True):
+    if is_site_recognized:
+        registration_mail_sent.confirm_email_to_site_recognized(
+            confirmation_code)
+    else:
+        site_not_recognized = \
+            registration_mail_sent.confirm_email_to_site_not_recognized(
+                confirmation_code)
+        site_not_recognized.make_all_information_available_to_website()
+        site_not_recognized.yes_sign_me_in()
+
+
+def sign_in(user=None, is_site_recognized=True):
     """Log in with an Ubuntu Single Sign On account.
 
     The browser must be on the Log in page.
@@ -62,20 +61,21 @@
         If its value is None, it means that the user has already started a
         session on Ubuntu Single Sign On and it's not necessary to enter the
         credentials again. Default is None.
-    not_recognized -- If True, the user will be logged in even if the site
-        is not recognized, and all the user information will be made available
-        to the site. Default is True.
+    is_site_recognized -- Boolean indicating if the site that requested the
+        log in is recognized. If it is not recognized, all the user available
+        information will be send to it. Default is True.
 
     """
-    if user is not None:
-        log_in.assert_page()
-        log_in.log_in(user)
-    if not_recognized and site_not_recognised.is_site_not_recognized():
-        site_not_recognised.make_all_information_available_to_website()
-        site_not_recognised.yes_sign_me_in()
-
-
-def log_out():
+    log_in = pages.LogIn()
+    if is_site_recognized:
+        log_in.log_in_to_site_recognized(user)
+    else:
+        site_not_recognized = log_in.log_in_to_site_not_recognized(user)
+        site_not_recognized.make_all_information_available_to_website()
+        site_not_recognized.yes_sign_me_in()
+
+
+def log_out(user_name):
     """Log out from the Ubuntu Single Sign On site.
 
     The browser must be on the Single Sign On site, and the user must have the
@@ -84,6 +84,9 @@
     If the log out succeeds, the browser will be on the Ubuntu Single Sign On
     logout page.
 
+    Keyword arguments:
+    user_name -- The name of the logged in user.
+
     """
-    assert subheader.is_logged_in()
-    subheader.log_out()
+    your_account = pages.YourAccount(user_name)
+    return your_account.sub_header.log_out()

=== removed directory 'u1testutils/sst/sso/pages'
=== renamed file 'u1testutils/sst/sso/pages/__init__.py' => 'u1testutils/sst/sso/pages.py'
--- u1testutils/sst/sso/pages/__init__.py	2013-01-16 02:55:27 +0000
+++ u1testutils/sst/sso/pages.py	2013-01-29 14:54:22 +0000
@@ -0,0 +1,279 @@
+# -*- coding: utf-8 -*-
+
+# Copyright 2013 Canonical Ltd.
+#
+# This program is free software: you can redistribute it and/or modify it
+# under the terms of the GNU Lesser General Public License version 3, as
+# published by the Free Software Foundation.
+#
+# This program is distributed in the hope that it will be useful, but
+# WITHOUT ANY WARRANTY; without even the implied warranties of
+# MERCHANTABILITY, SATISFACTORY QUALITY, or FITNESS FOR A PARTICULAR
+# PURPOSE.  See the GNU Lesser General Public License for more details.
+#
+# You should have received a copy of the GNU General Public License along
+# with this program.  If not, see <http://www.gnu.org/licenses/>.
+
+import sst.actions
+
+
+class Page(object):
+    """Base class for the page objects used in acceptance testing.
+
+    Instance variables:
+    title -- The title of the page.
+
+    """
+
+    title = None
+
+    def __init__(self):
+        super(Page, self).__init__()
+        self.assert_page_is_open()
+
+    def assert_page_is_open(self):
+        """Assert that the page is open."""
+        sst.actions.assert_title(self.title)
+
+
+class LogIn(Page):
+    """Log in page of the Ubuntu Single Sign On website.
+
+    This is a subclass of the Page object. It extends the assert_page_is_open
+    method with additional verifications and adds methods for the actions
+    available in this page.
+
+    Instance variables:
+    title -- The title of the page.
+
+    """
+
+    title = 'Log in'
+
+    def assert_page_is_open(self):
+        """Assert that the page is open."""
+        super(LogIn, self).assert_page_is_open()
+        sst.actions.assert_element(tag='h1', text='Ubuntu Single Sign On')
+        sst.actions.assert_element(tag='h2', text='Are you new?')
+
+    def log_in_to_site_recognized(self, user=None):
+        """Fill the log in form and continue to the site that requested it.
+
+        Keyword arguments:
+        user -- The user credentials. It must have the attributes email and
+            password. If None is passed as the user, it means that the user
+            has already started session on the identity provider and it's not
+            necessary to enter the credentials again.
+
+        """
+        self._log_in(user)
+
+    def log_in_to_site_not_recognized(self, user=None):
+        """Fill the log in form and continue to the next step.
+
+        As the site is not recognized, the next step is the page where the
+        user can select the information that will be send to the site.
+
+        Keyword arguments:
+        user -- The user credentials. It must have the attributes email and
+            password. If None is passed as the user, it means that the user
+            has already started session on the identity provider and it's not
+            necessary to enter the credentials again.
+
+        """
+        self._log_in(user)
+        return SiteNotRecognized()
+
+    def _log_in(self, user=None):
+        if user is not None:
+            sst.actions.wait_for(sst.actions.assert_title, 'Log in')
+            self._fill_log_in_form(user.email, user.password)
+            self._click_continue_button()
+        else:
+            # If None is passed as the user, it means that the user has
+            # already started session on the identity provider and it's not
+            # necessary to enter the credentials again.
+            pass
+
+    def _fill_log_in_form(self, email, password):
+        sst.actions.write_textfield('id_email', email)
+        sst.actions.write_textfield('id_password', password)
+
+    def _click_continue_button(self):
+        continue_button = sst.actions.get_element(
+            css_class='btn', name='continue')
+        sst.actions.click_button(continue_button)
+
+    def go_to_create_new_account(self):
+        """Go to the Create new account page."""
+        new_account_link = sst.actions.get_element(id='new-account-link')
+        sst.actions.click_link(new_account_link)
+        return CreateAccount()
+
+
+class CreateAccount(Page):
+    """Create account page of the Ubuntu Single Sign On website.
+
+    This is a subclass of the Page object. It adds methods for the actions
+    available in this page.
+
+    Instance variables:
+    title -- The title of the page.
+
+    """
+
+    title = 'Create account'
+
+    def create_ubuntu_sso_account(self, user):
+        """Fill the new account form and continue to the next step.
+
+        Keyword arguments:
+        user -- The user credentials. It must have the attributes email and
+            password.
+
+        """
+        self._fill_new_account_form(user)
+        self._click_continue()
+        return RegistrationMailSent()
+
+    def _fill_new_account_form(self, user):
+        sst.actions.write_textfield('id_displayname', user.full_name)
+        sst.actions.write_textfield('id_email', user.email)
+        sst.actions.write_textfield('id_password', user.password)
+        sst.actions.write_textfield('id_passwordconfirm', user.password)
+
+    def _click_continue(self):
+        continue_button = sst.actions.get_element(name='continue')
+        sst.actions.click_button(continue_button)
+
+
+class RegistrationMailSent(Page):
+    """Registration mail sent page of the Ubuntu Single Sign On website.
+
+    This is a subclass of the Page object. It extends the assert_page_is_open
+    method with additional verifications and adds methods for the actions
+    available in this page.
+
+    Instance variables:
+    title -- The title of the page.
+
+    """
+
+    title = 'Registration mail sent'
+
+    def confirm_email_to_site_recognized(self, confirmation_code):
+        """Confirm email and continue to the site that requested the log in.
+
+        Keyword arguments:
+        confirmation_code -- The confirmation code sent to the user email
+            address.
+
+        """
+        self._confirm_email(confirmation_code)
+
+    def confirm_email_to_site_not_recognized(self, confirmation_code):
+        """Enter the confirmation code and continue to the next step.
+
+        As the site is not recognized, the next step is the page where the
+        user can select the information that will be send to the site.
+
+        Keyword arguments:
+        confirmation_code -- The confirmation code sent to the user email
+            address.
+
+        """
+        self._confirm_email(confirmation_code)
+        return SiteNotRecognized()
+
+    def _confirm_email(self, confirmation_code):
+        self._enter_confirmaiton_code(confirmation_code)
+        self._click_continue_button()
+
+    def _enter_confirmation_code(self, confirmation_code):
+        confirmation_code_text_field = sst.actions.get_element(
+            name='confirmation_code')
+        sst.actions.write_textfield(confirmation_code_text_field,
+                                    confirmation_code)
+
+    def _click_continue_button(self):
+        continue_button = sst.actions.get_element(css_class='btn',
+                                                  text='Continue')
+        sst.actions.click_button(continue_button)
+
+
+class SiteNotRecognized(Page):
+    """Site not Recognized page of the Ubuntu Single Sign On website.
+
+    This is a subclass of the Page object. It adds methods for the actions
+    available in this page.
+
+    Instance variables:
+    title -- The title of the page.
+
+    """
+    def make_all_information_available_to_website(self):
+        """Select all the user available information.
+
+        This information will be send to the site that requested the log in.
+
+        """
+        information_checkboxes = self._get_information_checkboxes()
+        for checkbox in information_checkboxes:
+            sst.actions.set_checkbox_value(checkbox, True)
+        return self
+
+    def _get_information_checkboxes(self):
+        return sst.actions.get_elements_by_css(
+            'form[name="decideform"] > .info-items > .list > li > '
+            'input[type="checkbox"]')
+
+    def yes_sign_me_in(self):
+        """Accept to sign in to the site not recognized and go back to it."""
+        sign_me_in_button = sst.actions.get_element(css_class='btn',
+                                                    name='yes')
+        sst.actions.click_button(sign_me_in_button)
+
+
+class YourAccount(Page):
+    """Your account page of the Ubuntu Single Sign On website.
+
+    This is a subclass of the Page object. It extends the constructor to
+    to receive the user name, and adds methods for the actions available in
+    this page.
+
+    Instance variables:
+    title -- The title of the page. It's build when the page is instantiated
+        using the user name.
+    sub_header -- The sub header menu displayed on the pages shown to logged
+        in users.
+
+    """
+
+    title = "{0}'s details"
+
+    def __init__(self, user_name):
+        super(YourAccount, self).__init__()
+        self.title = self.title.format(user_name)
+        self.sub_header = _UserSubHeader()
+
+
+class _UserSubHeader(object):
+
+    def log_out(self):
+        """Log out from the web site."""
+        sst.actions.click_link('logout-link')
+        return YouHaveBeenLoggedOut()
+
+
+class YouHaveBeenLoggedOut(Page):
+    """Your account page of the Ubuntu Single Sign On website.
+
+    This is a subclass of the Page object.
+
+    Instance variables:
+    title -- The title of the page. It's build when the page is instantiated
+        using the user name.
+
+    """
+
+    title = 'You have been logged out'

=== removed file 'u1testutils/sst/sso/pages/create_account.py'
--- u1testutils/sst/sso/pages/create_account.py	2013-01-16 15:16:50 +0000
+++ u1testutils/sst/sso/pages/create_account.py	1970-01-01 00:00:00 +0000
@@ -1,37 +0,0 @@
-# -*- coding: utf-8 -*-
-
-# Copyright 2012, 2013 Canonical Ltd.
-#
-# This program is free software: you can redistribute it and/or modify it
-# under the terms of the GNU Lesser General Public License version 3, as
-# published by the Free Software Foundation.
-#
-# This program is distributed in the hope that it will be useful, but
-# WITHOUT ANY WARRANTY; without even the implied warranties of
-# MERCHANTABILITY, SATISFACTORY QUALITY, or FITNESS FOR A PARTICULAR
-# PURPOSE.  See the GNU Lesser General Public License for more details.
-#
-# You should have received a copy of the GNU General Public License along
-# with this program.  If not, see <http://www.gnu.org/licenses/>.
-
-import sst.actions
-
-
-def create_ubuntu_sso_account(user):
-    _fill_new_account_form(user)
-    _click_continue()
-
-
-def _fill_new_account_form(user):
-    sst.actions.write_textfield('id_displayname', user.full_name)
-    sst.actions.write_textfield('id_email', user.email)
-    sst.actions.write_textfield('id_password', user.password)
-    sst.actions.write_textfield('id_passwordconfirm', user.password)
-
-
-def _click_continue():
-    sst.actions.click_button(_get_continue_button())
-
-
-def _get_continue_button():
-    return sst.actions.get_element(name='continue')

=== removed file 'u1testutils/sst/sso/pages/log_in.py'
--- u1testutils/sst/sso/pages/log_in.py	2013-01-16 15:16:50 +0000
+++ u1testutils/sst/sso/pages/log_in.py	1970-01-01 00:00:00 +0000
@@ -1,63 +0,0 @@
-# -*- coding: utf-8 -*-
-
-# Copyright 2012, 2013 Canonical Ltd.
-#
-# This program is free software: you can redistribute it and/or modify it
-# under the terms of the GNU Lesser General Public License version 3, as
-# published by the Free Software Foundation.
-#
-# This program is distributed in the hope that it will be useful, but
-# WITHOUT ANY WARRANTY; without even the implied warranties of
-# MERCHANTABILITY, SATISFACTORY QUALITY, or FITNESS FOR A PARTICULAR
-# PURPOSE.  See the GNU Lesser General Public License for more details.
-#
-# You should have received a copy of the GNU General Public License along
-# with this program.  If not, see <http://www.gnu.org/licenses/>.
-
-import sst.actions
-
-
-def assert_page():
-    _assert_page_heading1()
-    _assert_page_heading2()
-
-
-def _assert_page_heading1():
-    sst.actions.assert_element(tag='h1', text='Ubuntu Single Sign On')
-
-
-def _assert_page_heading2():
-    sst.actions.assert_element(tag='h2', text='Are you new?')
-
-
-def log_in(user=None):
-    if user is not None:
-        sst.actions.wait_for(sst.actions.assert_title, 'Log in')
-        _fill_log_in_form(user.email, user.password)
-        _click_continue_button()
-    else:
-        # If no user is passed as parameter, the user has already started
-        # session on the identity provider and it's not necessary to enter
-        # the credentials again.
-        pass
-
-
-def _fill_log_in_form(email, password):
-    sst.actions.write_textfield('id_email', email)
-    sst.actions.write_textfield('id_password', password)
-
-
-def _click_continue_button():
-    sst.actions.click_button(_get_continue_button())
-
-
-def _get_continue_button():
-    return sst.actions.get_element(css_class='btn', name='continue')
-
-
-def click_create_new_account():
-    sst.actions.click_link(_get_new_account_link())
-
-
-def _get_new_account_link():
-    return sst.actions.get_element(id='new-account-link')

=== removed file 'u1testutils/sst/sso/pages/registration_mail_sent.py'
--- u1testutils/sst/sso/pages/registration_mail_sent.py	2013-01-16 15:16:50 +0000
+++ u1testutils/sst/sso/pages/registration_mail_sent.py	1970-01-01 00:00:00 +0000
@@ -1,35 +0,0 @@
-# -*- coding: utf-8 -*-
-
-# Copyright 2012, 2013 Canonical Ltd.
-#
-# This program is free software: you can redistribute it and/or modify it
-# under the terms of the GNU Lesser General Public License version 3, as
-# published by the Free Software Foundation.
-#
-# This program is distributed in the hope that it will be useful, but
-# WITHOUT ANY WARRANTY; without even the implied warranties of
-# MERCHANTABILITY, SATISFACTORY QUALITY, or FITNESS FOR A PARTICULAR
-# PURPOSE.  See the GNU Lesser General Public License for more details.
-#
-# You should have received a copy of the GNU General Public License along
-# with this program.  If not, see <http://www.gnu.org/licenses/>.
-
-import sst.actions
-
-
-def confirm_email(confirmation_code):
-    sst.actions.write_textfield(
-        _get_confirmation_code_text_field(), confirmation_code)
-    _click_continue_button()
-
-
-def _get_confirmation_code_text_field():
-    return sst.actions.get_element(name='confirmation_code')
-
-
-def _click_continue_button():
-    sst.actions.click_button(_get_continue_button())
-
-
-def _get_continue_button():
-    return sst.actions.get_element(css_class='btn', text='Continue')

=== removed file 'u1testutils/sst/sso/pages/site_not_recognised.py'
--- u1testutils/sst/sso/pages/site_not_recognised.py	2013-01-16 15:16:50 +0000
+++ u1testutils/sst/sso/pages/site_not_recognised.py	1970-01-01 00:00:00 +0000
@@ -1,53 +0,0 @@
-# -*- coding: utf-8 -*-
-
-# Copyright 2012, 2013 Canonical Ltd.
-#
-# This program is free software: you can redistribute it and/or modify it
-# under the terms of the GNU Lesser General Public License version 3, as
-# published by the Free Software Foundation.
-#
-# This program is distributed in the hope that it will be useful, but
-# WITHOUT ANY WARRANTY; without even the implied warranties of
-# MERCHANTABILITY, SATISFACTORY QUALITY, or FITNESS FOR A PARTICULAR
-# PURPOSE.  See the GNU Lesser General Public License for more details.
-#
-# You should have received a copy of the GNU General Public License along
-# with this program.  If not, see <http://www.gnu.org/licenses/>.
-
-import sst.actions
-
-
-def is_site_not_recognized():
-    return _is_decide_form_present()
-
-
-def _is_decide_form_present():
-    return sst.actions.exists_element(name='decideform')
-
-
-def make_all_information_available_to_website():
-    information_checkboxes = _get_information_checkboxes()
-    for checkbox in information_checkboxes:
-        sst.actions.set_checkbox_value(checkbox, True)
-
-
-def _get_information_checkboxes():
-    return sst.actions.get_elements_by_css(
-        'form[name="decideform"] > .info-items > .list > li > '
-        'input[type="checkbox"]')
-
-
-def yes_sign_me_in():
-    sst.actions.click_button(_get_sign_me_in_button())
-
-
-def _get_sign_me_in_button():
-    return sst.actions.get_element(css_class='btn', name='yes')
-
-
-def cancel():
-    sst.actions.click_link(_get_cancel_link())
-
-
-def _get_cancel_link():
-    return sst.actions.get_element(tag='a', text='cancel')

=== removed file 'u1testutils/sst/sso/pages/subheader.py'
--- u1testutils/sst/sso/pages/subheader.py	2013-01-16 02:55:27 +0000
+++ u1testutils/sst/sso/pages/subheader.py	1970-01-01 00:00:00 +0000
@@ -1,9 +0,0 @@
-import sst.actions
-
-
-def is_logged_in():
-    return sst.actions.exists_element(id='logout-link')
-
-
-def log_out():
-    sst.actions.click_link('logout-link')