diff options
author | Ryan Ernst <ryan@iernst.net> | 2015-08-03 17:43:00 -0700 |
---|---|---|
committer | Ryan Ernst <ryan@iernst.net> | 2015-08-03 17:43:00 -0700 |
commit | 1e12d03252590bb44ae0cc3771c82c63730de212 (patch) | |
tree | a699635b3946b5541aeeaf9b0c0adcbeeeb62077 /core/src/test/java/org/elasticsearch/common/bytes | |
parent | 3c5b43a5bd426e4e1b041de31a0f7051081b3921 (diff) |
Tests: Rename base tests cases to use "TestCase" suffix
Most of the abstract base test classes we have were previously @Ignored.
However, there were also some other tests ignored. Having two ways to
quiet tests is confusing, and clearly it has caused some tests
to get lost in the fold.
This change moves all base test classes to use the "TestCase" suffix,
which is not picked up by the test class name pattern. It also removes
@Ignore from (almost) all tests, and adds it to forbidden apis.
And since we were renaming, I shorted base test class names to use
"ES" instead of "Elasticsearch". I type this a lot of types a day,
and I have heard others express a similar desire for a shorter name.
closes #10659
Diffstat (limited to 'core/src/test/java/org/elasticsearch/common/bytes')
-rw-r--r-- | core/src/test/java/org/elasticsearch/common/bytes/BytesReferenceTests.java | 4 | ||||
-rw-r--r-- | core/src/test/java/org/elasticsearch/common/bytes/PagedBytesReferenceTest.java | 4 |
2 files changed, 4 insertions, 4 deletions
diff --git a/core/src/test/java/org/elasticsearch/common/bytes/BytesReferenceTests.java b/core/src/test/java/org/elasticsearch/common/bytes/BytesReferenceTests.java index aaf2ef557f..b18e6f62d2 100644 --- a/core/src/test/java/org/elasticsearch/common/bytes/BytesReferenceTests.java +++ b/core/src/test/java/org/elasticsearch/common/bytes/BytesReferenceTests.java @@ -20,11 +20,11 @@ package org.elasticsearch.common.bytes; -import org.elasticsearch.test.ElasticsearchTestCase; +import org.elasticsearch.test.ESTestCase; import java.util.Arrays; -public class BytesReferenceTests extends ElasticsearchTestCase { +public class BytesReferenceTests extends ESTestCase { public void testEquals() { final int len = randomIntBetween(0, randomBoolean() ? 10: 100000); diff --git a/core/src/test/java/org/elasticsearch/common/bytes/PagedBytesReferenceTest.java b/core/src/test/java/org/elasticsearch/common/bytes/PagedBytesReferenceTest.java index aa0f411039..59a3d01255 100644 --- a/core/src/test/java/org/elasticsearch/common/bytes/PagedBytesReferenceTest.java +++ b/core/src/test/java/org/elasticsearch/common/bytes/PagedBytesReferenceTest.java @@ -27,7 +27,7 @@ import org.elasticsearch.common.io.stream.StreamInput; import org.elasticsearch.common.util.BigArrays; import org.elasticsearch.common.util.ByteArray; import org.elasticsearch.indices.breaker.NoneCircuitBreakerService; -import org.elasticsearch.test.ElasticsearchTestCase; +import org.elasticsearch.test.ESTestCase; import org.hamcrest.Matchers; import org.jboss.netty.buffer.ChannelBuffer; import org.junit.After; @@ -42,7 +42,7 @@ import java.nio.file.Path; import java.nio.file.StandardOpenOption; import java.util.Arrays; -public class PagedBytesReferenceTest extends ElasticsearchTestCase { +public class PagedBytesReferenceTest extends ESTestCase { private static final int PAGE_SIZE = BigArrays.BYTE_PAGE_SIZE; |