nits

Author: Datata1

examples/scripts/create_workspace_with_landscape.py

@@ -11,7 +11,7 @@
 )
 from codesphere.resources.workspace.logs import LogStage
 
-TEAM_ID = 35698
+TEAM_ID = 123
 
 
 async def main():
@@ -29,7 +29,6 @@ async def main():
         )
         print(f"✓ Workspace created (ID: {workspace.id})")
 
-        # Get the team for usage history access
         team = await sdk.teams.get(team_id=TEAM_ID)
 
         print("Waiting for workspace to start...")
@@ -94,9 +93,6 @@ async def main():
 
         print(f"\n✓ Stream ended ({count} log entries)")
 
-        # ========================================
-        # Usage History Collection
-        # ========================================
         print("\n--- Usage History ---")
 
         end_date = datetime.now(timezone.utc)

tests/resources/team/usage/test_usage.py

@@ -14,7 +14,6 @@
 
 @pytest.fixture
 def sample_usage_summary_data():
-    """Sample response data for landscape service usage summary."""
     return {
         "totalItems": 3,
         "limit": 25,
@@ -230,7 +229,6 @@ def test_total_pages(self):
         assert response.total_pages == 4
 
     def test_total_pages_with_remainder(self):
-        """total_pages should round up when items don't divide evenly."""
         response = UsageSummaryResponse(
             total_items=101,
             limit=25,
@@ -243,10 +241,7 @@ def test_total_pages_with_remainder(self):
 
 
 class TestUsageSummaryResponse:
-    """Tests for the UsageSummaryResponse schema."""
-
     def test_parse_full_response(self, sample_usage_summary_data):
-        """Should correctly parse a full usage summary response."""
         response = UsageSummaryResponse.model_validate(sample_usage_summary_data)
 
         assert response.total_items == 3
@@ -256,18 +251,14 @@ def test_parse_full_response(self, sample_usage_summary_data):
         assert len(response.items) == 3
 
     def test_items_property(self, sample_usage_summary_data):
-        """items property should return the summary list."""
         response = UsageSummaryResponse.model_validate(sample_usage_summary_data)
 
         assert response.items is response.summary
         assert isinstance(response.items[0], LandscapeServiceSummary)
 
 
 class TestUsageEventsResponse:
-    """Tests for the UsageEventsResponse schema."""
-
     def test_parse_full_response(self, sample_usage_events_data):
-        """Should correctly parse a full usage events response."""
         response = UsageEventsResponse.model_validate(sample_usage_events_data)
 
         assert response.total_items == 4
@@ -277,19 +268,15 @@ def test_parse_full_response(self, sample_usage_events_data):
         assert len(response.items) == 4
 
     def test_items_property(self, sample_usage_events_data):
-        """items property should return the events list."""
         response = UsageEventsResponse.model_validate(sample_usage_events_data)
 
         assert response.items is response.events
         assert isinstance(response.items[0], LandscapeServiceEvent)
 
 
 class TestTeamUsageManager:
-    """Tests for the TeamUsageManager class."""
-
     @pytest.fixture
     def usage_manager(self, mock_http_client_for_resource, sample_usage_summary_data):
-        """Create a TeamUsageManager with mock HTTP client."""
         mock_client = mock_http_client_for_resource(sample_usage_summary_data)
         manager = TeamUsageManager(http_client=mock_client, team_id=12345)
         return manager, mock_client
@@ -298,7 +285,6 @@ def usage_manager(self, mock_http_client_for_resource, sample_usage_summary_data
     async def test_get_landscape_summary(
         self, mock_http_client_for_resource, sample_usage_summary_data
     ):
-        """get_landscape_summary should return UsageSummaryResponse."""
         mock_client = mock_http_client_for_resource(sample_usage_summary_data)
         manager = TeamUsageManager(http_client=mock_client, team_id=12345)
 
@@ -316,7 +302,6 @@ async def test_get_landscape_summary(
     async def test_get_landscape_summary_with_pagination(
         self, mock_http_client_for_resource, sample_usage_summary_data
     ):
-        """get_landscape_summary should pass pagination parameters."""
         mock_client = mock_http_client_for_resource(sample_usage_summary_data)
         manager = TeamUsageManager(http_client=mock_client, team_id=12345)
 
@@ -336,11 +321,9 @@ async def test_get_landscape_summary_with_pagination(
     async def test_get_landscape_summary_clamps_limit(
         self, mock_http_client_for_resource, sample_usage_summary_data
     ):
-        """get_landscape_summary should clamp limit to 1-100 range."""
         mock_client = mock_http_client_for_resource(sample_usage_summary_data)
         manager = TeamUsageManager(http_client=mock_client, team_id=12345)
 
-        # Test upper bound
         await manager.get_landscape_summary(
             begin_date=datetime(2024, 1, 1),
             end_date=datetime(2024, 1, 31),
@@ -353,7 +336,6 @@ async def test_get_landscape_summary_clamps_limit(
     async def test_get_landscape_events(
         self, mock_http_client_for_resource, sample_usage_events_data
     ):
-        """get_landscape_events should return UsageEventsResponse."""
         mock_client = mock_http_client_for_resource(sample_usage_events_data)
         manager = TeamUsageManager(http_client=mock_client, team_id=12345)
 
@@ -372,7 +354,6 @@ async def test_get_landscape_events(
     async def test_iter_all_landscape_summary(
         self, mock_http_client_for_resource, sample_usage_summary_data
     ):
-        """iter_all_landscape_summary should yield all items across pages."""
         mock_client = mock_http_client_for_resource(sample_usage_summary_data)
         manager = TeamUsageManager(http_client=mock_client, team_id=12345)
 
@@ -390,7 +371,6 @@ async def test_iter_all_landscape_summary(
     async def test_iter_all_landscape_events(
         self, mock_http_client_for_resource, sample_usage_events_data
     ):
-        """iter_all_landscape_events should yield all items across pages."""
         mock_client = mock_http_client_for_resource(sample_usage_events_data)
         manager = TeamUsageManager(http_client=mock_client, team_id=12345)
 
@@ -407,11 +387,8 @@ async def test_iter_all_landscape_events(
 
 
 class TestTeamUsageProperty:
-    """Tests for the Team.usage property."""
-
     @pytest.mark.asyncio
     async def test_team_has_usage_property(self, team_model_factory):
-        """Team model should have a usage property that returns TeamUsageManager."""
         team, _ = team_model_factory()
 
         usage_manager = team.usage