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.

Sign up for GitHub

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

Jump to bottom

Possibly nest teamcity suites by launcher (prefix) #1187

Open
makepanic opened this issue Oct 23, 2017 · 2 comments · May be fixed by #1262
Open

Possibly nest teamcity suites by launcher (prefix) #1187

makepanic opened this issue Oct 23, 2017 · 2 comments · May be fixed by #1262
Labels
feature

Comments

@makepanic
Copy link
Contributor

Hi,
the teamcity spec allows to nest test suites:

##teamcity[testSuiteStarted name='suite.name']
##teamcity[testSuiteStarted name='nested.suite']
##teamcity[testStarted name='package_or_namespace.ClassName.TestName']
##teamcity[testFailed name='package_or_namespace.ClassName.TestName' message='The number should be 20000' details='junit.framework.AssertionFailedError: expected:<20000> but was:<10000>|n|r    at junit.framework.Assert.fail(Assert.java:47)|n|r    at junit.framework.Assert.failNotEquals(Assert.java:280)|n|r...']
##teamcity[testFinished name='package_or_namespace.ClassName.TestName']
##teamcity[testSuiteFinished name='nested.suite']
##teamcity[testSuiteFinished name='suite.name']

The current implementation only emits a flat list of test results.
I'm unsure if we could nest the output because this requires the reporter to wait for all tests inside a launcher to be finished.

Nested output allows an easier navigation through test results. See for example:

20171021-121058

Which is nested by grouping the results by their first /(.*) - / match.
@makepanic makepanic changed the title Possibly nest teamcity suites by launcher Possibly nest teamcity suites by launcher (prefix) Oct 23, 2017
@makepanic makepanic mentioned this issue Oct 24, 2017
Merged
makepanic added a commit to makepanic/testem that referenced this issue Oct 24, 2017
@makepanic
fix(teamcity_reporter): nest tests by prefix
aa84fb0 
fixes testem#1187
@stefanpenner
Copy link
Contributor

Nested output allows an easier navigation through test results. See for example:

that does look better. Any interest in pursuing this?

@makepanic
Copy link
Contributor Author

Sure.
I'll write some tests and create a PR.

makepanic added a commit to makepanic/testem that referenced this issue Jun 4, 2018
@makepanic
fix(teamcity_reporter): nest tests by prefix
619d620 
fixes testem#1187
@makepanic makepanic linked a pull request Jun 4, 2018 that will close this issue
Open
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
feature
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

fix(teamcity_reporter): nest tests by prefix makepanic/testem
2 participants
@stefanpenner @makepanic