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

configuration comment in acceptance.suite.dist.yml #139

Open
peterpeter opened this issue Oct 31, 2015 · 4 comments
Open

configuration comment in acceptance.suite.dist.yml #139

peterpeter opened this issue Oct 31, 2015 · 4 comments

Comments

@peterpeter
Copy link
Contributor

url: 'http://localhost/tests/joomla-cms3' # the url that points to the joomla installation at /tests/system/joomla-cms

....is confusing, since there isn't such a path at this moment, at least before the first run (I guess)

[edit] Or is this related to the 'cms-path'-value in the robo file?

@therunnergit
Copy link
Contributor

You are totally right. The local path now can be configured to whatever you want so the comment is not correct. Note: of course the local path will never exist before you run the test.

@peterpeter can you suggest a better comment?

@peterpeter
Copy link
Contributor Author

I'm not sure. I've just started with weblinks, and I'm still discovering the components used within. But, if I'm not guessing wrong, these URL depends on what path is set in the RoboFile.dist.ini. If so, a reference to that file would be not bad in that comment. Like
the url that points to the joomla installation (see value for cmsPath in RoboFile.dist.ini)
If I'm wrong, I fear you have to ask someone who knows that ;-)

@chrisdavenport
Copy link
Contributor

Suggestion: "The URL that will point to the Joomla installation that the test will create"

Does that sound more accurate? @peterpeter @therunnergit

@puneet0191
Copy link
Member

"The URL points to Joomla Installation where the test should be executed" @chrisdavenport @zanardigit @peterpeter

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

No branches or pull requests

5 participants