1
92%
master: 92%

Ran 04 Apr 2019 10:02AM UTC

Files 114

Run time 7s

Badge

Embed ▾

Committed 04 Apr 2019 09:58AM UTC coverage: 86.438% (+0.1%) from 86.33%

Job # 835.1

Build Type

push

travis-ci

Committed by

lycjackie

Commit Message

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

Run Details

2594 of 3001 relevant lines covered (86.44%)

0.86 hits per line

CS2113-AY1819S2-T09-1 / main / 835 / 1
92%
master: 92%

README BADGES
x

Markdown

Textile

RDoc

HTML

Rst

Source Files on job 835.1

CS2113-AY1819S2-T09-1 / main / 835 / 1 92% master: 92%

README BADGES x

Markdown

Textile

RDoc

HTML

Rst

Source Files on job 835.1

CS2113-AY1819S2-T09-1 / main / 835 / 1
92%
master: 92%

README BADGES
x