CS2113-AY1819S2-T09-1 / main / 835

92%

push

travis-ci

App: improve overall code quality (#170)

There's inconsistent/insufficient use of defensive programming in our
codebase.

Let's update the codebase to add precondition checks by using:
* `requireNonNull` to verify preconditions in public methods
* `checkArgument` to verify preconditions in public methods
* `assert` to verify preconditions in private/protected methods

Besides that, let's also rename `ApplicationParser` to `CommandParser`
so that it makes more sense, and reflects its actual purpose.

Precondition checks are done in accordance to guidelines obtained from:
https://docs.oracle.com/javase/8/docs/technotes/guides/language/assert.html

230 of 230 new or added lines in 75 files covered. (100.0%)

2594 of 3001 relevant lines covered (86.44%)

0.86 hits per line