diff options
author | zlg <zlg@zlg.space> | 2018-10-18 16:36:39 -0700 |
---|---|---|
committer | zlg <zlg@zlg.space> | 2018-10-18 16:36:39 -0700 |
commit | 3ec2a2902caaf3e70deea650425339299af5d821 (patch) | |
tree | 158d1c8baf932bd05af7f5ac7878243358beb240 /tests/test_vgstash_cli.py | |
parent | cli: Add zero-game import/export messages (diff) | |
download | vgstash-3ec2a2902caaf3e70deea650425339299af5d821.tar.gz vgstash-3ec2a2902caaf3e70deea650425339299af5d821.tar.bz2 vgstash-3ec2a2902caaf3e70deea650425339299af5d821.tar.xz vgstash-3ec2a2902caaf3e70deea650425339299af5d821.zip |
Catch when an invalid list filter is passed
Before, vgstash.DB.list_games() would default to 'allgames' and silently
hide it when a filter wasn't found. This commit ensures that the vgstash
package and CLI both indicate when an invalid filter is passed to them:
* vgstash.DB.list_games() will return False on a failure to match;
* vgstash_cli uses Click's Choice object to enforce the constraint
Diffstat (limited to 'tests/test_vgstash_cli.py')
-rw-r--r-- | tests/test_vgstash_cli.py | 7 |
1 files changed, 7 insertions, 0 deletions
diff --git a/tests/test_vgstash_cli.py b/tests/test_vgstash_cli.py index 02f0d81..f34f2d1 100644 --- a/tests/test_vgstash_cli.py +++ b/tests/test_vgstash_cli.py @@ -99,6 +99,13 @@ 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']) + if verbose: + print(result.output) + assert result.exit_code != 0 + def test_list_pretty(): runner = CliRunner() |