Skip to content

Commit

Permalink
Switched CLI to Cement and improved error handling + logging faciliti…
Browse files Browse the repository at this point in the history
…es (fixes #216 and fixes #219) (#289)

* Ported CLI to cement (#218)

* added super minimal cement-based CLI

* added basic bug list command to cement CLI

* added -q option to bug list command

* updated force parameter to True for bug download

* added upload and download commands to bug CLI

* updated BugManager to set force to True

* added uninstall command to cement CLI

* added bug coverage method to cement CLI

* added bug validate command to cement CLI

* added commented-out code for --installed and --uninstalled flags to bug list command

* added basic source group to cement CLI

* added source add command to cement CLI

* added source update command to cement CLI

* added __delitem__ to source manager

* added source remove command to cement CLI

* ported tool commands to cement CLI

* ported container commands to cement CLI

* finished porting existing CLI to cement

* fixed logging bugs and tweaked CLI to exit with code 1 on failure

* refactored cli/__init__.py

* updated changelog

* code style violation

* updated base controller

* updated bug controller

* updated container controller

* updated source controller

* updated tool controller

* updated app module

* fixed source controller

* removed manybugs test

* removed test_manybugs from .travis.yml

* added better error handling to CLI

* added logging to file and prettier error formatting

* tidied up error reporting

* updated logger to use mode w

* ensure log directory exists

* write logs to .bugzoo/logs

* updated changelog

* removed date from CHANGELOG

* updated bug controller to throw BugNotFound errors

* added ToolNotFound exception

* updated container controller to throw better errors

* updated source controller to raise SourceNotFoundWithName

* updated tool controller to throw ToolNotFound exceptions

* renamed bug uninstall to bug remove

* renamed tool uninstall to tool remove

* updated change log
  • Loading branch information
ChrisTimperley authored Nov 1, 2018
1 parent 55640ec commit 56092dc
Show file tree
Hide file tree
Showing 14 changed files with 643 additions and 606 deletions.
1 change: 0 additions & 1 deletion .travis.yml
Original file line number Diff line number Diff line change
Expand Up @@ -17,7 +17,6 @@ script:
- bugzoo tool list
- bugzoo source list
- bugzoo bug list
- python3 test/test_manybugs.py
- mypy bugzoo --ignore-missing-imports
before_deploy:
- "(cd docs && make html && cd _build/html && touch .nojekyll)"
Expand Down
14 changes: 13 additions & 1 deletion CHANGELOG.md
Original file line number Diff line number Diff line change
@@ -1,8 +1,20 @@
## 2.1.20 (2018-10-25)
## 2.1.20 (XXXX-XX-XX)

### Features

* Added `write` method to client-side and server-side file manager.
* Individual log files for each CLI session are now saved to `.bugzoo/logs`.
* Added better error reporting to CLI, which now reports the location of the
relevant log file to the user.


### Changes

* Switched implementation of command-line interface from `argparse` to
`cement`.
* Replaced `bug uninstall` command with `bug rm` and `bug remove` commands.
* Replaced `tool uninstall` command with `tool rm` and `tool remove`
commands.


### Bug Fixes
Expand Down
Loading

0 comments on commit 56092dc

Please sign in to comment.