Return-Path: X-Original-To: archive-asf-public-internal@cust-asf2.ponee.io Delivered-To: archive-asf-public-internal@cust-asf2.ponee.io Received: from cust-asf.ponee.io (cust-asf.ponee.io [163.172.22.183]) by cust-asf2.ponee.io (Postfix) with ESMTP id 75A10200B80 for ; Wed, 31 Aug 2016 06:24:26 +0200 (CEST) Received: by cust-asf.ponee.io (Postfix) id 742A4160AD2; Wed, 31 Aug 2016 04:24:26 +0000 (UTC) Delivered-To: archive-asf-public@cust-asf.ponee.io Received: from mail.apache.org (hermes.apache.org [140.211.11.3]) by cust-asf.ponee.io (Postfix) with SMTP id 26650160ABA for ; Wed, 31 Aug 2016 06:24:24 +0200 (CEST) Received: (qmail 38629 invoked by uid 500); 31 Aug 2016 04:24:24 -0000 Mailing-List: contact commits-help@accumulo.apache.org; run by ezmlm Precedence: bulk List-Help: List-Unsubscribe: List-Post: List-Id: Reply-To: dev@accumulo.apache.org Delivered-To: mailing list commits@accumulo.apache.org Received: (qmail 38618 invoked by uid 99); 31 Aug 2016 04:24:24 -0000 Received: from git1-us-west.apache.org (HELO git1-us-west.apache.org) (140.211.11.23) by apache.org (qpsmtpd/0.29) with ESMTP; Wed, 31 Aug 2016 04:24:24 +0000 Received: by git1-us-west.apache.org (ASF Mail Server at git1-us-west.apache.org, from userid 33) id 311D9E024E; Wed, 31 Aug 2016 04:24:24 +0000 (UTC) Content-Type: text/plain; charset="us-ascii" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit From: elserj@apache.org To: commits@accumulo.apache.org Date: Wed, 31 Aug 2016 04:24:24 -0000 Message-Id: <10a98578cc244d1abefaa5e414bf60a2@git.apache.org> X-Mailer: ASF-Git Admin Mailer Subject: [01/10] accumulo git commit: ACCUMULO-4423 Annotate integration tests with categories archived-at: Wed, 31 Aug 2016 04:24:26 -0000 Repository: accumulo Updated Branches: refs/heads/1.7 2be85ade3 -> 661dac336 refs/heads/1.8 602799787 -> d28a3ee3e refs/heads/master 673fdb9ea -> 159560979 ACCUMULO-4423 Annotate integration tests with categories Differentiates tests which always use a minicluster and those which can use a minicluster or a standalone cluster. Out-of-the-box test invocation should not have changed. Includes updated documentation to TESTING.md as well. Closes apache/accumulo#144 Project: http://git-wip-us.apache.org/repos/asf/accumulo/repo Commit: http://git-wip-us.apache.org/repos/asf/accumulo/commit/661dac33 Tree: http://git-wip-us.apache.org/repos/asf/accumulo/tree/661dac33 Diff: http://git-wip-us.apache.org/repos/asf/accumulo/diff/661dac33 Branch: refs/heads/1.7 Commit: 661dac33648fb8bb311434720563c322611c1f12 Parents: 2be85ad Author: Josh Elser Authored: Tue Aug 30 16:23:48 2016 -0400 Committer: Josh Elser Committed: Tue Aug 30 23:27:09 2016 -0400 ---------------------------------------------------------------------- TESTING.md | 25 ++++++++++++-------- pom.xml | 24 ++++++++++++++++++- .../accumulo/harness/AccumuloClusterIT.java | 3 +++ .../accumulo/harness/SharedMiniClusterIT.java | 3 +++ .../org/apache/accumulo/test/NamespacesIT.java | 3 +++ .../test/categories/AnyClusterTest.java | 25 ++++++++++++++++++++ .../test/categories/MiniClusterOnlyTest.java | 24 +++++++++++++++++++ .../accumulo/test/categories/package-info.java | 21 ++++++++++++++++ .../accumulo/test/functional/ClassLoaderIT.java | 3 +++ .../test/functional/ConfigurableMacIT.java | 3 +++ .../accumulo/test/functional/KerberosIT.java | 3 +++ .../test/functional/KerberosProxyIT.java | 3 +++ .../test/functional/KerberosRenewalIT.java | 3 +++ .../accumulo/test/functional/PermissionsIT.java | 3 +++ .../accumulo/test/functional/TableIT.java | 3 +++ .../test/replication/KerberosReplicationIT.java | 3 +++ trace/pom.xml | 6 +++++ 17 files changed, 147 insertions(+), 11 deletions(-) ---------------------------------------------------------------------- http://git-wip-us.apache.org/repos/asf/accumulo/blob/661dac33/TESTING.md ---------------------------------------------------------------------- diff --git a/TESTING.md b/TESTING.md index de484ee..125110b 100644 --- a/TESTING.md +++ b/TESTING.md @@ -47,23 +47,27 @@ but are checking for regressions that were previously seen in the codebase. Thes resources, at least another gigabyte of memory over what Maven itself requires. As such, it's recommended to have at least 3-4GB of free memory and 10GB of free disk space. -## Accumulo for testing +## Test Categories -The primary reason these tests take so much longer than the unit tests is that most are using an Accumulo instance to -perform the test. It's a necessary evil; however, there are things we can do to improve this. +Accumulo uses JUnit Category annotations to categorize certain integration tests based on their runtime requirements. +Presently there are three different categories: -## MiniAccumuloCluster +### MiniAccumuloCluster (`MiniClusterOnlyTest`) -By default, these tests will use a MiniAccumuloCluster which is a multi-process "implementation" of Accumulo, managed -through Java interfaces. This MiniAccumuloCluster has the ability to use the local filesystem or Apache Hadoop's +These tests use MiniAccumuloCluster (MAC) which is a multi-process "implementation" of Accumulo, managed +through Java APIs. This MiniAccumuloCluster has the ability to use the local filesystem or Apache Hadoop's MiniDFSCluster, as well as starting one to many tablet servers. MiniAccumuloCluster tends to be a very useful tool in that it can automatically provide a workable instance that mimics how an actual deployment functions. The downside of using MiniAccumuloCluster is that a significant portion of each test is now devoted to starting and stopping the MiniAccumuloCluster. While this is a surefire way to isolate tests from interferring with one another, it -increases the actual runtime of the test by, on average, 10x. +increases the actual runtime of the test by, on average, 10x. Some times the tests require the use of MAC because the +test is being destructive or some special environment setup (e.g. Kerberos). -## Standalone Cluster +By default, these tests are run during the `integration-test` lifecycle phase using `mvn verify`. These tests can +also be run at the `test` lifecycle phase using `mvn package -Pminicluster-unit-tests`. + +### Standalone Cluster (`AnyClusterTest`) An alternative to the MiniAccumuloCluster for testing, a standalone Accumulo cluster can also be configured for use by most tests. This requires a manual step of building and deploying the Accumulo cluster by hand. The build can then be @@ -75,7 +79,9 @@ Use of a standalone cluster can be enabled using system properties on the Maven providing a Java properties file on the Maven command line. The use of a properties file is recommended since it is typically a fixed file per standalone cluster you want to run the tests against. -### Configuration +These tests will always run during the `integration-test` lifecycle phase using `mvn verify`. + +## Configuration for Standalone clusters The following properties can be used to configure a standalone cluster: @@ -128,4 +134,3 @@ at a time, for example the [Continuous Ingest][1] and [Randomwalk test][2] suite [3]: https://maven.apache.org/guides/introduction/introduction-to-the-lifecycle.html [4]: http://maven.apache.org/surefire/maven-surefire-plugin/ [5]: http://maven.apache.org/surefire/maven-failsafe-plugin/ - http://git-wip-us.apache.org/repos/asf/accumulo/blob/661dac33/pom.xml ---------------------------------------------------------------------- diff --git a/pom.xml b/pom.xml index 0f57f62..d6393d2 100644 --- a/pom.xml +++ b/pom.xml @@ -115,6 +115,10 @@ https://builds.apache.org/view/A-D/view/Accumulo/ + org.apache.accumulo.test.categories.AnyClusterTest + + ${accumulo.anyClusterTests},${accumulo.miniclusterTests} + org.apache.accumulo.test.categories.MiniClusterOnlyTest ${project.version} posix @@ -240,7 +244,7 @@ junit junit - 4.11 + 4.12 log4j @@ -1006,6 +1010,10 @@ integration-test verify + + ${accumulo.it.excludedGroups} + ${accumulo.it.groups} + @@ -1399,5 +1407,19 @@ + + only-minicluster-tests + + ${accumulo.anyClusterTests} + ${accumulo.miniclusterTests} + + + + standalone-capable-tests + + ${accumulo.miniclusterTests} + ${accumulo.anyClusterTests} + + http://git-wip-us.apache.org/repos/asf/accumulo/blob/661dac33/test/src/test/java/org/apache/accumulo/harness/AccumuloClusterIT.java ---------------------------------------------------------------------- diff --git a/test/src/test/java/org/apache/accumulo/harness/AccumuloClusterIT.java b/test/src/test/java/org/apache/accumulo/harness/AccumuloClusterIT.java index e2b35f4..436ceb5 100644 --- a/test/src/test/java/org/apache/accumulo/harness/AccumuloClusterIT.java +++ b/test/src/test/java/org/apache/accumulo/harness/AccumuloClusterIT.java @@ -43,6 +43,7 @@ import org.apache.accumulo.harness.conf.AccumuloMiniClusterConfiguration; import org.apache.accumulo.harness.conf.StandaloneAccumuloClusterConfiguration; import org.apache.accumulo.minicluster.impl.MiniAccumuloClusterImpl; import org.apache.accumulo.minicluster.impl.MiniAccumuloConfigImpl; +import org.apache.accumulo.test.categories.AnyClusterTest; import org.apache.hadoop.conf.Configuration; import org.apache.hadoop.fs.FileSystem; import org.apache.hadoop.fs.Path; @@ -52,12 +53,14 @@ import org.junit.AfterClass; import org.junit.Assume; import org.junit.Before; import org.junit.BeforeClass; +import org.junit.experimental.categories.Category; import org.slf4j.Logger; import org.slf4j.LoggerFactory; /** * General Integration-Test base class that provides access to an Accumulo instance for testing. This instance could be MAC or a standalone instance. */ +@Category(AnyClusterTest.class) public abstract class AccumuloClusterIT extends AccumuloIT implements MiniClusterConfigurationCallback, ClusterUsers { private static final Logger log = LoggerFactory.getLogger(AccumuloClusterIT.class); private static final String TRUE = Boolean.toString(true); http://git-wip-us.apache.org/repos/asf/accumulo/blob/661dac33/test/src/test/java/org/apache/accumulo/harness/SharedMiniClusterIT.java ---------------------------------------------------------------------- diff --git a/test/src/test/java/org/apache/accumulo/harness/SharedMiniClusterIT.java b/test/src/test/java/org/apache/accumulo/harness/SharedMiniClusterIT.java index f66a192..644055f 100644 --- a/test/src/test/java/org/apache/accumulo/harness/SharedMiniClusterIT.java +++ b/test/src/test/java/org/apache/accumulo/harness/SharedMiniClusterIT.java @@ -31,9 +31,11 @@ import org.apache.accumulo.core.client.security.tokens.PasswordToken; import org.apache.accumulo.core.conf.Property; import org.apache.accumulo.core.security.TablePermission; import org.apache.accumulo.minicluster.impl.MiniAccumuloClusterImpl; +import org.apache.accumulo.test.categories.MiniClusterOnlyTest; import org.apache.hadoop.conf.Configuration; import org.apache.hadoop.fs.CommonConfigurationKeysPublic; import org.apache.hadoop.security.UserGroupInformation; +import org.junit.experimental.categories.Category; import org.slf4j.Logger; import org.slf4j.LoggerFactory; @@ -48,6 +50,7 @@ import org.slf4j.LoggerFactory; * a method annotated with the {@link org.junit.BeforeClass} JUnit annotation and {@link #stopMiniCluster()} in a method annotated with the * {@link org.junit.AfterClass} JUnit annotation. */ +@Category(MiniClusterOnlyTest.class) public abstract class SharedMiniClusterIT extends AccumuloIT implements ClusterUsers { private static final Logger log = LoggerFactory.getLogger(SharedMiniClusterIT.class); public static final String TRUE = Boolean.toString(true); http://git-wip-us.apache.org/repos/asf/accumulo/blob/661dac33/test/src/test/java/org/apache/accumulo/test/NamespacesIT.java ---------------------------------------------------------------------- diff --git a/test/src/test/java/org/apache/accumulo/test/NamespacesIT.java b/test/src/test/java/org/apache/accumulo/test/NamespacesIT.java index aaa6a6e..6ec2127 100644 --- a/test/src/test/java/org/apache/accumulo/test/NamespacesIT.java +++ b/test/src/test/java/org/apache/accumulo/test/NamespacesIT.java @@ -77,14 +77,17 @@ import org.apache.accumulo.core.security.TablePermission; import org.apache.accumulo.core.util.UtilWaitThread; import org.apache.accumulo.examples.simple.constraints.NumericValueConstraint; import org.apache.accumulo.harness.AccumuloClusterIT; +import org.apache.accumulo.test.categories.MiniClusterOnlyTest; import org.apache.hadoop.io.Text; import org.junit.After; import org.junit.Assume; import org.junit.Before; import org.junit.Test; +import org.junit.experimental.categories.Category; // Testing default namespace configuration with inheritance requires altering the system state and restoring it back to normal // Punt on this for now and just let it use a minicluster. +@Category(MiniClusterOnlyTest.class) public class NamespacesIT extends AccumuloClusterIT { private Connector c; http://git-wip-us.apache.org/repos/asf/accumulo/blob/661dac33/test/src/test/java/org/apache/accumulo/test/categories/AnyClusterTest.java ---------------------------------------------------------------------- diff --git a/test/src/test/java/org/apache/accumulo/test/categories/AnyClusterTest.java b/test/src/test/java/org/apache/accumulo/test/categories/AnyClusterTest.java new file mode 100644 index 0000000..765057e --- /dev/null +++ b/test/src/test/java/org/apache/accumulo/test/categories/AnyClusterTest.java @@ -0,0 +1,25 @@ +/* + * Licensed to the Apache Software Foundation (ASF) under one or more + * contributor license agreements. See the NOTICE file distributed with + * this work for additional information regarding copyright ownership. + * The ASF licenses this file to you under the Apache License, Version 2.0 + * (the "License"); you may not use this file except in compliance with + * the License. You may obtain a copy of the License at + * + * http://www.apache.org/licenses/LICENSE-2.0 + * + * Unless required by applicable law or agreed to in writing, software + * distributed under the License is distributed on an "AS IS" BASIS, + * WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied. + * See the License for the specific language governing permissions and + * limitations under the License. + */ +package org.apache.accumulo.test.categories; + +/** + * Interface to be used with JUnit Category annotation to denote that the IntegrationTest can be used with any kind of cluster (a MiniAccumuloCluster or a + * StandaloneAccumuloCluster). + */ +public interface AnyClusterTest { + +} http://git-wip-us.apache.org/repos/asf/accumulo/blob/661dac33/test/src/test/java/org/apache/accumulo/test/categories/MiniClusterOnlyTest.java ---------------------------------------------------------------------- diff --git a/test/src/test/java/org/apache/accumulo/test/categories/MiniClusterOnlyTest.java b/test/src/test/java/org/apache/accumulo/test/categories/MiniClusterOnlyTest.java new file mode 100644 index 0000000..1a972ef --- /dev/null +++ b/test/src/test/java/org/apache/accumulo/test/categories/MiniClusterOnlyTest.java @@ -0,0 +1,24 @@ +/* + * Licensed to the Apache Software Foundation (ASF) under one or more + * contributor license agreements. See the NOTICE file distributed with + * this work for additional information regarding copyright ownership. + * The ASF licenses this file to you under the Apache License, Version 2.0 + * (the "License"); you may not use this file except in compliance with + * the License. You may obtain a copy of the License at + * + * http://www.apache.org/licenses/LICENSE-2.0 + * + * Unless required by applicable law or agreed to in writing, software + * distributed under the License is distributed on an "AS IS" BASIS, + * WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied. + * See the License for the specific language governing permissions and + * limitations under the License. + */ +package org.apache.accumulo.test.categories; + +/** + * Interface to be used with JUnit Category annotation to denote that the IntegrationTest requires the use of a MiniAccumuloCluster. + */ +public interface MiniClusterOnlyTest { + +} http://git-wip-us.apache.org/repos/asf/accumulo/blob/661dac33/test/src/test/java/org/apache/accumulo/test/categories/package-info.java ---------------------------------------------------------------------- diff --git a/test/src/test/java/org/apache/accumulo/test/categories/package-info.java b/test/src/test/java/org/apache/accumulo/test/categories/package-info.java new file mode 100644 index 0000000..e7071fc --- /dev/null +++ b/test/src/test/java/org/apache/accumulo/test/categories/package-info.java @@ -0,0 +1,21 @@ +/* + * Licensed to the Apache Software Foundation (ASF) under one or more + * contributor license agreements. See the NOTICE file distributed with + * this work for additional information regarding copyright ownership. + * The ASF licenses this file to you under the Apache License, Version 2.0 + * (the "License"); you may not use this file except in compliance with + * the License. You may obtain a copy of the License at + * + * http://www.apache.org/licenses/LICENSE-2.0 + * + * Unless required by applicable law or agreed to in writing, software + * distributed under the License is distributed on an "AS IS" BASIS, + * WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied. + * See the License for the specific language governing permissions and + * limitations under the License. + */ +/** + * JUnit categories for the various types of Accumulo integration tests. + */ +package org.apache.accumulo.test.categories; + http://git-wip-us.apache.org/repos/asf/accumulo/blob/661dac33/test/src/test/java/org/apache/accumulo/test/functional/ClassLoaderIT.java ---------------------------------------------------------------------- diff --git a/test/src/test/java/org/apache/accumulo/test/functional/ClassLoaderIT.java b/test/src/test/java/org/apache/accumulo/test/functional/ClassLoaderIT.java index 4b51bd2..d09e2a6 100644 --- a/test/src/test/java/org/apache/accumulo/test/functional/ClassLoaderIT.java +++ b/test/src/test/java/org/apache/accumulo/test/functional/ClassLoaderIT.java @@ -40,13 +40,16 @@ import org.apache.accumulo.core.util.CachedConfiguration; import org.apache.accumulo.core.util.UtilWaitThread; import org.apache.accumulo.harness.AccumuloClusterIT; import org.apache.accumulo.minicluster.impl.MiniAccumuloClusterImpl; +import org.apache.accumulo.test.categories.MiniClusterOnlyTest; import org.apache.hadoop.fs.FileSystem; import org.apache.hadoop.fs.Path; import org.hamcrest.CoreMatchers; import org.junit.Assume; import org.junit.Before; import org.junit.Test; +import org.junit.experimental.categories.Category; +@Category(MiniClusterOnlyTest.class) public class ClassLoaderIT extends AccumuloClusterIT { private static final long ZOOKEEPER_PROPAGATION_TIME = 10 * 1000; http://git-wip-us.apache.org/repos/asf/accumulo/blob/661dac33/test/src/test/java/org/apache/accumulo/test/functional/ConfigurableMacIT.java ---------------------------------------------------------------------- diff --git a/test/src/test/java/org/apache/accumulo/test/functional/ConfigurableMacIT.java b/test/src/test/java/org/apache/accumulo/test/functional/ConfigurableMacIT.java index 53eb8e4..6d04610 100644 --- a/test/src/test/java/org/apache/accumulo/test/functional/ConfigurableMacIT.java +++ b/test/src/test/java/org/apache/accumulo/test/functional/ConfigurableMacIT.java @@ -40,12 +40,14 @@ import org.apache.accumulo.minicluster.MiniAccumuloCluster; import org.apache.accumulo.minicluster.impl.MiniAccumuloClusterImpl; import org.apache.accumulo.minicluster.impl.MiniAccumuloConfigImpl; import org.apache.accumulo.minicluster.impl.ZooKeeperBindException; +import org.apache.accumulo.test.categories.MiniClusterOnlyTest; import org.apache.accumulo.test.util.CertUtils; import org.apache.commons.io.FileUtils; import org.apache.hadoop.conf.Configuration; import org.apache.zookeeper.KeeperException; import org.junit.After; import org.junit.Before; +import org.junit.experimental.categories.Category; import org.slf4j.Logger; import org.slf4j.LoggerFactory; @@ -53,6 +55,7 @@ import org.slf4j.LoggerFactory; * General Integration-Test base class that provides access to a {@link MiniAccumuloCluster} for testing. Tests using these typically do very disruptive things * to the instance, and require specific configuration. Most tests don't need this level of control and should extend {@link AccumuloClusterIT} instead. */ +@Category(MiniClusterOnlyTest.class) public class ConfigurableMacIT extends AccumuloIT { public static final Logger log = LoggerFactory.getLogger(ConfigurableMacIT.class); http://git-wip-us.apache.org/repos/asf/accumulo/blob/661dac33/test/src/test/java/org/apache/accumulo/test/functional/KerberosIT.java ---------------------------------------------------------------------- diff --git a/test/src/test/java/org/apache/accumulo/test/functional/KerberosIT.java b/test/src/test/java/org/apache/accumulo/test/functional/KerberosIT.java index 612718d..a3da827 100644 --- a/test/src/test/java/org/apache/accumulo/test/functional/KerberosIT.java +++ b/test/src/test/java/org/apache/accumulo/test/functional/KerberosIT.java @@ -68,6 +68,7 @@ import org.apache.accumulo.harness.TestingKdc; import org.apache.accumulo.minicluster.ServerType; import org.apache.accumulo.minicluster.impl.MiniAccumuloClusterImpl; import org.apache.accumulo.minicluster.impl.MiniAccumuloConfigImpl; +import org.apache.accumulo.test.categories.MiniClusterOnlyTest; import org.apache.hadoop.conf.Configuration; import org.apache.hadoop.fs.CommonConfigurationKeysPublic; import org.apache.hadoop.minikdc.MiniKdc; @@ -77,6 +78,7 @@ import org.junit.AfterClass; import org.junit.Before; import org.junit.BeforeClass; import org.junit.Test; +import org.junit.experimental.categories.Category; import org.slf4j.Logger; import org.slf4j.LoggerFactory; @@ -86,6 +88,7 @@ import com.google.common.collect.Sets; /** * MAC test which uses {@link MiniKdc} to simulate ta secure environment. Can be used as a sanity check for Kerberos/SASL testing. */ +@Category(MiniClusterOnlyTest.class) public class KerberosIT extends AccumuloIT { private static final Logger log = LoggerFactory.getLogger(KerberosIT.class); http://git-wip-us.apache.org/repos/asf/accumulo/blob/661dac33/test/src/test/java/org/apache/accumulo/test/functional/KerberosProxyIT.java ---------------------------------------------------------------------- diff --git a/test/src/test/java/org/apache/accumulo/test/functional/KerberosProxyIT.java b/test/src/test/java/org/apache/accumulo/test/functional/KerberosProxyIT.java index af6310c..2bef539 100644 --- a/test/src/test/java/org/apache/accumulo/test/functional/KerberosProxyIT.java +++ b/test/src/test/java/org/apache/accumulo/test/functional/KerberosProxyIT.java @@ -56,6 +56,7 @@ import org.apache.accumulo.proxy.thrift.ScanResult; import org.apache.accumulo.proxy.thrift.TimeType; import org.apache.accumulo.proxy.thrift.WriterOptions; import org.apache.accumulo.server.util.PortUtils; +import org.apache.accumulo.test.categories.MiniClusterOnlyTest; import org.apache.hadoop.conf.Configuration; import org.apache.hadoop.fs.CommonConfigurationKeysPublic; import org.apache.hadoop.security.UserGroupInformation; @@ -71,6 +72,7 @@ import org.junit.Before; import org.junit.BeforeClass; import org.junit.Rule; import org.junit.Test; +import org.junit.experimental.categories.Category; import org.junit.rules.ExpectedException; import org.slf4j.Logger; import org.slf4j.LoggerFactory; @@ -78,6 +80,7 @@ import org.slf4j.LoggerFactory; /** * Tests impersonation of clients by the proxy over SASL */ +@Category(MiniClusterOnlyTest.class) public class KerberosProxyIT extends AccumuloIT { private static final Logger log = LoggerFactory.getLogger(KerberosProxyIT.class); http://git-wip-us.apache.org/repos/asf/accumulo/blob/661dac33/test/src/test/java/org/apache/accumulo/test/functional/KerberosRenewalIT.java ---------------------------------------------------------------------- diff --git a/test/src/test/java/org/apache/accumulo/test/functional/KerberosRenewalIT.java b/test/src/test/java/org/apache/accumulo/test/functional/KerberosRenewalIT.java index 28c1dfc..07e0662 100644 --- a/test/src/test/java/org/apache/accumulo/test/functional/KerberosRenewalIT.java +++ b/test/src/test/java/org/apache/accumulo/test/functional/KerberosRenewalIT.java @@ -45,6 +45,7 @@ import org.apache.accumulo.harness.MiniClusterHarness; import org.apache.accumulo.harness.TestingKdc; import org.apache.accumulo.minicluster.impl.MiniAccumuloClusterImpl; import org.apache.accumulo.minicluster.impl.MiniAccumuloConfigImpl; +import org.apache.accumulo.test.categories.MiniClusterOnlyTest; import org.apache.hadoop.conf.Configuration; import org.apache.hadoop.fs.CommonConfigurationKeysPublic; import org.apache.hadoop.minikdc.MiniKdc; @@ -54,6 +55,7 @@ import org.junit.AfterClass; import org.junit.Before; import org.junit.BeforeClass; import org.junit.Test; +import org.junit.experimental.categories.Category; import org.slf4j.Logger; import org.slf4j.LoggerFactory; @@ -62,6 +64,7 @@ import com.google.common.collect.Iterables; /** * MAC test which uses {@link MiniKdc} to simulate ta secure environment. Can be used as a sanity check for Kerberos/SASL testing. */ +@Category(MiniClusterOnlyTest.class) public class KerberosRenewalIT extends AccumuloIT { private static final Logger log = LoggerFactory.getLogger(KerberosRenewalIT.class); http://git-wip-us.apache.org/repos/asf/accumulo/blob/661dac33/test/src/test/java/org/apache/accumulo/test/functional/PermissionsIT.java ---------------------------------------------------------------------- diff --git a/test/src/test/java/org/apache/accumulo/test/functional/PermissionsIT.java b/test/src/test/java/org/apache/accumulo/test/functional/PermissionsIT.java index 4aea354..6967a48 100644 --- a/test/src/test/java/org/apache/accumulo/test/functional/PermissionsIT.java +++ b/test/src/test/java/org/apache/accumulo/test/functional/PermissionsIT.java @@ -53,15 +53,18 @@ import org.apache.accumulo.core.security.Authorizations; import org.apache.accumulo.core.security.SystemPermission; import org.apache.accumulo.core.security.TablePermission; import org.apache.accumulo.harness.AccumuloClusterIT; +import org.apache.accumulo.test.categories.MiniClusterOnlyTest; import org.apache.hadoop.io.Text; import org.junit.Assume; import org.junit.Before; import org.junit.Test; +import org.junit.experimental.categories.Category; import org.slf4j.Logger; import org.slf4j.LoggerFactory; // This test verifies the default permissions so a clean instance must be used. A shared instance might // not be representative of a fresh installation. +@Category(MiniClusterOnlyTest.class) public class PermissionsIT extends AccumuloClusterIT { private static final Logger log = LoggerFactory.getLogger(PermissionsIT.class); http://git-wip-us.apache.org/repos/asf/accumulo/blob/661dac33/test/src/test/java/org/apache/accumulo/test/functional/TableIT.java ---------------------------------------------------------------------- diff --git a/test/src/test/java/org/apache/accumulo/test/functional/TableIT.java b/test/src/test/java/org/apache/accumulo/test/functional/TableIT.java index 3061b87..0bfdc00 100644 --- a/test/src/test/java/org/apache/accumulo/test/functional/TableIT.java +++ b/test/src/test/java/org/apache/accumulo/test/functional/TableIT.java @@ -39,15 +39,18 @@ import org.apache.accumulo.harness.AccumuloClusterIT; import org.apache.accumulo.minicluster.impl.MiniAccumuloClusterImpl; import org.apache.accumulo.test.TestIngest; import org.apache.accumulo.test.VerifyIngest; +import org.apache.accumulo.test.categories.MiniClusterOnlyTest; import org.apache.hadoop.fs.FileSystem; import org.apache.hadoop.fs.Path; import org.apache.hadoop.io.Text; import org.hamcrest.CoreMatchers; import org.junit.Assume; import org.junit.Test; +import org.junit.experimental.categories.Category; import com.google.common.collect.Iterators; +@Category(MiniClusterOnlyTest.class) public class TableIT extends AccumuloClusterIT { @Override http://git-wip-us.apache.org/repos/asf/accumulo/blob/661dac33/test/src/test/java/org/apache/accumulo/test/replication/KerberosReplicationIT.java ---------------------------------------------------------------------- diff --git a/test/src/test/java/org/apache/accumulo/test/replication/KerberosReplicationIT.java b/test/src/test/java/org/apache/accumulo/test/replication/KerberosReplicationIT.java index be9e320..933dfb8 100644 --- a/test/src/test/java/org/apache/accumulo/test/replication/KerberosReplicationIT.java +++ b/test/src/test/java/org/apache/accumulo/test/replication/KerberosReplicationIT.java @@ -41,6 +41,7 @@ import org.apache.accumulo.minicluster.impl.MiniAccumuloClusterImpl; import org.apache.accumulo.minicluster.impl.MiniAccumuloConfigImpl; import org.apache.accumulo.minicluster.impl.ProcessReference; import org.apache.accumulo.server.replication.ReplicaSystemFactory; +import org.apache.accumulo.test.categories.MiniClusterOnlyTest; import org.apache.accumulo.test.functional.KerberosIT; import org.apache.accumulo.tserver.TabletServer; import org.apache.accumulo.tserver.replication.AccumuloReplicaSystem; @@ -54,6 +55,7 @@ import org.junit.Assert; import org.junit.Before; import org.junit.BeforeClass; import org.junit.Test; +import org.junit.experimental.categories.Category; import org.slf4j.Logger; import org.slf4j.LoggerFactory; @@ -62,6 +64,7 @@ import com.google.common.collect.Iterators; /** * Ensure that replication occurs using keytabs instead of password (not to mention SASL) */ +@Category(MiniClusterOnlyTest.class) public class KerberosReplicationIT extends AccumuloIT { private static final Logger log = LoggerFactory.getLogger(KerberosIT.class); http://git-wip-us.apache.org/repos/asf/accumulo/blob/661dac33/trace/pom.xml ---------------------------------------------------------------------- diff --git a/trace/pom.xml b/trace/pom.xml index 2b79288..2d93a84 100644 --- a/trace/pom.xml +++ b/trace/pom.xml @@ -34,5 +34,11 @@ org.apache.htrace htrace-core + + + junit + junit + test +