fz0x1/jrnl
1
0
Fork 0
mirror of https://github.com/jrnl-org/jrnl.git synced 2025-05-10 16:48:31 +02:00
Code Issues Projects Releases Packages Wiki Activity Actions

Fix unit test that did not account for new short-circuit when displaying journal with no entries

Browse source
This commit is contained in:
Micah Jerome Ellison 2023-02-18 15:06:23 -08:00 • committed by Jonathan Wren
parent 46b7a64549
commit 8ee1632760
2 changed files with 6 additions and 2 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

2
jrnl/controller.py
View file

@ -403,7 +403,7 @@ def _change_time_search_results(
def _display_search_results(args: "Namespace", journal: "Journal", **kwargs) -> None: def _display_search_results(args: "Namespace", journal: "Journal", **kwargs) -> None:
if len(journal.entries) == 0: if len(journal) == 0:
return return
# Get export format from config file if not provided at the command line # Get export format from config file if not provided at the command line

6
tests/unit/test_controller.py
View file

@ -20,7 +20,11 @@ def random_string():
@pytest.mark.parametrize("export_format", ["pretty", "short"]) @pytest.mark.parametrize("export_format", ["pretty", "short"])
def test_display_search_results_pretty_short(export_format): def test_display_search_results_pretty_short(export_format):
mock_args = parse_args(["--format", export_format]) mock_args = parse_args(["--format", export_format])
test_journal = mock.Mock(wraps=jrnl.journals.Journal('default'))
test_journal = jrnl.journals.Journal()
test_journal.new_entry("asdf")
test_journal.pprint = mock.Mock()
_display_search_results(mock_args, test_journal) _display_search_results(mock_args, test_journal)