diff --git a/swh/search/journal_client.py b/swh/search/journal_client.py --- a/swh/search/journal_client.py +++ b/swh/search/journal_client.py @@ -48,7 +48,6 @@ if isinstance(visit["origin"], str) else visit["origin"]["url"] ), - "has_visits": True, } for visit in visits ] diff --git a/swh/search/tests/test_cli.py b/swh/search/tests/test_cli.py --- a/swh/search/tests/test_cli.py +++ b/swh/search/tests/test_cli.py @@ -147,14 +147,14 @@ swh_search.flush() - # Both search returns the visit actual_page = swh_search.origin_search(url_pattern="foobar", with_visit=False) assert actual_page.next_page_token is None assert actual_page.results == [origin_foobar] + # Not considered visited unless the visit is full actual_page = swh_search.origin_search(url_pattern="foobar", with_visit=True) assert actual_page.next_page_token is None - assert actual_page.results == [origin_foobar] + assert actual_page.results == [] def test__journal_client__origin_visit_status( diff --git a/swh/search/tests/test_journal_client.py b/swh/search/tests/test_journal_client.py --- a/swh/search/tests/test_journal_client.py +++ b/swh/search/tests/test_journal_client.py @@ -34,7 +34,7 @@ worker_fn({"origin_visit": [{"origin": {"url": "http://foobar.baz"},}]}) search_mock.origin_update.assert_called_once_with( - [{"url": "http://foobar.baz", "has_visits": True},] + [{"url": "http://foobar.baz"},] )