Clean up tests

Author: JooHyukKim

protobuf/src/test/java/com/fasterxml/jackson/dataformat/protobuf/testutil/failure/JacksonTestFailureExpected.java

@@ -21,7 +21,6 @@
  *
  *     @Test
  *     @JacksonTestFailureExpected
- *     @Test
     public void testFeatureNotYetImplemented() {
  *         // Test code that is expected to fail
  *     }

smile/src/test/java/com/fasterxml/jackson/dataformat/smile/async/AsyncReaderWrapperForByteArray.java

@@ -6,6 +6,9 @@
 import com.fasterxml.jackson.core.JsonToken;
 import com.fasterxml.jackson.core.async.ByteArrayFeeder;
 
+/**
+ * Helper class used with async parser
+ */
 public class AsyncReaderWrapperForByteArray extends AsyncReaderWrapper
 {
     private final byte[] _doc;

smile/src/test/java/com/fasterxml/jackson/dataformat/smile/async/AsyncSharedStringsTest.java

@@ -13,6 +13,11 @@
 
 import static org.junit.jupiter.api.Assertions.*;
 
+/**
+ * Unit tests for verifying that symbol handling works as planned, including
+ * efficient reuse of names encountered during parsing.
+ * Copied and slightly modified version of `ParserSymbolHandlingTest`
+ */
 public class AsyncSharedStringsTest
 	extends AsyncTestBase
 {

smile/src/test/java/com/fasterxml/jackson/dataformat/smile/constraints/DeeplyNestedSmileReadWriteTest.java

@@ -12,8 +12,7 @@
 import com.fasterxml.jackson.dataformat.smile.SmileFactory;
 import com.fasterxml.jackson.dataformat.smile.databind.SmileMapper;
 
-import static org.junit.jupiter.api.Assertions.assertEquals;
-import static org.junit.jupiter.api.Assertions.fail;
+import static org.junit.jupiter.api.Assertions.*;
 
 public class DeeplyNestedSmileReadWriteTest extends BaseTestForSmile
 {

smile/src/test/java/com/fasterxml/jackson/dataformat/smile/constraints/LongDocumentSmileReadTest.java

@@ -13,8 +13,7 @@
 import com.fasterxml.jackson.dataformat.smile.SmileFactory;
 import com.fasterxml.jackson.dataformat.smile.databind.SmileMapper;
 
-import static org.junit.jupiter.api.Assertions.assertTrue;
-import static org.junit.jupiter.api.Assertions.fail;
+import static org.junit.jupiter.api.Assertions.*;
 
 public class LongDocumentSmileReadTest extends BaseTestForSmile
 {

smile/src/test/java/com/fasterxml/jackson/dataformat/smile/fuzz/Fuzz32168BigDecimalTest.java

@@ -11,6 +11,7 @@
 import static org.junit.jupiter.api.Assertions.assertEquals;
 import static org.junit.jupiter.api.Assertions.assertTrue;
 
+// For [dataformats-binary#257]
 public class Fuzz32168BigDecimalTest extends BaseTestForSmile
 {
     private final ObjectMapper MAPPER = smileMapper();

smile/src/test/java/com/fasterxml/jackson/dataformat/smile/fuzz/Fuzz32169HeaderDecodeTest.java

@@ -8,6 +8,7 @@
 
 import static org.junit.jupiter.api.Assertions.fail;
 
+//For [dataformats-binary#258]
 public class Fuzz32169HeaderDecodeTest extends BaseTestForSmile
 {
     private final ObjectMapper MAPPER = smileMapper();

smile/src/test/java/com/fasterxml/jackson/dataformat/smile/fuzz/Fuzz32180RawBinaryTest.java

@@ -9,6 +9,7 @@
 import com.fasterxml.jackson.databind.ObjectMapper;
 import com.fasterxml.jackson.dataformat.smile.BaseTestForSmile;
 
+// For [dataformats-binary#260]
 public class Fuzz32180RawBinaryTest extends BaseTestForSmile
 {
     private final ObjectMapper MAPPER = smileMapper();

smile/src/test/java/com/fasterxml/jackson/dataformat/smile/fuzz/Fuzz32339_7BitBinaryTest.java

@@ -9,6 +9,7 @@
 import com.fasterxml.jackson.databind.ObjectMapper;
 import com.fasterxml.jackson.dataformat.smile.BaseTestForSmile;
 
+//For [dataformats-binary#263]
 public class Fuzz32339_7BitBinaryTest extends BaseTestForSmile
 {
     private final ObjectMapper MAPPER = smileMapper();

smile/src/test/java/com/fasterxml/jackson/dataformat/smile/fuzz/Fuzz32665RepeatedHeaderTest.java

@@ -12,6 +12,7 @@
 import static org.junit.jupiter.api.Assertions.assertNull;
 import static org.junit.jupiter.api.Assertions.fail;
 
+// for [dataformats-binary#268]
 public class Fuzz32665RepeatedHeaderTest extends BaseTestForSmile
 {
     private final ObjectMapper MAPPER = smileMapper();