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

GEODE-2901: Adding integration tests of session replication #584

Closed
wants to merge 4 commits into from

Conversation

DivineEnder
Copy link
Contributor

@DivineEnder DivineEnder commented Jun 15, 2017

Adding integration tests that test both session replications
frameworks shipped with geode. These tests use cargo to download
and launch different J2EE containers, so the tests can run
against many different containers.

Thank you for submitting a contribution to Apache Geode.

In order to streamline the review of the contribution we ask you
to ensure the following steps have been taken:

For all changes:

  • Is there a JIRA ticket associated with this PR? Is it referenced in the commit message?

  • Has your PR been rebased against the latest commit within the target branch (typically develop)?

  • Is your initial contribution a single, squashed commit?

  • Does gradlew build run cleanly?

  • Have you written or updated unit tests to verify your changes?

  • If adding new dependencies to the code, are these dependencies licensed in a way that is compatible for inclusion under ASF 2.0?

Note:

Please ensure that once the PR is submitted, you check travis-ci for build issues and
submit an update to your PR as soon as possible. If you need help, please send an
email to dev@geode.apache.org.

@DivineEnder
Copy link
Contributor Author

@upthewaterspout
Copy link
Contributor

+1 - Looks good to me!

@DivineEnder DivineEnder force-pushed the feature/GEODE-2901 branch 7 times, most recently from 1d048c9 to 1a568d5 Compare June 28, 2017 18:18
@DivineEnder
Copy link
Contributor Author

DivineEnder commented Jun 28, 2017

Single successful run on course when only running the Distributed tests in geode-assembly. Testing again with full concourse run. 🤞

Adding integration tests that test both session replications
frameworks shipped with geode. These tests use cargo to download
and launch different J2EE containers, so the tests can run
against many different containers.
Also added cargo_logs folder files to rat exclude
* Changed containers so that they installed in seperate directory across classes. Meaning that client server and peer to peer tests now use separate installs.
* Cache XML files now also create separate instances for each test.
Copy link
Contributor

@upthewaterspout upthewaterspout left a comment

Choose a reason for hiding this comment

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

+1 I'll merge this.

@asfgit asfgit closed this in aa68f04 Jun 30, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
2 participants