Skip to content

Conversation

@individual-it
Copy link
Member

@individual-it individual-it commented May 12, 2017

Description

UI tests for renaming files
and a lot of underlying page Object stuff

Related Issue

https://github.com/owncloud/QA/blob/master/Server/Test_Plan_WebDav_Interface_WebUI.md
This tests should cover:
3. Rename a file using special characters
5. Rename a file putting a name of a file which already exists
6. Try to rename a file using forbidden characters

Motivation and Context

make Test plan run automatically
https://github.com/owncloud/QA/blob/master/Server/Test_Plan_WebDav_Interface_WebUI.md

How Has This Been Tested?

run the tests in Chrome on local machine

Types of changes

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to change)

Checklist:

  • My code follows the code style of this project.
  • My change requires a change to the documentation.
  • I have updated the documentation accordingly.
  • I have read the CONTRIBUTING document.
  • I have added tests to cover my changes.
  • All new and existing tests passed.

@individual-it individual-it force-pushed the ui-tests-rename-feature branch from 0a2aa0b to e8e6456 Compare May 19, 2017 04:13
@individual-it individual-it changed the title UI tests for renaming feature [WIP] UI tests for renaming feature May 19, 2017
need to get this PR 244 MinkSelenium2Driver
It fixes an issue with typing into autocomplete boxes
This was never needed, only after updating mink-selenium2-driver
And then it would not give any usefull output why it failed only 
Expected to be on "http://localhost/owncloud-core/index.php/login" but found "" instead (SensioLabs\Behat\PageObjectExtension\PageObject\Exception\UnexpectedPageException)

Had to do network sniffing beween mink and saucelabs proxy to find out that it reported:
The requested combination of browser, version and OS is unsupported by the requested Selenium version and would lead to a test failure. Please set a different Selenium version, or set none to get the default, working Selenium version for this platform: firefox 47.0. on Windows 2008 with Selenium 2.31.0

What is also not true, because the correct version of selenium has to be 2.53.1 as per documentation: https://wiki.saucelabs.com/display/DOCS/Test+Configuration+Options#TestConfigurationOptions-SeleniumVersion
@individual-it individual-it force-pushed the ui-tests-rename-feature branch from 21cc1c9 to 93198f1 Compare May 19, 2017 09:20
this was needed after updating mink-selenium2-driver
before update FF returned:
<option value="default" selected="selected">Default</option>

after update:
<option value="default">Default</option>

if no option is selected the first one is selected, so getQuotaOfUser now finds the value of the select field and then returns the text that is associated with that value
double quotes are not valid characters in filenames in Windows, having them in the repro would result Windows user not beeing able to clone
Copy link
Contributor

@phil-davis phil-davis left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good now. The little bug that was found has been fixed in master - so testing attempts to rename to an existing file name now pass. Other hassles with Firefox versions and the latest Mink/Selenium versions are sorted. These tests should run reliably.

@individual-it individual-it changed the title [WIP] UI tests for renaming feature UI tests for renaming feature May 22, 2017
@phil-davis phil-davis merged commit 31b0c41 into master May 22, 2017
@DeepDiver1975 DeepDiver1975 added this to the 10.0.1 milestone May 22, 2017
@DeepDiver1975 DeepDiver1975 deleted the ui-tests-rename-feature branch May 22, 2017 10:10
@DeepDiver1975
Copy link
Member

@individual-it @phil-davis please hold your horses for now in merging further PRs - we are right in the middle of finishing the PR - we needs as little interruptions as possible - THX

@lock
Copy link

lock bot commented Aug 3, 2019

This thread has been automatically locked since there has not been any recent activity after it was closed. Please open a new issue for related bugs.

@lock lock bot locked as resolved and limited conversation to collaborators Aug 3, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants