diff --git a/.travis.yml b/.travis.yml index 514dad2..66ba481 100644 --- a/.travis.yml +++ b/.travis.yml @@ -14,7 +14,7 @@ install: before_script: - "export DISPLAY=:99.0" - "sh -e /etc/init.d/xvfb start" - - sleep 15 # give xvfb some time to start + - sleep 3 # give xvfb some time to start script: - sh bin/travis-run.sh after_success: coveralls diff --git a/ckanext/privatedatasets/plugin.py b/ckanext/privatedatasets/plugin.py index bfa430d..0ab370e 100755 --- a/ckanext/privatedatasets/plugin.py +++ b/ckanext/privatedatasets/plugin.py @@ -20,9 +20,10 @@ import ckan.lib.search as search import ckan.model as model import ckan.plugins as p +import ckan.plugins.toolkit as tk + from ckan.lib.plugins import DefaultPermissionLabels -import ckan.plugins.toolkit as tk import auth import actions import constants diff --git a/ckanext/privatedatasets/tests/test_selenium.py b/ckanext/privatedatasets/tests/test_selenium.py index 3339199..fe7001f 100644 --- a/ckanext/privatedatasets/tests/test_selenium.py +++ b/ckanext/privatedatasets/tests/test_selenium.py @@ -1,6 +1,6 @@ # -*- coding: utf-8 -*- -# Copyright (c) 2014 CoNWeT Lab., Universidad Politécnica de Madrid +# Copyright (c) 2014 - 2017 CoNWeT Lab., Universidad Politécnica de Madrid # This file is part of CKAN Private Dataset Extension. @@ -17,12 +17,15 @@ # You should have received a copy of the GNU Affero General Public License # along with CKAN Private Dataset Extension. If not, see . +from __future__ import unicode_literals + from nose_parameterized import parameterized from selenium import webdriver -from selenium.webdriver.support.ui import Select +from selenium.webdriver.common.by import By +from selenium.webdriver.support.ui import Select, WebDriverWait +from selenium.webdriver.support import expected_conditions as EC from subprocess import Popen - import ckan.lib.search.index as search_index import ckan.model as model import ckanext.privatedatasets.db as db @@ -82,7 +85,6 @@ class TestSelenium(unittest.TestCase): self.clearBBDD() self.driver.quit() - def assert_fields_disabled(self, fields): for field in fields: self.assertFalse(self.driver.find_element_by_id(field).is_enabled()) @@ -111,7 +113,11 @@ class TestSelenium(unittest.TestCase): def login(self, username, password): driver = self.driver driver.get(self.base_url) - driver.find_element_by_link_text('Log in').click() + login_btn = WebDriverWait(driver, 15).until( + EC.element_to_be_clickable((By.LINK_TEXT, 'Log in')) + ) + login_btn.click() + driver.find_element_by_id('field-login').clear() driver.find_element_by_id('field-login').send_keys(username) driver.find_element_by_id('field-password').clear()