Skip to content

Commit a53cd81

Browse files
authored
Merge pull request nus-cs2103-AY2021S1#69 from lucastai98/branch-meeting
add tests for meeting class
2 parents fadbb34 + c3570dd commit a53cd81

2 files changed

Lines changed: 28 additions & 3 deletions

File tree

src/main/java/seedu/address/model/meeting/Meeting.java

Lines changed: 8 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -1,4 +1,7 @@
11
package seedu.address.model.meeting;
2+
3+
import static seedu.address.commons.util.CollectionUtil.requireAllNonNull;
4+
25
import java.time.LocalDate;
36
import java.time.LocalDateTime;
47
import java.time.temporal.Temporal;
@@ -17,11 +20,13 @@ public class Meeting {
1720

1821
/**
1922
* Constructor for meeting.
23+
* Date and timing for the meeting should be present and not null.
2024
*
21-
* @param name name of meeting
25+
* @param localDateTime date and time of meeting
2226
*/
23-
public Meeting(String name) {
24-
this.name = name;
27+
public Meeting(String localDateTime) {
28+
requireAllNonNull(localDateTime);
29+
startDateTime = LocalDateTime.parse(localDateTime);
2530
isDone = false;
2631
}
2732

Lines changed: 20 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,20 @@
1+
package seedu.address.model.meeting;
2+
3+
import static seedu.address.testutil.Assert.assertThrows;
4+
5+
import org.junit.jupiter.api.Test;
6+
7+
public class MeetingTest {
8+
9+
@Test
10+
public void constructor_null_throwsNullPointerException() {
11+
assertThrows(NullPointerException.class, () -> new Meeting(null));
12+
}
13+
14+
@Test
15+
public void constructor_invalidMeeting_throwsIllegalArgumentException() {
16+
assertThrows(java.time.format.DateTimeParseException.class, () -> new Meeting(""));
17+
18+
}
19+
20+
}

0 commit comments

Comments
 (0)