Skip to content
This repository was archived by the owner on Jun 17, 2020. It is now read-only.

add testcase, mod pom.xml refs #93#94

Merged
TKosch merged 1 commit intoCloudCycle2:masterfrom
eiselems:93
Dec 16, 2014
Merged

add testcase, mod pom.xml refs #93#94
TKosch merged 1 commit intoCloudCycle2:masterfrom
eiselems:93

Conversation

@eiselems
Copy link
Copy Markdown
Member

added equals methode to Csar for testCase usage
added surefire plugin for test execution

@eiselems eiselems added this to the Week 2 milestone Dec 13, 2014
@Nik101010
Copy link
Copy Markdown
Contributor

Linking this PR to the issue #93

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

What has this file to do with the PR?

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

Needed a method to validate the results inside the testcase, as == comparison isn't enough. Hibernate generates different objects for the same logical object (csar1!=csar2 even if csar1.id==csar2.id).

@Nik101010
Copy link
Copy Markdown
Contributor

Seems like this PR contains changes that are not related to the issue. Can you please review @eiselems

@eiselems eiselems changed the title add testcase, mod pom.xml #ref 93 add testcase, mod pom.xml ref #93 Dec 14, 2014
@eiselems eiselems changed the title add testcase, mod pom.xml ref #93 add testcase, mod pom.xml refs #93 Dec 14, 2014
@eiselems
Copy link
Copy Markdown
Member Author

I think you're right, i will remove this from the PR by rebasing it, i will at-mention you when it's done and i think it can be merged.

Edit: @Nik101010 think it should be better now ;)

*add newLine to end of pom.xml
*add surefire plugin for testexecution
*add first testcase in test package
TKosch added a commit that referenced this pull request Dec 16, 2014
add testcase, mod pom.xml fixes #93
@TKosch TKosch merged commit 69d1d1b into CloudCycle2:master Dec 16, 2014
@eiselems eiselems deleted the 93 branch December 22, 2014 16:23
@eiselems eiselems restored the 93 branch December 22, 2014 16:23
@eiselems eiselems deleted the 93 branch December 22, 2014 16:23
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants