Don’t allow null for the text attribute of PlainTextPart
authorDavid ‘Bombe’ Roden <bombe@pterodactylus.net>
Wed, 5 Oct 2016 19:09:56 +0000 (21:09 +0200)
committerDavid ‘Bombe’ Roden <bombe@pterodactylus.net>
Wed, 5 Oct 2016 19:09:56 +0000 (21:09 +0200)
src/main/java/net/pterodactylus/sone/text/PlainTextPart.java
src/test/java/net/pterodactylus/sone/text/PlainTextPartTest.java

index 4f15a73..7bdbee9 100644 (file)
 
 package net.pterodactylus.sone.text;
 
+import java.util.Objects;
+
+import javax.annotation.Nonnull;
+
 /**
  * {@link Part} implementation that holds a single piece of text.
  *
@@ -24,29 +28,14 @@ package net.pterodactylus.sone.text;
  */
 public class PlainTextPart implements Part {
 
-       /** The text of the part. */
        private final String text;
 
-       /**
-        * Creates a new plain-text part.
-        *
-        * @param text
-        *            The text of the part
-        */
-       public PlainTextPart(String text) {
-               this.text = text;
+       public PlainTextPart(@Nonnull String text) {
+               this.text = Objects.requireNonNull(text);
        }
 
-       //
-       // PART METHODS
-       //
-
-       /**
-        * Returns the text of this part.
-        *
-        * @return The text of this part
-        */
        @Override
+       @Nonnull
        public String getText() {
                return text;
        }
index 8e6f33a..72161ab 100644 (file)
@@ -19,4 +19,9 @@ public class PlainTextPartTest {
                assertThat(part.getText(), is("text"));
        }
 
+       @Test(expected = NullPointerException.class)
+       public void nullIsNotAllowedForText() {
+           new PlainTextPart(null);
+       }
+
 }