summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorzlg <zlg@zlg.space>2018-10-18 21:02:53 -0700
committerzlg <zlg@zlg.space>2018-10-18 21:02:53 -0700
commit69373e362637b5e89ee347c9874f5f8c4ec6a294 (patch)
treed6ef21d2f8b3e105f1375680081f6ae3096f401b
parentREADME: expand on usage, cover shell quoting (diff)
downloadvgstash-69373e362637b5e89ee347c9874f5f8c4ec6a294.tar.gz
vgstash-69373e362637b5e89ee347c9874f5f8c4ec6a294.tar.bz2
vgstash-69373e362637b5e89ee347c9874f5f8c4ec6a294.tar.xz
vgstash-69373e362637b5e89ee347c9874f5f8c4ec6a294.zip
cli: show msg if game to be deleted is not in DB
-rw-r--r--src/vgstash_cli.py2
-rw-r--r--tests/test_vgstash_cli.py12
2 files changed, 12 insertions, 2 deletions
diff --git a/src/vgstash_cli.py b/src/vgstash_cli.py
index c8e8d24..7996086 100644
--- a/src/vgstash_cli.py
+++ b/src/vgstash_cli.py
@@ -145,6 +145,8 @@ def delete_game(title, system):
target_game = vgstash.Game(title, system)
if db.delete_game(target_game):
click.echo("Removed {} for {} from your collection.".format(title, system))
+ else:
+ click.echo("That game does not exist in your collection. Please try again.")
@cli.command('update')
diff --git a/tests/test_vgstash_cli.py b/tests/test_vgstash_cli.py
index 831fe80..c3b60d6 100644
--- a/tests/test_vgstash_cli.py
+++ b/tests/test_vgstash_cli.py
@@ -6,8 +6,6 @@ import vgstash_cli
from click.testing import CliRunner
-# TODO: Setup a testing directory for test data and integrate with pytest
-
verbose = True
interactive = False
@@ -99,6 +97,7 @@ def test_list_filter():
'The Legend of Zelda|NES|2|2|\n',
))
+
def test_list_filter_invalid():
runner = CliRunner()
result = runner.invoke(vgstash_cli.cli, ['list', '-r', 'derp'])
@@ -170,6 +169,15 @@ def test_delete():
assert result.output == "Removed Vectorman for Genesis from your collection.\n"
+def test_delete_invalid():
+ runner = CliRunner()
+ result = runner.invoke(vgstash_cli.cli, ['delete', 'Vectorman 3', 'Genesis'])
+ if verbose:
+ print(result.output)
+ assert result.exit_code == 0
+ assert result.output == "That game does not exist in your collection. Please try again.\n"
+
+
def test_update():
runner = CliRunner()
result = runner.invoke(vgstash_cli.cli, ['update', 'Super Mario Bros.', 'NES', 'progress', 'c'])
;follow=1'>cli: change "Status" heading to "Progress"zlg2-36/+40 2018-09-29Bump to 0.3alpha5 for PyPIzlg1-1/+1 2018-09-29cli: Add pretty printing to 'list' commandzlg3-17/+107 2018-09-08setup.py: Bump to alpha4 for PyPIzlg1-1/+1 2018-09-08cli: add '--raw' option to list commandzlg2-9/+45 2018-09-08Add remaining filters to vgstash packagezlg1-2/+11 2018-09-04Update LICENSE to match setup.pyzlg1-80/+67 2018-09-03Branch off from master with pytest, tox, clickzlg16-778/+779 2018-03-18Flesh out filter types and ownership statuszlg3-82/+144 2018-03-18README.mdown: break line correctlyzlg1-1/+1 2018-03-18add 'playlog' list filterzlg2-2/+9 2018-03-13Update helpers a bitzlg1-2/+9 2018-03-13Make VGSTASH_DB_LOCATION point to a filezlg2-21/+20 2016-11-18Remove settings from helpers.shZe Libertine Gamer1-5/+0 2016-11-15Correct phrasing in README.Ze Libertine Gamer1-4/+4 2016-11-13DerpZe Libertine Gamer1-0/+1 2016-11-03Improve error handling in shell scriptsZe Libertine Gamer4-3/+23 2016-10-24Correct run_again, add recursionZe Libertine Gamer1-0/+4 2016-10-21Add quotes to correct behavior for arglistZe Libertine Gamer1-1/+1 2016-10-14updater.sh: add recursion, error handlingZe Libertine Gamer1-43/+101 2016-10-14Correct pipe-handling behaviorZe Libertine Gamer1-1/+9 2016-10-12Clarify a method to move between platformsZe Libertine Gamer1-2/+5