diff --git a/moto/logs/logs_query/query_parser.py b/moto/logs/logs_query/query_parser.py index 7d325567a41f..8d3cbef71fa2 100644 --- a/moto/logs/logs_query/query_parser.py +++ b/moto/logs/logs_query/query_parser.py @@ -12,9 +12,9 @@ def __init__(self) -> None: def sort_reversed(self) -> bool: # Descending is the default if self.sort: - # sort_reversed is True if we want to sort in ascending order - return self.sort[-1][-1] == "asc" - return False + # sorted(..., reverse=True) gives descending order + return self.sort[-1][-1] == "desc" + return True def parse_query(query: str) -> ParsedQuery: diff --git a/tests/test_logs/test_logs_query/test_query.py b/tests/test_logs/test_logs_query/test_query.py index 20e28e82122c..6fb5a9199a1c 100644 --- a/tests/test_logs/test_logs_query/test_query.py +++ b/tests/test_logs/test_logs_query/test_query.py @@ -44,7 +44,7 @@ def test_default_query(self): "@logStream": self.stream_1_name, "@log": "test", } - for event in self.events + for event in reversed(self.events) ] def test_simplified_query(self): @@ -58,5 +58,5 @@ def test_simplified_query(self): event.pop("@ptr") assert resp == [ {"@timestamp": event["timestamp"], "@message": event["message"]} - for event in reversed(self.events) + for event in self.events ]