Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Functional Tests Updated #1448

Closed
wants to merge 1 commit into from
Closed

Functional Tests Updated #1448

wants to merge 1 commit into from

Conversation

devenbhooshan
Copy link
Contributor

Signed-off-by: Deven Bhooshan devenbhooshan@gmail.com
Tests have been updated in according to the changes like https://sourceforge.net/p/phpmyadmin/bugs/4684/
Now only few errors will come while running the functional tests.

Signed-off-by: Deven Bhooshan <devenbhooshan@gmail.com>
@devenbhooshan
Copy link
Contributor Author

why the selenium tests skipped ?

@lem9
Copy link
Contributor

lem9 commented Jan 4, 2015

We had timing issues with Selenium tests on Travis, see
https://wiki.phpmyadmin.net/pma/2014-05_Meeting_%28LinuxTag%29

@devenbhooshan
Copy link
Contributor Author

But tests skipped only when the following variables in the configuration file are not set. I didn't understand it.
1- TESTSUITE_SELENIUM_HOST
or
2- TESTSUITE_BROWSERSTACK_USER && TESTSUITE_BROWSERSTACK_KEY

@chirayuchiripal
Copy link
Contributor

@devenbhooshan The tests are getting skipped because BrowserStack credentials are secure environment variables which is not made available to pull requests due to security restrictions by Travis. Please see links below which explains it.

http://docs.travis-ci.com/user/pull-requests/
http://docs.travis-ci.com/user/environment-variables/

@devenbhooshan
Copy link
Contributor Author

So how should I run my changed tests on Travis?
I have run the tests on my machine.

@chirayuchiripal
Copy link
Contributor

You have to run it locally on your machine.

@devenbhooshan
Copy link
Contributor Author

I have run the tests on my machine already. I mean to say how are you guys going to moderate my changed tests? I want to contribute in the functional tests. So if I write some tests and I make a pull request for that , then how are you guys going to see whether the tests written by me are correct or not ?
Am I missing something ?

@chirayuchiripal
Copy link
Contributor

@devenbhooshan We will test your changes locally at our end.

@devenbhooshan
Copy link
Contributor Author

So I have changed some tests in this pr . Please review them locally .

@chirayuchiripal
Copy link
Contributor

$this->byId("buttonGo")->click();
$this->byXPath(
"//*[contains(text(), 'OK')]"
Copy link
Contributor

Choose a reason for hiding this comment

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

I think that a more specific XPath here will be good as this will select all tags containing text "OK" and will attempt to click it.

@devenbhooshan
Copy link
Contributor Author

ok. I will change the tests. Thanks for your comment :)

@chirayuchiripal
Copy link
Contributor

Closing this as of now.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants