Permalink
Browse files

PEP8 review

  • Loading branch information...
1 parent eab37bb commit 16ee3beff6f4dca801f2e5357bcb2d87c3266237 @bebef1987 bebef1987 committed Oct 3, 2012
Showing with 11 additions and 2 deletions.
  1. +1 −1 mozwebqa.cfg
  2. +1 −0 pages/about.py
  3. +5 −0 pages/edit_profile.py
  4. +1 −0 pages/invite.py
  5. +1 −0 pages/invite_success.py
  6. +1 −1 pages/page.py
  7. +1 −0 pages/search.py
View
@@ -1,3 +1,3 @@
[DEFAULT]
api = webdriver
-baseurl = https://mozillians-dev.allizom.org
+baseurl = https://mozillians-dev.allizom.org
View
@@ -9,6 +9,7 @@
from pages.base import Base
+
class About(Base):
_privacy_section_locator = (By.ID, 'privacy')
View
@@ -11,6 +11,7 @@
from pages.page import Page
from pages.profile import Profile
+
class EditProfile(Base):
_cancel_button_locator = (By.CSS_SELECTOR, "#edit_controls a")
@@ -43,6 +44,7 @@ def go_to_tab(self, tab_name):
self.selenium.find_element(*self._account_tab_locator).click()
return AccountTab(self.testsetup)
+
class ProfileTab(EditProfile):
_first_name_field_locator = (By.ID, 'id_first_name')
@@ -66,6 +68,7 @@ def set_bio(self, biography):
element = self.selenium.find_element(*self._bio_field_locator)
element.send_keys(biography)
+
class SkillsAndGroupsTab(EditProfile):
_groups_field_locator = (By.CSS_SELECTOR, '#id_groups + ul input')
@@ -79,6 +82,7 @@ def add_skill(self, skill_name):
element = self.selenium.send_keys(*self._skill_field_locator)
element.send_keys(skill_name)
+
class VouchesAndInvitesTab(EditProfile):
_voucher_name_locator = (By.CSS_SELECTOR, '#vouches .vouched')
@@ -87,6 +91,7 @@ class VouchesAndInvitesTab(EditProfile):
def vouched_by(self):
return self.selenium.find_element(*self._voucher_name_locator).text
+
class AccountTab(EditProfile):
_username_field_locator = (By.ID, 'id_username')
View
@@ -9,6 +9,7 @@
from pages.base import Base
+
class Invite(Base):
_recipient_field_locator = (By.ID, 'id_recipient')
View
@@ -9,6 +9,7 @@
from pages.base import Base
+
class InviteSuccess(Base):
_success_message_head_locator = (By.CSS_SELECTOR, '#main > h1')
View
@@ -62,4 +62,4 @@ def is_element_visible(self, *locator):
return False
def return_to_previous_page(self):
- self.selenium.back()
+ self.selenium.back()
View
@@ -9,6 +9,7 @@
from pages.base import Base
+
class Search(Base):
_result_locator = (By.CSS_SELECTOR, 'div.row > div.result')

0 comments on commit 16ee3be

Please sign in to comment.