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

[tests] switch lighttpd check to AgentCheckTest. #1473

Merged
merged 1 commit into from
Mar 26, 2015

Conversation

hkaj
Copy link
Member

@hkaj hkaj commented Mar 25, 2015

No description provided.

@LeoCavaille
Copy link
Member

You removed the test with the instance ?auto URL, was it just not useful or redundant?

@LeoCavaille LeoCavaille added this to the 5.3.0 milestone Mar 26, 2015
@hkaj
Copy link
Member Author

hkaj commented Mar 26, 2015

@LeoCavaille it's redundant. The check adds the ?auto automatically in the first config after trying without it and failing to parse the result.

@hkaj
Copy link
Member Author

hkaj commented Mar 26, 2015

Also the 3 build failures in Travis are due to timeouts.

@LeoCavaille
Copy link
Member

nice 👍

LeoCavaille added a commit that referenced this pull request Mar 26, 2015
[tests] switch lighttpd check to AgentCheckTest.
@LeoCavaille LeoCavaille merged commit b67bb4f into master Mar 26, 2015
@LeoCavaille LeoCavaille deleted the haissam/lighttpd-improve-tests branch March 26, 2015 16:26
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants