add --all to "get leases"

this indicates to the server that we want to see ended leases as well.
Default behavior is to only get non-ended leases (pending or currently active)

(cherry picked from commit 6d94924)

Author: michalskrivanek

packages/jumpstarter-cli/jumpstarter_cli/get.py

@@ -41,12 +41,19 @@ def get_exporters(config, selector: str | None, output: OutputType, with_options
 @opt_config(exporter=False)
 @opt_selector
 @opt_output_all
+@click.option(
+    "--all",
+    "show_all",
+    is_flag=True,
+    default=False,
+    help="Show all leases including expired ones"
+)
 @handle_exceptions_with_reauthentication(relogin_client)
-def get_leases(config, selector: str | None, output: OutputType):
+def get_leases(config, selector: str | None, output: OutputType, show_all: bool):
     """
     Display one or many leases
     """
 
-    leases = config.list_leases(filter=selector)
+    leases = config.list_leases(filter=selector, only_active=not show_all)
 
     model_print(leases, output)

packages/jumpstarter-cli/jumpstarter_cli/get_test.py

@@ -1,10 +1,11 @@
+from datetime import datetime, timedelta
 from unittest.mock import Mock
 
 import click
 import pytest
 from jumpstarter_cli_common.opt import parse_comma_separated
 
-from jumpstarter.client.grpc import Exporter, ExporterList, Lease
+from jumpstarter.client.grpc import Exporter, ExporterList, Lease, LeaseList
 from jumpstarter.config.client import ClientConfigV1Alpha1
 
 
@@ -239,3 +240,109 @@ def test_exporter_to_exporter_list_flow(self):
         assert exporter_list.exporters[1].name == "server-001"
         assert exporter_list.include_online is True
         assert exporter_list.include_leases is False
+
+
+class TestGetLeasesLogic:
+    """Tests for get leases command logic (simulating server-side filtering)"""
+
+    def create_test_lease(self, namespace="default", name="lease-1", status="In-Use",
+                          effective_begin_time=None, effective_end_time=None,
+                          duration=timedelta(hours=1)):
+        """Create a mock lease for testing"""
+        lease = Mock(spec=Lease)
+        lease.namespace = namespace
+        lease.name = name
+        lease.client = "test-client"
+        lease.exporter = "test-exporter"
+        lease.get_status.return_value = status
+        lease.effective_begin_time = effective_begin_time
+        lease.effective_end_time = effective_end_time
+        lease.duration = duration
+        lease.effective_duration = timedelta(minutes=30) if effective_begin_time else None
+        lease.begin_time = None
+        return lease
+
+    def test_only_active_excludes_expired_leases(self):
+        """Test that server returns only active leases when only_active=True"""
+        # When only_active=True, server returns only active lease
+        active_lease = self.create_test_lease(
+            name="active-lease",
+            status="In-Use",
+            effective_begin_time=datetime(2023, 1, 1, 10, 0, 0)
+        )
+
+        leases_from_server = LeaseList(leases=[active_lease], next_page_token=None)
+
+        assert len(leases_from_server.leases) == 1
+        assert leases_from_server.leases[0].name == "active-lease"
+        assert leases_from_server.leases[0].get_status() == "In-Use"
+
+    def test_show_all_includes_expired_leases(self):
+        """Test that server returns all leases including expired when only_active=False"""
+        # When only_active=False, server returns both active and expired
+        active_lease = self.create_test_lease(
+            name="active-lease",
+            status="In-Use",
+            effective_begin_time=datetime(2023, 1, 1, 10, 0, 0)
+        )
+        expired_lease = self.create_test_lease(
+            name="expired-lease",
+            status="Expired",
+            effective_begin_time=datetime(2023, 1, 1, 8, 0, 0),
+            effective_end_time=datetime(2023, 1, 1, 9, 0, 0)
+        )
+
+        leases_from_server = LeaseList(leases=[active_lease, expired_lease], next_page_token=None)
+
+        assert len(leases_from_server.leases) == 2
+        assert leases_from_server.leases[0].name == "active-lease"
+        assert leases_from_server.leases[1].name == "expired-lease"
+
+    def test_multiple_active_leases_returned(self):
+        """Test that server returns all active leases when only_active=True"""
+        # Server returns multiple active leases (different statuses but all non-expired)
+        lease1 = self.create_test_lease(
+            name="lease-1",
+            status="In-Use",
+            effective_begin_time=datetime(2023, 1, 1, 10, 0, 0)
+        )
+        lease2 = self.create_test_lease(
+            name="lease-2",
+            status="Waiting",
+            effective_begin_time=datetime(2023, 1, 1, 11, 0, 0)
+        )
+        lease3 = self.create_test_lease(
+            name="lease-3",
+            status="In-Use",
+            effective_begin_time=datetime(2023, 1, 1, 12, 0, 0)
+        )
+
+        leases_from_server = LeaseList(leases=[lease1, lease2, lease3], next_page_token=None)
+
+        assert len(leases_from_server.leases) == 3
+        assert all(lease.get_status() != "Expired" for lease in leases_from_server.leases)
+
+    def test_all_expired_when_show_all(self):
+        """Test that server can return only expired leases when only_active=False"""
+        # When only_active=False and all leases happen to be expired
+        expired1 = self.create_test_lease(
+            name="expired-1",
+            status="Expired",
+            effective_end_time=datetime(2023, 1, 1, 8, 0, 0)
+        )
+        expired2 = self.create_test_lease(
+            name="expired-2",
+            status="Expired",
+            effective_end_time=datetime(2023, 1, 1, 9, 0, 0)
+        )
+
+        leases_from_server = LeaseList(leases=[expired1, expired2], next_page_token=None)
+
+        assert len(leases_from_server.leases) == 2
+        assert all(lease.get_status() == "Expired" for lease in leases_from_server.leases)
+
+    def test_empty_lease_list(self):
+        """Test that server can return empty lease list"""
+        leases_from_server = LeaseList(leases=[], next_page_token=None)
+
+        assert len(leases_from_server.leases) == 0

packages/jumpstarter/jumpstarter/client/grpc.py

@@ -370,6 +370,7 @@ async def ListLeases(
         page_size: int | None = None,
         page_token: str | None = None,
         filter: str | None = None,
+        only_active: bool = True,
     ):
         with translate_grpc_exceptions():
             leases = await self.stub.ListLeases(
@@ -378,6 +379,7 @@ async def ListLeases(
                     page_size=page_size,
                     page_token=page_token,
                     filter=filter,
+                    only_active=only_active,
                 )
             )
         return LeaseList.from_protobuf(leases)

packages/jumpstarter/jumpstarter/config/client.py

@@ -227,12 +227,14 @@ async def list_leases(
         page_size: int | None = None,
         page_token: str | None = None,
         filter: str | None = None,
+        only_active: bool = True,
     ):
         svc = ClientService(channel=await self.channel(), namespace=self.metadata.namespace)
         return await svc.ListLeases(
             page_size=page_size,
             page_token=page_token,
             filter=filter,
+            only_active=only_active,
         )
 
     @_blocking_compat