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

Fix parsing of test runner's output messages #8254

Merged
merged 6 commits into from
Jan 12, 2018
Merged

Fix parsing of test runner's output messages #8254

merged 6 commits into from
Jan 12, 2018

Conversation

svor
Copy link
Contributor

@svor svor commented Jan 11, 2018

Signed-off-by: Valeriy Svydenko vsvydenk@redhat.com

What does this PR do?

Checks for the end of test output messages.

What issues does this PR fix or reference?

#7338

Signed-off-by: Valeriy Svydenko <vsvydenk@redhat.com>
@svor svor added kind/bug Outline of a bug - must adhere to the bug report template. status/code-review This issue has a pull request posted for it and is awaiting code review completion by the community. labels Jan 11, 2018
@svor svor self-assigned this Jan 11, 2018
@svor svor requested a review from evidolob as a code owner January 11, 2018 13:21
@svor
Copy link
Contributor Author

svor commented Jan 11, 2018

ci-test

Signed-off-by: Valeriy Svydenko <vsvydenk@redhat.com>
import org.testng.annotations.Test;

@Listeners(MockitoTestNGListener.class)
public class LineSplitterTest {
Copy link
Contributor

Choose a reason for hiding this comment

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

Add javadoc pls

Copy link
Contributor Author

Choose a reason for hiding this comment

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

added

Signed-off-by: Valeriy Svydenko <vsvydenk@redhat.com>
@codenvy-ci
Copy link

ci-test build report:
Build details
Test report
selenium tests report data
docker image: eclipseche/che-server:8254
https://github.com/orgs/eclipse/teams/eclipse-che-qa please check this report.

Signed-off-by: Valeriy Svydenko <vsvydenk@redhat.com>
@svor svor requested a review from vparfonov as a code owner January 12, 2018 09:38
@svor
Copy link
Contributor Author

svor commented Jan 12, 2018

ci-build

Signed-off-by: Valeriy Svydenko <vsvydenk@redhat.com>
@svor svor merged commit 6f6780b into master Jan 12, 2018
@svor svor deleted the che-7338 branch January 12, 2018 11:01
@svor svor removed the status/code-review This issue has a pull request posted for it and is awaiting code review completion by the community. label Jan 12, 2018
@benoitf benoitf added this to the 6.0.0-M5 milestone Jan 12, 2018
@codenvy-ci
Copy link

Build # 4416 - FAILED

Please check console output at https://ci.codenvycorp.com/job/che-pullrequests-build/4416/ to view the results.

This pull request was closed.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
kind/bug Outline of a bug - must adhere to the bug report template.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

7 participants