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

Junit xml reporter grouping by classname #172

Merged
merged 6 commits into from Jan 18, 2014

Conversation

PaulTaykalo
Copy link
Contributor

Junit reporter will generate unique testcase "classname" attribute from original filename.

<testcase classname="Cedar" name="Some test" time="0.002539"/>
<testcase classname="Cedar" name="Yet another test" time="0.090917"/>
<testcase classname="Cedar" name="Third one" time="0.090917"/>

became

<testcase classname="MySpec" name="Some test" time="0.002539"/>
<testcase classname="MyAnotherSpec" name="Yet another test" time="0.090917"/>
<testcase classname="MyAnotherSpec" name="Third one" time="0.090917"/>

When it's not possible for some reason to resolve filename from Spec, it will fallback to default "Cedar" classname

Related to #171

@PaulTaykalo
Copy link
Contributor Author

That wasn't a good idea to merge with master branch

This prevent crashes when running tests on with CEDAR_SDK_RUNTIME_VERSION < 7.0
jeffh added a commit that referenced this pull request Jan 18, 2014
Junit xml reporter grouping by classname.

Reverted memory management semantics in CDRSpy.
@jeffh jeffh merged commit 61e81bf into cedarbdd:master Jan 18, 2014
@jeffh
Copy link
Contributor

jeffh commented Jan 18, 2014

Yeah, @idoru cherry-picked my memory leak PR which caused that crash. I was going to fix it up but thanks for doing it @PaulTaykalo.

@idoru
Copy link
Contributor

idoru commented Jan 20, 2014

Yeah, my bad, I'm still remembering how to write software properly again after a 1 month vacation. Thanks for fixing that up @PaulTaykalo.

@PaulTaykalo
Copy link
Contributor Author

np.
I just though that you wouldn't accept a build that fails on Travis :)
I didn't have a choice.

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

Successfully merging this pull request may close these issues.

None yet

3 participants