Skip to content

Commit

Permalink
Set rows_per_page to zero by default
Browse files Browse the repository at this point in the history
Signed-off-by: Aurélien Bompard <[email protected]>
  • Loading branch information
abompard committed Jul 2, 2024
1 parent b2da683 commit 1278e62
Show file tree
Hide file tree
Showing 2 changed files with 3 additions and 2 deletions.
1 change: 1 addition & 0 deletions fedbadges/rules.py
Original file line number Diff line number Diff line change
Expand Up @@ -449,6 +449,7 @@ def _make_query(self, search_kwargs):
log.debug("Making datanommer query: %r", search_kwargs)
_search_kwargs = search_kwargs.copy()
_search_kwargs["defer"] = True
_search_kwargs.setdefault("rows_per_page", 0)
total, pages, query = datanommer.models.Message.grep(**_search_kwargs)
return total, pages, query

Expand Down
4 changes: 2 additions & 2 deletions tests/test_datanommer_counter.py
Original file line number Diff line number Diff line change
Expand Up @@ -64,7 +64,7 @@ def test_basic_datanommer(cache_configured):
result = counter.count(message, "dummy-user")
assert result == 42
grep.assert_called_once_with(
topics=["org.fedoraproject.dev.something.sometopic"], defer=True
topics=["org.fedoraproject.dev.something.sometopic"], defer=True, rows_per_page=0
)


Expand Down Expand Up @@ -118,4 +118,4 @@ def test_datanommer_with_lambda_filter(cache_configured):
grep.return_value = returned_count, 1, MockQuery(returned_count)
result = counter.count(message, "dummy-user")
assert result == returned_count
grep.assert_called_once_with(users=["lmacken"], defer=True)
grep.assert_called_once_with(users=["lmacken"], defer=True, rows_per_page=0)

0 comments on commit 1278e62

Please sign in to comment.