[ https://issues.apache.org/jira/browse/GEODE-308?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15310793#comment-15310793 ]
ASF GitHub Bot commented on GEODE-308: -------------------------------------- GitHub user kjduling opened a pull request: https://github.com/apache/incubator-geode/pull/150 Feature/geode 308 Partially addresses GEODE-308, removing several Hydra classes. You can merge this pull request into a Git repository by running: $ git pull https://github.com/kjduling/incubator-geode feature/GEODE-308 Alternatively you can review and apply these changes as the patch at: https://github.com/apache/incubator-geode/pull/150.patch To close this pull request, make a commit to your master/trunk branch with (at least) the following in the commit message: This closes #150 ---- commit b3613606edf154315f98dbc3f9f0bfb9e2fb9a73 Author: Kevin J. Duling <kdul...@pivotal.io> Date: 2016-05-31T22:39:16Z GEODE-308: Separate hydra from dunit and junit tests in gemfire-core Removed hydra classes. commit 0caa1c8cef35ee8ac24dae604e7cfae422db361f Author: Kevin J. Duling <kdul...@pivotal.io> Date: 2016-05-31T22:58:21Z Merge branch 'develop' into feature/GEODE-308 commit eb37b1cac9d611d77374949065364f79861f62d1 Author: Kevin J. Duling <kdul...@pivotal.io> Date: 2016-06-01T17:14:47Z Merge branch 'develop' into feature/GEODE-308 ---- > Separate hydra from dunit and junit tests in gemfire-core > --------------------------------------------------------- > > Key: GEODE-308 > URL: https://issues.apache.org/jira/browse/GEODE-308 > Project: Geode > Issue Type: Bug > Components: tests > Affects Versions: 1.0.0-incubating > Reporter: Kirk Lund > Assignee: Kevin Duling > Labels: hydra > > Usage of Hydra needs to be removed from dunit and junit tests in gemfire-core > and any other project. > The following hydra classes in gemfire-core should be removed (and replaced > if needed by dunit/junit tests): > src/test/java/hydra/GsRandom.java > src/test/java/hydra/HydraRuntimeException.java > src/test/java/hydra/Log.java > src/test/java/hydra/LogVersionHelper.java > src/test/java/hydra/MethExecutor.java > src/test/java/hydra/MethExecutorResult.java > src/test/java/hydra/SchedulingOrder.java > src/test/java/hydra/log/AnyLogWriter.java > src/test/java/hydra/log/CircularOutputStream.java > The following are also not com.gemstone packages and should be removed if > they're specific to Hydra (or Hydra tests) or repackaged if they're actually > used in dunit/junit tests: > src/test/java/batterytest/greplogs/ExpectedStrings.java > src/test/java/batterytest/greplogs/LogConsumer.java > src/test/java/cacheRunner/Portfolio.java > src/test/java/cacheRunner/Position.java > src/test/java/parReg/query/unittest/NewPortfolio.java > src/test/java/parReg/query/unittest/Position.java > src/test/java/perffmwk/Formatter.java > src/test/java/templates/security/DummyAuthenticator.java > src/test/java/templates/security/DummyAuthorization.java > src/test/java/templates/security/FunctionSecurityPrmsHolder.java > src/test/java/templates/security/LdapUserAuthenticator.java > src/test/java/templates/security/PKCSAuthenticator.java > src/test/java/templates/security/PKCSAuthInt.java > src/test/java/templates/security/PKCSPrincipal.java > src/test/java/templates/security/UsernamePrincipal.java > src/test/java/templates.security/UserPasswordAuthInit.java > src/test/java/templates.security/XmlAuthorization.java > src/test/java/templates.security/XmlErrorHandler.java > src/test/java/util/TestException.java > The following are Hydra-related resources in src/test/resources that also > need to be removed: > src/test/resources/jta/cachejta.xml > src/test/resources/ssl/trusted.keystore > src/test/resources/templates/security/authz5_5.dtd > src/test/resources/templates/security/authz6_0.dtd -- This message was sent by Atlassian JIRA (v6.3.4#6332)