NebuPookins / toggl-cli / #4 / 1

52%

master: 52%

push

Merge remote-tracking branch 'nethunter/master'

Looks like nethunter did not run `npm test`, so this commit also fixes
some of the lint issues and tests so that they pass again.

Minor behaviour changes from nethunter's code:

* Previously the project property could be parsed as hexdecimal, e.g. if
  the user entered in `proj:FF`. I suspect this was unintentional. Now
  project numbers must be in decimal. `proj:FF` will NOT be interpreted
  as referring to project number 255, but instead be interpreted as the
  project whose name starts with "FF".
* Previously there was no handling for unrecognized properties, e.g.
  (`foo:bar`). Now, if we encounter an unrecognized property, we assume
  it's actually part of the name of the task, thus allowing for tasks to
  contain colons in them.

Testing done:
* `npm start pr`: Confirmed my projects were printed.
* `npm start cl`: Confirmed my clients were printed.
* `npm start start proj:7 Test` confirmed the task was started with project 7.
* `npm start stop proj:7 Test` confirmed the task was stopped with project 7.
* `npm start start proj:MyProj Test` confirmed the task was started with project whose name starts with "MyProj".
* `npm start stop proj:MyProj Test` confirmed the task was stopped with project whose name starts with "MyProj".
* `npm start -- --help`: Confirmed help info is displayed.
* `npm start -- --examples`: Confirmed examples are displayed.
* `npm start foo:bar Test`: Confirmed this starts a task with the name "foo:bar Test" (i.e. unknown properties are interpreted as part of the task name.)

88 of 299 branches covered (29.43%)

Branch coverage included in aggregate %.

243 of 607 relevant lines covered (40.03%)

4.44 hits per line