LO-JUnit is outdated
Closed this issue · 2 comments
damithc commented
The given exercise is outdated as the test code in concern has already been added.
Should be fixed before the next semester.
PierceAndy commented
Concerns:
- Currently, the Learning Outcome's exercise for JUnit tests is to create unit tests for the
Utils
class attest/seedu/addressbook/common/UtilsTest.java
. - However this exercise was addressed as issue #107, and closed by PR #127, as mentioned here: #107 (comment)
- There is no other equivalent LO for JUnit tests.
- The LO for JUnit tests serves as an introduction, and shouldn't be too complicated to implement.
Solutions:
UniquePersonList
, andUniqueTagList
classes still lack JUnit tests (See: #118).- There is an existing PR that is on hold that address this (See: #126).
- We could redirect students to create JUnit tests for either of these instead.
- The class being used for the updated LO should not be referenced as an issue.
damithc commented
Fixed in a recent commit.