From 7b91e4cc18e77fa5862864fcc1150fd1eb86a01a Mon Sep 17 00:00:00 2001 From: Caleb Rackliffe Date: Thu, 10 Mar 2022 16:19:45 -0600 Subject: [PATCH] Remove all usages of junit.framework and ban them via Checkstyle patch by Caleb Rackliffe; reviewed by Marcus Eriksson for CASSANDRA-17316 Co-authored-by: Marcus Eriksson --- CHANGES.txt | 1 + build.xml | 17 +++++- checkstyle.xml | 2 +- checkstyle_test.xml | 61 +++++++++++++++++++ .../CassandraBriefJUnitResultFormatter.java | 4 +- .../CassandraXMLJUnitResultFormatter.java | 4 +- .../cql3/NonNativeTimestampTest.java | 4 +- .../apache/cassandra/cql3/OutOfSpaceTest.java | 4 +- .../org/apache/cassandra/cql3/PagingTest.java | 2 +- .../validation/entities/TimestampTest.java | 2 +- .../miscellaneous/OverflowTest.java | 10 +-- .../miscellaneous/TombstonesTest.java | 6 +- .../operations/CompactStorageTest.java | 2 +- .../SelectOrderedPartitionerTest.java | 8 +-- .../cassandra/db/ColumnFamilyStoreTest.java | 13 ++-- .../cassandra/db/DeletePartitionTest.java | 4 +- .../cassandra/db/RowIndexEntryTest.java | 4 +- .../db/compaction/LeveledGenerationsTest.java | 2 +- .../cassandra/db/lifecycle/HelpersTest.java | 4 +- .../hints/ChecksummedDataInputTest.java | 22 +++---- .../org/apache/cassandra/hints/HintTest.java | 3 +- .../cassandra/hints/HintsBufferPoolTest.java | 4 +- .../cassandra/hints/HintsBufferTest.java | 6 +- .../cassandra/hints/HintsCatalogTest.java | 7 ++- .../cassandra/hints/HintsDescriptorTest.java | 6 +- .../cassandra/hints/HintsReaderTest.java | 4 +- .../cassandra/hints/HintsStoreTest.java | 6 +- .../apache/cassandra/hints/HintsTestUtil.java | 4 +- .../io/sstable/SSTableWriterTest.java | 11 ++-- .../io/sstable/format/VersionAndTypeTest.java | 2 +- .../cassandra/io/util/MmappedRegionsTest.java | 5 +- .../cassandra/metrics/LatencyMetricsTest.java | 4 +- .../cassandra/net/MockMessagingSpy.java | 4 +- .../asymmetric/DifferenceHolderTest.java | 2 +- .../repair/asymmetric/RangeMapTest.java | 2 +- .../repair/asymmetric/ReduceHelperTest.java | 2 +- .../asymmetric/StreamFromOptionsTest.java | 2 +- .../cassandra/service/ClientWarningsTest.java | 2 +- .../service/ProtocolBetaVersionTest.java | 6 +- 39 files changed, 174 insertions(+), 84 deletions(-) create mode 100644 checkstyle_test.xml diff --git a/CHANGES.txt b/CHANGES.txt index f22e088193..e99d8d8396 100644 --- a/CHANGES.txt +++ b/CHANGES.txt @@ -1,4 +1,5 @@ 4.1 + * Remove all usages of junit.framework and ban them via Checkstyle (CASSANDRA-17316) * Add guardrails for read/write consistency levels (CASSANDRA-17188) * Add guardrail for SELECT IN terms and their cartesian product (CASSANDRA-17187) * remove unused imports in cqlsh.py and cqlshlib (CASSANDRA-17413) diff --git a/build.xml b/build.xml index 30097acae7..4cec4d70a7 100644 --- a/build.xml +++ b/build.xml @@ -1369,7 +1369,7 @@ - @@ -2253,6 +2253,21 @@ + + + + + + + + + + + + + - + diff --git a/checkstyle_test.xml b/checkstyle_test.xml new file mode 100644 index 0000000000..68e5e87c12 --- /dev/null +++ b/checkstyle_test.xml @@ -0,0 +1,61 @@ + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + diff --git a/test/unit/org/apache/cassandra/CassandraBriefJUnitResultFormatter.java b/test/unit/org/apache/cassandra/CassandraBriefJUnitResultFormatter.java index 88dbc52941..2befb5c631 100644 --- a/test/unit/org/apache/cassandra/CassandraBriefJUnitResultFormatter.java +++ b/test/unit/org/apache/cassandra/CassandraBriefJUnitResultFormatter.java @@ -24,8 +24,8 @@ import java.io.OutputStream; import java.io.StringWriter; import java.text.NumberFormat; -import junit.framework.AssertionFailedError; -import junit.framework.Test; +import junit.framework.AssertionFailedError; // checkstyle: permit this import +import junit.framework.Test; // checkstyle: permit this import import org.apache.tools.ant.BuildException; import org.apache.tools.ant.taskdefs.optional.junit.IgnoredTestListener; diff --git a/test/unit/org/apache/cassandra/CassandraXMLJUnitResultFormatter.java b/test/unit/org/apache/cassandra/CassandraXMLJUnitResultFormatter.java index df0ce7f327..de8fb4e5d5 100644 --- a/test/unit/org/apache/cassandra/CassandraXMLJUnitResultFormatter.java +++ b/test/unit/org/apache/cassandra/CassandraXMLJUnitResultFormatter.java @@ -33,8 +33,8 @@ import java.util.Properties; import javax.xml.parsers.DocumentBuilder; import javax.xml.parsers.DocumentBuilderFactory; -import junit.framework.AssertionFailedError; -import junit.framework.Test; +import junit.framework.AssertionFailedError; // checkstyle: permit this import +import junit.framework.Test; // checkstyle: permit this import import org.apache.tools.ant.BuildException; import org.apache.tools.ant.taskdefs.optional.junit.IgnoredTestListener; diff --git a/test/unit/org/apache/cassandra/cql3/NonNativeTimestampTest.java b/test/unit/org/apache/cassandra/cql3/NonNativeTimestampTest.java index 7ba9889613..975f3cf967 100644 --- a/test/unit/org/apache/cassandra/cql3/NonNativeTimestampTest.java +++ b/test/unit/org/apache/cassandra/cql3/NonNativeTimestampTest.java @@ -19,8 +19,8 @@ package org.apache.cassandra.cql3; import org.junit.Test; -import static junit.framework.Assert.assertEquals; -import static junit.framework.Assert.assertFalse; +import static org.junit.Assert.assertEquals; +import static org.junit.Assert.assertFalse; import static org.junit.Assert.assertTrue; public class NonNativeTimestampTest extends CQLTester diff --git a/test/unit/org/apache/cassandra/cql3/OutOfSpaceTest.java b/test/unit/org/apache/cassandra/cql3/OutOfSpaceTest.java index b4fe0f5fd2..1ebd9ec96e 100644 --- a/test/unit/org/apache/cassandra/cql3/OutOfSpaceTest.java +++ b/test/unit/org/apache/cassandra/cql3/OutOfSpaceTest.java @@ -17,8 +17,6 @@ */ package org.apache.cassandra.cql3; -import static junit.framework.Assert.fail; - import java.io.Closeable; import java.util.concurrent.ExecutionException; @@ -37,6 +35,8 @@ import org.apache.cassandra.schema.TableId; import org.apache.cassandra.utils.JVMStabilityInspector; import org.apache.cassandra.utils.KillerForTests; +import static org.junit.Assert.fail; + /** * Test that TombstoneOverwhelmingException gets thrown when it should be and doesn't when it shouldn't be. */ diff --git a/test/unit/org/apache/cassandra/cql3/PagingTest.java b/test/unit/org/apache/cassandra/cql3/PagingTest.java index 0b8028de3e..9a95e03210 100644 --- a/test/unit/org/apache/cassandra/cql3/PagingTest.java +++ b/test/unit/org/apache/cassandra/cql3/PagingTest.java @@ -37,7 +37,7 @@ import org.apache.cassandra.service.EmbeddedCassandraService; import org.apache.cassandra.service.StorageService; import org.apache.cassandra.utils.FBUtilities; -import static junit.framework.Assert.assertFalse; +import static org.junit.Assert.assertFalse; import static org.junit.Assert.assertEquals; diff --git a/test/unit/org/apache/cassandra/cql3/validation/entities/TimestampTest.java b/test/unit/org/apache/cassandra/cql3/validation/entities/TimestampTest.java index 63cd2b7445..13090a6b62 100644 --- a/test/unit/org/apache/cassandra/cql3/validation/entities/TimestampTest.java +++ b/test/unit/org/apache/cassandra/cql3/validation/entities/TimestampTest.java @@ -22,7 +22,7 @@ import org.junit.Test; import org.junit.Assert; import org.apache.cassandra.cql3.CQLTester; -import static junit.framework.Assert.assertNull; +import static org.junit.Assert.assertNull; import static org.junit.Assert.assertTrue; public class TimestampTest extends CQLTester diff --git a/test/unit/org/apache/cassandra/cql3/validation/miscellaneous/OverflowTest.java b/test/unit/org/apache/cassandra/cql3/validation/miscellaneous/OverflowTest.java index 9fc50531aa..0d9e043935 100644 --- a/test/unit/org/apache/cassandra/cql3/validation/miscellaneous/OverflowTest.java +++ b/test/unit/org/apache/cassandra/cql3/validation/miscellaneous/OverflowTest.java @@ -21,15 +21,15 @@ import java.math.BigInteger; import org.junit.Test; -import static junit.framework.Assert.assertNull; -import static org.junit.Assert.assertArrayEquals; -import static org.junit.Assert.assertEquals; -import static org.junit.Assert.assertTrue; - import org.apache.cassandra.cql3.CQLTester; import org.apache.cassandra.exceptions.SyntaxException; import org.apache.cassandra.utils.ByteBufferUtil; +import static org.junit.Assert.assertNull; +import static org.junit.Assert.assertArrayEquals; +import static org.junit.Assert.assertEquals; +import static org.junit.Assert.assertTrue; + /** * Any tests that do not fit in any other category, * migrated from python dtests, CASSANDRA-9160 diff --git a/test/unit/org/apache/cassandra/cql3/validation/miscellaneous/TombstonesTest.java b/test/unit/org/apache/cassandra/cql3/validation/miscellaneous/TombstonesTest.java index 85048ae2c1..dc7cccfcb4 100644 --- a/test/unit/org/apache/cassandra/cql3/validation/miscellaneous/TombstonesTest.java +++ b/test/unit/org/apache/cassandra/cql3/validation/miscellaneous/TombstonesTest.java @@ -31,9 +31,9 @@ import org.apache.cassandra.db.ColumnFamilyStore; import org.apache.cassandra.db.Keyspace; import org.apache.cassandra.db.filter.TombstoneOverwhelmingException; -import static junit.framework.Assert.assertEquals; -import static junit.framework.Assert.assertTrue; -import static junit.framework.Assert.fail; +import static org.junit.Assert.assertEquals; +import static org.junit.Assert.assertTrue; +import static org.junit.Assert.fail; /** * Test that TombstoneOverwhelmingException gets thrown when it should be and doesn't when it shouldn't be. diff --git a/test/unit/org/apache/cassandra/cql3/validation/operations/CompactStorageTest.java b/test/unit/org/apache/cassandra/cql3/validation/operations/CompactStorageTest.java index 212e698aad..4939895184 100644 --- a/test/unit/org/apache/cassandra/cql3/validation/operations/CompactStorageTest.java +++ b/test/unit/org/apache/cassandra/cql3/validation/operations/CompactStorageTest.java @@ -45,10 +45,10 @@ import org.apache.cassandra.schema.TableMetadata; import org.apache.cassandra.utils.ByteBufferUtil; import org.apache.cassandra.utils.FBUtilities; -import static junit.framework.TestCase.assertTrue; import static org.apache.cassandra.utils.ByteBufferUtil.EMPTY_BYTE_BUFFER; import static org.apache.cassandra.utils.ByteBufferUtil.bytes; import static org.junit.Assert.assertArrayEquals; +import static org.junit.Assert.assertTrue; import static org.junit.Assert.fail; import static org.reflections.util.Utils.isEmpty; diff --git a/test/unit/org/apache/cassandra/cql3/validation/operations/SelectOrderedPartitionerTest.java b/test/unit/org/apache/cassandra/cql3/validation/operations/SelectOrderedPartitionerTest.java index a14a2a421d..fbac1ffaf7 100644 --- a/test/unit/org/apache/cassandra/cql3/validation/operations/SelectOrderedPartitionerTest.java +++ b/test/unit/org/apache/cassandra/cql3/validation/operations/SelectOrderedPartitionerTest.java @@ -25,14 +25,14 @@ import java.util.Arrays; import org.junit.BeforeClass; import org.junit.Test; -import static junit.framework.Assert.assertNull; -import static org.junit.Assert.assertEquals; - import org.apache.cassandra.config.DatabaseDescriptor; import org.apache.cassandra.cql3.CQLTester; import org.apache.cassandra.cql3.restrictions.StatementRestrictions; import org.apache.cassandra.dht.ByteOrderedPartitioner; +import static org.junit.Assert.assertEquals; +import static org.junit.Assert.assertNull; + /** * SELECT statement tests that require a ByteOrderedPartitioner */ @@ -368,7 +368,7 @@ public class SelectOrderedPartitionerTest extends CQLTester Object[][] rows = getRows(execute("SELECT v2 FROM %s")); assertEquals(0, rows[0][0]); - assertEquals(null, rows[1][0]); + assertNull(rows[1][0]); assertEquals(2, rows[2][0]); rows = getRows(execute("SELECT v2 FROM %s WHERE k = 1")); diff --git a/test/unit/org/apache/cassandra/db/ColumnFamilyStoreTest.java b/test/unit/org/apache/cassandra/db/ColumnFamilyStoreTest.java index f6efe6ade1..e7c7e225f3 100644 --- a/test/unit/org/apache/cassandra/db/ColumnFamilyStoreTest.java +++ b/test/unit/org/apache/cassandra/db/ColumnFamilyStoreTest.java @@ -26,7 +26,6 @@ import java.util.*; import org.junit.Assert; import org.junit.Before; -import org.junit.Assume; import org.junit.BeforeClass; import org.junit.Test; @@ -35,11 +34,6 @@ import org.apache.cassandra.schema.SchemaConstants; import org.apache.cassandra.service.snapshot.SnapshotManifest; import org.apache.cassandra.service.snapshot.TableSnapshot; -import static org.assertj.core.api.Assertions.assertThat; -import static org.junit.Assert.assertEquals; -import static org.junit.Assert.assertFalse; -import static org.junit.Assert.assertTrue; - import com.google.common.collect.Iterators; import org.apache.cassandra.*; import org.apache.cassandra.cql3.Operator; @@ -58,7 +52,12 @@ import org.apache.cassandra.schema.KeyspaceParams; import org.apache.cassandra.utils.ByteBufferUtil; import org.apache.cassandra.utils.FBUtilities; import org.apache.cassandra.utils.WrappedRunnable; -import static junit.framework.Assert.assertNotNull; + +import static org.assertj.core.api.Assertions.assertThat; +import static org.junit.Assert.assertEquals; +import static org.junit.Assert.assertFalse; +import static org.junit.Assert.assertTrue; +import static org.junit.Assert.assertNotNull; public class ColumnFamilyStoreTest { diff --git a/test/unit/org/apache/cassandra/db/DeletePartitionTest.java b/test/unit/org/apache/cassandra/db/DeletePartitionTest.java index 6ed43f7262..d2f599d836 100644 --- a/test/unit/org/apache/cassandra/db/DeletePartitionTest.java +++ b/test/unit/org/apache/cassandra/db/DeletePartitionTest.java @@ -30,8 +30,8 @@ import org.apache.cassandra.schema.KeyspaceParams; import org.apache.cassandra.utils.ByteBufferUtil; import org.apache.cassandra.utils.FBUtilities; -import static junit.framework.Assert.assertFalse; -import static junit.framework.Assert.assertTrue; +import static org.junit.Assert.assertFalse; +import static org.junit.Assert.assertTrue; public class DeletePartitionTest { diff --git a/test/unit/org/apache/cassandra/db/RowIndexEntryTest.java b/test/unit/org/apache/cassandra/db/RowIndexEntryTest.java index d1b21b40eb..a2fb57dbc6 100644 --- a/test/unit/org/apache/cassandra/db/RowIndexEntryTest.java +++ b/test/unit/org/apache/cassandra/db/RowIndexEntryTest.java @@ -64,8 +64,8 @@ import org.apache.cassandra.utils.FBUtilities; import org.apache.cassandra.utils.ObjectSizes; import org.apache.cassandra.utils.btree.BTree; -import static junit.framework.Assert.assertEquals; -import static junit.framework.Assert.assertTrue; +import static org.junit.Assert.assertEquals; +import static org.junit.Assert.assertTrue; public class RowIndexEntryTest extends CQLTester { diff --git a/test/unit/org/apache/cassandra/db/compaction/LeveledGenerationsTest.java b/test/unit/org/apache/cassandra/db/compaction/LeveledGenerationsTest.java index 0e20d637c1..efcb102d5c 100644 --- a/test/unit/org/apache/cassandra/db/compaction/LeveledGenerationsTest.java +++ b/test/unit/org/apache/cassandra/db/compaction/LeveledGenerationsTest.java @@ -37,7 +37,7 @@ import org.apache.cassandra.io.sstable.format.SSTableReader; import org.apache.cassandra.schema.MockSchema; import org.apache.cassandra.utils.ByteBufferUtil; -import static junit.framework.Assert.assertFalse; +import static org.junit.Assert.assertFalse; import static org.junit.Assert.assertEquals; import static org.junit.Assert.fail; diff --git a/test/unit/org/apache/cassandra/db/lifecycle/HelpersTest.java b/test/unit/org/apache/cassandra/db/lifecycle/HelpersTest.java index 7acd3e62f2..1b121c1e14 100644 --- a/test/unit/org/apache/cassandra/db/lifecycle/HelpersTest.java +++ b/test/unit/org/apache/cassandra/db/lifecycle/HelpersTest.java @@ -38,8 +38,8 @@ import org.apache.cassandra.db.compaction.OperationType; import org.apache.cassandra.io.sstable.format.SSTableReader; import org.apache.cassandra.schema.MockSchema; -import static junit.framework.Assert.assertEquals; -import static junit.framework.Assert.assertNotNull; +import static org.junit.Assert.assertEquals; +import static org.junit.Assert.assertNotNull; import static org.junit.Assert.assertTrue; public class HelpersTest diff --git a/test/unit/org/apache/cassandra/hints/ChecksummedDataInputTest.java b/test/unit/org/apache/cassandra/hints/ChecksummedDataInputTest.java index 4642584005..cc29163cd4 100644 --- a/test/unit/org/apache/cassandra/hints/ChecksummedDataInputTest.java +++ b/test/unit/org/apache/cassandra/hints/ChecksummedDataInputTest.java @@ -20,7 +20,6 @@ package org.apache.cassandra.hints; import java.io.IOException; import java.nio.ByteBuffer; import java.nio.charset.StandardCharsets; -import java.util.Arrays; import java.util.zip.CRC32; import org.apache.cassandra.io.util.File; @@ -35,9 +34,10 @@ import org.apache.cassandra.io.util.SequentialWriter; import org.apache.cassandra.utils.ByteBufferUtil; import org.apache.cassandra.utils.FBUtilities; -import static junit.framework.Assert.assertEquals; -import static junit.framework.Assert.assertTrue; +import static org.junit.Assert.assertArrayEquals; +import static org.junit.Assert.assertEquals; import static org.junit.Assert.assertFalse; +import static org.junit.Assert.assertTrue; public class ChecksummedDataInputTest { @@ -104,12 +104,12 @@ public class ChecksummedDataInputTest assertEquals(127, reader.read()); byte[] bytes = new byte[b.length]; reader.readFully(bytes); - assertTrue(Arrays.equals(bytes, b)); - assertEquals(false, reader.readBoolean()); + assertArrayEquals(bytes, b); + assertFalse(reader.readBoolean()); assertEquals(10, reader.readByte()); assertEquals('t', reader.readChar()); - assertEquals(3.3, reader.readDouble()); - assertEquals(2.2f, reader.readFloat()); + assertEquals(3.3, reader.readDouble(), 0.0); + assertEquals(2.2f, reader.readFloat(), 0.0); assertEquals(42, reader.readInt()); assertEquals(Long.MAX_VALUE, reader.readLong()); assertEquals(Short.MIN_VALUE, reader.readShort()); @@ -176,14 +176,14 @@ public class ChecksummedDataInputTest // assert that we read all the right values back assertEquals(127, reader.read()); - assertEquals(false, reader.readBoolean()); + assertFalse(reader.readBoolean()); assertEquals(10, reader.readByte()); assertEquals('t', reader.readChar()); assertTrue(reader.checkCrc()); reader.resetCrc(); - assertEquals(3.3, reader.readDouble()); - assertEquals(2.2f, reader.readFloat()); + assertEquals(3.3, reader.readDouble(), 0.0); + assertEquals(2.2f, reader.readFloat(), 0.0); assertEquals(42, reader.readInt()); assertTrue(reader.checkCrc()); assertTrue(reader.isEOF()); @@ -232,7 +232,7 @@ public class ChecksummedDataInputTest // assert that we read all the right values back assertEquals(127, reader.read()); - assertEquals(false, reader.readBoolean()); + assertFalse(reader.readBoolean()); assertEquals(10, reader.readByte()); assertEquals('t', reader.readChar()); assertFalse(reader.checkCrc()); diff --git a/test/unit/org/apache/cassandra/hints/HintTest.java b/test/unit/org/apache/cassandra/hints/HintTest.java index 67e969b32c..c762973b74 100644 --- a/test/unit/org/apache/cassandra/hints/HintTest.java +++ b/test/unit/org/apache/cassandra/hints/HintTest.java @@ -52,7 +52,8 @@ import org.apache.cassandra.service.StorageProxy; import org.apache.cassandra.service.StorageService; import org.apache.cassandra.utils.FBUtilities; -import static junit.framework.Assert.*; +import static org.junit.Assert.assertEquals; +import static org.junit.Assert.assertFalse; import static org.apache.cassandra.Util.dk; import static org.apache.cassandra.hints.HintsTestUtil.assertHintsEqual; diff --git a/test/unit/org/apache/cassandra/hints/HintsBufferPoolTest.java b/test/unit/org/apache/cassandra/hints/HintsBufferPoolTest.java index 1374d80a49..d658c88985 100644 --- a/test/unit/org/apache/cassandra/hints/HintsBufferPoolTest.java +++ b/test/unit/org/apache/cassandra/hints/HintsBufferPoolTest.java @@ -26,12 +26,12 @@ import org.junit.runner.RunWith; import com.google.common.collect.ImmutableList; -import static junit.framework.Assert.*; - import java.util.Queue; import java.util.UUID; import java.util.concurrent.ConcurrentLinkedQueue; +import static org.junit.Assert.assertTrue; + @RunWith(BMUnitRunner.class) public class HintsBufferPoolTest { diff --git a/test/unit/org/apache/cassandra/hints/HintsBufferTest.java b/test/unit/org/apache/cassandra/hints/HintsBufferTest.java index 3020fb9504..42daebf872 100644 --- a/test/unit/org/apache/cassandra/hints/HintsBufferTest.java +++ b/test/unit/org/apache/cassandra/hints/HintsBufferTest.java @@ -45,7 +45,11 @@ import org.apache.cassandra.utils.Clock; import org.jboss.byteman.contrib.bmunit.BMRule; import org.jboss.byteman.contrib.bmunit.BMUnitRunner; -import static junit.framework.Assert.*; +import static org.junit.Assert.assertEquals; +import static org.junit.Assert.assertNotNull; +import static org.junit.Assert.assertNull; +import static org.junit.Assert.assertTrue; +import static org.junit.Assert.fail; import static org.apache.cassandra.utils.ByteBufferUtil.bytes; import static org.apache.cassandra.utils.FBUtilities.updateChecksum; diff --git a/test/unit/org/apache/cassandra/hints/HintsCatalogTest.java b/test/unit/org/apache/cassandra/hints/HintsCatalogTest.java index 8b377b2154..af73d1b009 100644 --- a/test/unit/org/apache/cassandra/hints/HintsCatalogTest.java +++ b/test/unit/org/apache/cassandra/hints/HintsCatalogTest.java @@ -34,7 +34,12 @@ import org.junit.Rule; import org.junit.Test; import org.junit.rules.TemporaryFolder; -import static junit.framework.Assert.*; +import static org.junit.Assert.assertEquals; +import static org.junit.Assert.assertFalse; +import static org.junit.Assert.assertNotNull; +import static org.junit.Assert.assertNull; +import static org.junit.Assert.assertTrue; + import static org.apache.cassandra.Util.dk; public class HintsCatalogTest diff --git a/test/unit/org/apache/cassandra/hints/HintsDescriptorTest.java b/test/unit/org/apache/cassandra/hints/HintsDescriptorTest.java index 04fd8c3940..596727fad1 100644 --- a/test/unit/org/apache/cassandra/hints/HintsDescriptorTest.java +++ b/test/unit/org/apache/cassandra/hints/HintsDescriptorTest.java @@ -33,9 +33,9 @@ import org.apache.cassandra.io.compress.LZ4Compressor; import org.apache.cassandra.io.util.DataOutputBuffer; import org.apache.cassandra.net.MessagingService; -import static junit.framework.Assert.assertEquals; -import static junit.framework.Assert.assertNotSame; -import static junit.framework.Assert.fail; +import static org.junit.Assert.assertEquals; +import static org.junit.Assert.assertNotSame; +import static org.junit.Assert.fail; import static org.assertj.core.api.Assertions.assertThat; public class HintsDescriptorTest diff --git a/test/unit/org/apache/cassandra/hints/HintsReaderTest.java b/test/unit/org/apache/cassandra/hints/HintsReaderTest.java index 249772bb8e..3e3c64953b 100644 --- a/test/unit/org/apache/cassandra/hints/HintsReaderTest.java +++ b/test/unit/org/apache/cassandra/hints/HintsReaderTest.java @@ -47,8 +47,8 @@ import org.apache.cassandra.schema.Schema; import org.apache.cassandra.schema.SchemaTestUtil; import org.apache.cassandra.schema.TableMetadata; -import static junit.framework.Assert.assertEquals; -import static junit.framework.Assert.assertNotNull; +import static org.junit.Assert.assertEquals; +import static org.junit.Assert.assertNotNull; import static org.apache.cassandra.Util.dk; import static org.apache.cassandra.utils.ByteBufferUtil.bytes; diff --git a/test/unit/org/apache/cassandra/hints/HintsStoreTest.java b/test/unit/org/apache/cassandra/hints/HintsStoreTest.java index c1847786b6..93e6a1197f 100644 --- a/test/unit/org/apache/cassandra/hints/HintsStoreTest.java +++ b/test/unit/org/apache/cassandra/hints/HintsStoreTest.java @@ -43,8 +43,12 @@ import org.apache.cassandra.db.RowUpdateBuilder; import org.apache.cassandra.io.util.FileUtils; import org.apache.cassandra.schema.KeyspaceParams; +import static org.junit.Assert.assertEquals; +import static org.junit.Assert.assertFalse; +import static org.junit.Assert.assertNull; +import static org.junit.Assert.assertTrue; + import static org.apache.cassandra.utils.ByteBufferUtil.bytes; -import static junit.framework.Assert.*; public class HintsStoreTest { diff --git a/test/unit/org/apache/cassandra/hints/HintsTestUtil.java b/test/unit/org/apache/cassandra/hints/HintsTestUtil.java index 6c95f3eeae..727404e6e8 100644 --- a/test/unit/org/apache/cassandra/hints/HintsTestUtil.java +++ b/test/unit/org/apache/cassandra/hints/HintsTestUtil.java @@ -35,8 +35,8 @@ import org.apache.cassandra.schema.TableMetadata; import org.apache.cassandra.service.StorageService; import org.apache.cassandra.utils.Clock; -import static junit.framework.Assert.assertEquals; -import static junit.framework.Assert.assertTrue; +import static org.junit.Assert.assertEquals; +import static org.junit.Assert.assertTrue; import static org.apache.cassandra.Util.dk; import static org.apache.cassandra.net.MockMessagingService.verb; import static org.apache.cassandra.net.Verb.HINT_REQ; diff --git a/test/unit/org/apache/cassandra/io/sstable/SSTableWriterTest.java b/test/unit/org/apache/cassandra/io/sstable/SSTableWriterTest.java index b4b9fbccdf..1ebb3c6fb4 100644 --- a/test/unit/org/apache/cassandra/io/sstable/SSTableWriterTest.java +++ b/test/unit/org/apache/cassandra/io/sstable/SSTableWriterTest.java @@ -34,12 +34,13 @@ import org.apache.cassandra.io.sstable.format.SSTableReadsListener; import org.apache.cassandra.io.sstable.format.SSTableWriter; import org.apache.cassandra.utils.TimeUUID; -import static junit.framework.Assert.fail; +import static org.junit.Assert.fail; +import static org.junit.Assert.assertEquals; +import static org.junit.Assert.assertTrue; + import static org.apache.cassandra.service.ActiveRepairService.NO_PENDING_REPAIR; import static org.apache.cassandra.service.ActiveRepairService.UNREPAIRED_SSTABLE; import static org.apache.cassandra.utils.TimeUUID.Generator.nextTimeUUID; -import static org.junit.Assert.assertEquals; -import static org.junit.Assert.assertTrue; public class SSTableWriterTest extends SSTableWriterTestBase { @@ -95,7 +96,7 @@ public class SSTableWriterTest extends SSTableWriterTestBase @Test - public void testAbortTxnWithClosedWriterShouldRemoveSSTable() throws InterruptedException + public void testAbortTxnWithClosedWriterShouldRemoveSSTable() { Keyspace keyspace = Keyspace.open(KEYSPACE); ColumnFamilyStore cfs = keyspace.getColumnFamilyStore(CF); @@ -139,7 +140,7 @@ public class SSTableWriterTest extends SSTableWriterTestBase } @Test - public void testAbortTxnWithClosedAndOpenWriterShouldRemoveAllSSTables() throws InterruptedException + public void testAbortTxnWithClosedAndOpenWriterShouldRemoveAllSSTables() { Keyspace keyspace = Keyspace.open(KEYSPACE); ColumnFamilyStore cfs = keyspace.getColumnFamilyStore(CF); diff --git a/test/unit/org/apache/cassandra/io/sstable/format/VersionAndTypeTest.java b/test/unit/org/apache/cassandra/io/sstable/format/VersionAndTypeTest.java index 633993f0f9..4e62b9c07c 100644 --- a/test/unit/org/apache/cassandra/io/sstable/format/VersionAndTypeTest.java +++ b/test/unit/org/apache/cassandra/io/sstable/format/VersionAndTypeTest.java @@ -20,7 +20,7 @@ package org.apache.cassandra.io.sstable.format; import org.junit.Test; import static org.assertj.core.api.Assertions.assertThatThrownBy; -import static junit.framework.Assert.assertEquals; +import static org.junit.Assert.assertEquals; public class VersionAndTypeTest { diff --git a/test/unit/org/apache/cassandra/io/util/MmappedRegionsTest.java b/test/unit/org/apache/cassandra/io/util/MmappedRegionsTest.java index 5692342d49..7194d3042f 100644 --- a/test/unit/org/apache/cassandra/io/util/MmappedRegionsTest.java +++ b/test/unit/org/apache/cassandra/io/util/MmappedRegionsTest.java @@ -37,13 +37,14 @@ import org.apache.cassandra.io.compress.CompressionMetadata; import org.apache.cassandra.io.sstable.metadata.MetadataCollector; import org.apache.cassandra.schema.CompressionParams; -import static junit.framework.Assert.assertNull; -import static org.apache.cassandra.utils.Clock.Global.nanoTime; +import static org.junit.Assert.assertNull; import static org.junit.Assert.assertEquals; import static org.junit.Assert.assertFalse; import static org.junit.Assert.assertNotNull; import static org.junit.Assert.assertTrue; +import static org.apache.cassandra.utils.Clock.Global.nanoTime; + public class MmappedRegionsTest { private static final Logger logger = LoggerFactory.getLogger(MmappedRegionsTest.class); diff --git a/test/unit/org/apache/cassandra/metrics/LatencyMetricsTest.java b/test/unit/org/apache/cassandra/metrics/LatencyMetricsTest.java index 6206ba64f1..3a4dc8af98 100644 --- a/test/unit/org/apache/cassandra/metrics/LatencyMetricsTest.java +++ b/test/unit/org/apache/cassandra/metrics/LatencyMetricsTest.java @@ -22,8 +22,8 @@ import java.util.concurrent.TimeUnit; import org.junit.Test; -import static junit.framework.Assert.assertEquals; -import static junit.framework.Assert.assertFalse; +import static org.junit.Assert.assertEquals; +import static org.junit.Assert.assertFalse; public class LatencyMetricsTest { diff --git a/test/unit/org/apache/cassandra/net/MockMessagingSpy.java b/test/unit/org/apache/cassandra/net/MockMessagingSpy.java index f10d1ee74d..d3ea8f714b 100644 --- a/test/unit/org/apache/cassandra/net/MockMessagingSpy.java +++ b/test/unit/org/apache/cassandra/net/MockMessagingSpy.java @@ -32,8 +32,6 @@ import com.google.common.util.concurrent.MoreExecutors; import org.slf4j.Logger; import org.slf4j.LoggerFactory; -import junit.framework.AssertionFailedError; - import static org.apache.cassandra.utils.concurrent.BlockingQueues.newBlockingQueue; /** @@ -233,7 +231,7 @@ public class MockMessagingSpy { T result = queue.poll(time, unit); if (result != null) - setException(new AssertionFailedError("Received unexpected message: " + result)); + setException(new AssertionError("Received unexpected message: " + result)); else set(true); } diff --git a/test/unit/org/apache/cassandra/repair/asymmetric/DifferenceHolderTest.java b/test/unit/org/apache/cassandra/repair/asymmetric/DifferenceHolderTest.java index c37e3d7d9a..cab28e9d1c 100644 --- a/test/unit/org/apache/cassandra/repair/asymmetric/DifferenceHolderTest.java +++ b/test/unit/org/apache/cassandra/repair/asymmetric/DifferenceHolderTest.java @@ -36,7 +36,7 @@ import org.apache.cassandra.utils.MerkleTree; import org.apache.cassandra.utils.MerkleTrees; import org.apache.cassandra.utils.MerkleTreesTest; -import static junit.framework.Assert.assertEquals; +import static org.junit.Assert.assertEquals; import static org.junit.Assert.assertTrue; public class DifferenceHolderTest diff --git a/test/unit/org/apache/cassandra/repair/asymmetric/RangeMapTest.java b/test/unit/org/apache/cassandra/repair/asymmetric/RangeMapTest.java index 0805ea7756..40ac7d6271 100644 --- a/test/unit/org/apache/cassandra/repair/asymmetric/RangeMapTest.java +++ b/test/unit/org/apache/cassandra/repair/asymmetric/RangeMapTest.java @@ -31,7 +31,7 @@ import org.apache.cassandra.dht.Murmur3Partitioner; import org.apache.cassandra.dht.Range; import org.apache.cassandra.dht.Token; -import static junit.framework.TestCase.assertEquals; +import static org.junit.Assert.assertEquals; import static org.junit.Assert.assertFalse; public class RangeMapTest diff --git a/test/unit/org/apache/cassandra/repair/asymmetric/ReduceHelperTest.java b/test/unit/org/apache/cassandra/repair/asymmetric/ReduceHelperTest.java index be5a553ba5..f2ee14f11c 100644 --- a/test/unit/org/apache/cassandra/repair/asymmetric/ReduceHelperTest.java +++ b/test/unit/org/apache/cassandra/repair/asymmetric/ReduceHelperTest.java @@ -44,9 +44,9 @@ import org.apache.cassandra.dht.Range; import org.apache.cassandra.dht.Token; import org.apache.cassandra.locator.InetAddressAndPort; -import static junit.framework.TestCase.fail; import static org.junit.Assert.assertEquals; import static org.junit.Assert.assertTrue; +import static org.junit.Assert.fail; public class ReduceHelperTest { diff --git a/test/unit/org/apache/cassandra/repair/asymmetric/StreamFromOptionsTest.java b/test/unit/org/apache/cassandra/repair/asymmetric/StreamFromOptionsTest.java index e2a7700dce..d13937e611 100644 --- a/test/unit/org/apache/cassandra/repair/asymmetric/StreamFromOptionsTest.java +++ b/test/unit/org/apache/cassandra/repair/asymmetric/StreamFromOptionsTest.java @@ -31,7 +31,7 @@ import org.apache.cassandra.dht.Range; import org.apache.cassandra.dht.Token; import org.apache.cassandra.locator.InetAddressAndPort; -import static junit.framework.Assert.assertEquals; +import static org.junit.Assert.assertEquals; import static org.junit.Assert.assertFalse; import static org.junit.Assert.assertTrue; diff --git a/test/unit/org/apache/cassandra/service/ClientWarningsTest.java b/test/unit/org/apache/cassandra/service/ClientWarningsTest.java index e7a34e0aa5..d9b2b1144a 100644 --- a/test/unit/org/apache/cassandra/service/ClientWarningsTest.java +++ b/test/unit/org/apache/cassandra/service/ClientWarningsTest.java @@ -38,7 +38,7 @@ import org.apache.cassandra.transport.ProtocolVersion; import org.apache.cassandra.transport.SimpleClient; import org.apache.cassandra.transport.messages.QueryMessage; -import static junit.framework.Assert.assertEquals; +import static org.junit.Assert.assertEquals; import static org.junit.Assert.assertNull; @RunWith(Parameterized.class) diff --git a/test/unit/org/apache/cassandra/service/ProtocolBetaVersionTest.java b/test/unit/org/apache/cassandra/service/ProtocolBetaVersionTest.java index bfb464dbdc..a5b32bfa1f 100644 --- a/test/unit/org/apache/cassandra/service/ProtocolBetaVersionTest.java +++ b/test/unit/org/apache/cassandra/service/ProtocolBetaVersionTest.java @@ -25,9 +25,9 @@ import org.apache.cassandra.cql3.*; import org.apache.cassandra.transport.*; import org.apache.cassandra.transport.messages.*; -import static junit.framework.Assert.assertEquals; -import static junit.framework.Assert.assertTrue; -import static junit.framework.Assert.fail; +import static org.junit.Assert.assertEquals; +import static org.junit.Assert.assertTrue; +import static org.junit.Assert.fail; public class ProtocolBetaVersionTest extends CQLTester {