Remove unnecessary code. Changed test from:

if ((to == -1) || (to == termnator)) {
to:
if (to == termnator) {
since we are guaranteed to find the separator because we have tested for its presence at the beginning of the method.

git-svn-id: https://svn.apache.org/repos/asf/commons/proper/codec/trunk@798427 13f79535-47bb-0310-9956-ffa450edef68

Author: garydgregory

src/java/org/apache/commons/codec/net/RFC1522Codec.java

@@ -130,7 +130,7 @@ protected String decodeText(final String text)
         }
         from = to + 1;
         to = text.indexOf(SEP, from);
-        if ((to == -1) || (to == termnator)) {
+        if (to == termnator) {
             throw new DecoderException("RFC 1522 violation: encoding token not found");
         }
         String encoding = text.substring(from, to);

src/test/org/apache/commons/codec/net/RFC1522CodecTest.java

@@ -29,12 +29,12 @@
  * @version $Id$
  */
 public class RFC1522CodecTest extends TestCase {
-    
+
     public RFC1522CodecTest(String name) {
         super(name);
     }
 
-    static class RFC1522TestCodec extends RFC1522Codec { 
+    static class RFC1522TestCodec extends RFC1522Codec {
 
         protected byte[] doDecoding(byte[] bytes) {
             return bytes;
@@ -56,50 +56,29 @@ public void testNullInput() throws Exception {
         assertNull(testcodec.encodeText(null, CharEncoding.UTF_8));
     }
 
-    public void testDecodeInvalid() throws Exception {
+    private void assertExpectedDecoderException(String s) throws Exception {
         RFC1522TestCodec testcodec = new RFC1522TestCodec();
         try {
-            testcodec.decodeText("whatever");
-            fail("DecoderException should have been thrown");
-        } catch (DecoderException e) {
-            // Expected. Move on
-        }
-        try {
-            testcodec.decodeText("=?stuff?=");
+            testcodec.decodeText(s);
             fail("DecoderException should have been thrown");
         } catch (DecoderException e) {
-            // Expected. Move on
-        }
-        try {
-            testcodec.decodeText("=?UTF-8?stuff?=");
-            fail("DecoderException should have been thrown");
-        } catch (DecoderException e) {
-            // Expected. Move on
-        }
-        try {
-            testcodec.decodeText("=?UTF-8?T?stuff");
-            fail("DecoderException should have been thrown");
-        } catch (DecoderException e) {
-            // Expected. Move on
-        }
-        try {
-            testcodec.decodeText("=??T?stuff?=");
-            fail("DecoderException should have been thrown");
-        } catch (DecoderException e) {
-            // Expected. Move on
-        }
-        try {
-            testcodec.decodeText("=?UTF-8??stuff?=");
-            fail("DecoderException should have been thrown");
-        } catch (DecoderException e) {
-            // Expected. Move on
-        }
-        try {
-            testcodec.decodeText("=?UTF-8?W?stuff?=");
-            fail("DecoderException should have been thrown");
-        } catch (DecoderException e) {
-            // Expected. Move on
+            // Expected.
         }
     }
 
+    public void testDecodeInvalid() throws Exception {
+        assertExpectedDecoderException("whatever");
+        assertExpectedDecoderException("=?");
+        assertExpectedDecoderException("?=");
+        assertExpectedDecoderException("==");
+        assertExpectedDecoderException("=??=");
+        assertExpectedDecoderException("=?stuff?=");
+        assertExpectedDecoderException("=?UTF-8??=");
+        assertExpectedDecoderException("=?UTF-8?stuff?=");
+        assertExpectedDecoderException("=?UTF-8?T?stuff");
+        assertExpectedDecoderException("=??T?stuff?=");
+        assertExpectedDecoderException("=?UTF-8??stuff?=");
+        assertExpectedDecoderException("=?UTF-8?W?stuff?=");
+    }
+
 }