Update tests to work with root-less travis-ci #961

Merged
merged 7 commits into from Feb 16, 2016

Conversation

Projects
None yet
2 participants
@kabel
Contributor

kabel commented Feb 15, 2016

Web server will be provided by a static http server from node and the
mod_include features will be handled in the PHP test.

Update tests to work with root-less travis-ci
Web server will be provided by a static http server from node and the
mod_include features will be handled in the PHP test.
@@ -12,40 +13,40 @@ if (file_exists(__DIR__ . '/../config.inc.php')) {
class AccessibilityTester {
protected $examples_directory = '';
protected $wrapper_html;
- public static $base_url = 'http://localhost/';
-
+ public static $base_url = 'http://localhost:8080/';

This comment has been minimized.

@mfairchild365

mfairchild365 Feb 16, 2016

Member

Is it possible to have the http-server use port 80?

@mfairchild365

mfairchild365 Feb 16, 2016

Member

Is it possible to have the http-server use port 80?

This comment has been minimized.

@kabel

kabel Feb 16, 2016

Contributor

No. Root is required to bind to ports < 1024.

@kabel

kabel Feb 16, 2016

Contributor

No. Root is required to bind to ports < 1024.

mfairchild365 added a commit that referenced this pull request Feb 16, 2016

Merge pull request #961 from unl/new-travis
Update tests to work with root-less travis-ci

@mfairchild365 mfairchild365 merged commit 69a72c2 into develop Feb 16, 2016

1 check passed

continuous-integration/travis-ci/pr The Travis CI build passed
Details

@kabel kabel deleted the new-travis branch Mar 1, 2016

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment