Skip to content

Commit

Permalink
Merge 16311cc into 0af9251
Browse files Browse the repository at this point in the history
  • Loading branch information
BillChee123 committed Nov 9, 2019
2 parents 0af9251 + 16311cc commit 14ef3ef
Showing 1 changed file with 11 additions and 11 deletions.
22 changes: 11 additions & 11 deletions src/test/java/seedu/weme/storage/JsonAdaptedMemeTest.java
Original file line number Diff line number Diff line change
Expand Up @@ -22,15 +22,15 @@ public class JsonAdaptedMemeTest extends ApplicationTest {
private static final String INVALID_TAG = "#friend";
private static final String INVALID_URL = "hello world";

private static String VALID_DESCRIPTION;
private static String VALID_URL;
private static List<JsonAdaptedTag> VALID_TAGS;
private static String validDescription;
private static String validUrl;
private static List<JsonAdaptedTag> validTags;

@BeforeEach
public void setup() {
VALID_DESCRIPTION = JOKER.getDescription().toString();
VALID_URL = JOKER.getImagePath().toString();
VALID_TAGS = JOKER.getTags().stream()
validDescription = JOKER.getDescription().toString();
validUrl = JOKER.getImagePath().toString();
validTags = JOKER.getTags().stream()
.map(JsonAdaptedTag::new)
.collect(Collectors.toList());
}
Expand All @@ -48,30 +48,30 @@ public void toModelType_validMemeDetails_returnsMeme() throws Exception {

@Test
public void toModelType_nullPath_throwsIllegalValueException() {
JsonAdaptedMeme meme = new JsonAdaptedMeme(null, VALID_DESCRIPTION, VALID_TAGS, false);
JsonAdaptedMeme meme = new JsonAdaptedMeme(null, validDescription, validTags, false);
String expectedMessage = String.format(MISSING_FIELD_MESSAGE_FORMAT, ImagePath.class.getSimpleName());
assertThrows(IllegalValueException.class, expectedMessage, meme::toModelType);
}

@Test
public void toModelType_invalidPath_throwsIllegalValueException() {
JsonAdaptedMeme meme = new JsonAdaptedMeme(INVALID_URL, VALID_DESCRIPTION, VALID_TAGS, false);
JsonAdaptedMeme meme = new JsonAdaptedMeme(INVALID_URL, validDescription, validTags, false);
assertThrows(IllegalValueException.class, meme::toModelType);
}

@Test
public void toModelType_nullDescription_throwsIllegalValueException() {
JsonAdaptedMeme meme = new JsonAdaptedMeme(VALID_URL, null, VALID_TAGS, false);
JsonAdaptedMeme meme = new JsonAdaptedMeme(validUrl, null, validTags, false);
String expectedMessage = String.format(MISSING_FIELD_MESSAGE_FORMAT, Description.class.getSimpleName());
assertThrows(IllegalValueException.class, expectedMessage, meme::toModelType);
}

@Test
public void toModelType_invalidTags_throwsIllegalValueException() {
List<JsonAdaptedTag> invalidTags = new ArrayList<>(VALID_TAGS);
List<JsonAdaptedTag> invalidTags = new ArrayList<>(validTags);
invalidTags.add(new JsonAdaptedTag(INVALID_TAG));
JsonAdaptedMeme meme =
new JsonAdaptedMeme(VALID_URL, VALID_DESCRIPTION, invalidTags, false);
new JsonAdaptedMeme(validUrl, validDescription, invalidTags, false);
assertThrows(IllegalValueException.class, meme::toModelType);
}

Expand Down

0 comments on commit 14ef3ef

Please sign in to comment.