EPA-PM-API-Wrapper 0.3.1__tar.gz → 0.3.3__tar.gz

epa_pm_api_wrapper-0.3.3/CHANGELOG/0.3.2.md

@@ -0,0 +1,20 @@
+# 0.3.2 (2026-05-07)
+
+### Fixed
+
+- **`consumption_data.add()`:** declared `response_type=Response`, but the EPA returns the created records inside a `<meterData>` envelope (with server-assigned `<id>` populated). Every successful POST raised `xsdata.exceptions.ParserError: Unknown property response:meterConsumption` even though the records were stored. The method now declares `response_type=MeterData` and returns the created records, so callers can read assigned IDs without a follow-up GET. Sync + async variants kept symmetric.
+- **`waste_data.add()`:** same bug, parallel fix. The EPA's published `meter/wasteData/post` docs return `<wasteDataList>` containing the created `<wasteData>` record(s). The method now declares `response_type=WasteDataList` and returns the created records with their IDs.
+
+### Documentation
+
+- Reworked `docs/api.md` from a single `::: epa_pm_api_wrapper` directive into a sectioned reference covering clients, exceptions, retry config, and one entry per resource module so the attribute-namespaced API surface (`client.account`, `client.property`, `client.meter`, …) actually renders in the published docs.
+- Tightened the `docs/usage.md` bulk-import recipe to read assigned IDs from the returned `MeterData` instead of the (now-nonexistent) `response.status`.
+- Moved the 2026 EPA Portfolio Manager XSD snapshot from `docs/_xml-schema/` to `schemas/` (it is a code-generation input, not docs content) and added a `CONTRIBUTING.md` note explaining provenance and refresh procedure.
+
+### Caller impact
+
+Any caller that swallowed the `ParserError` from `consumption_data.add()` or `waste_data.add()` to work around the bug should remove that workaround. Callers that branched on `Response.status` from these methods need to switch to checking the returned `MeterData` / `WasteDataList` (or simply rely on absence of an exception, which is the more idiomatic pattern).
+
+### Contributors
+
+[@harryw1](https://github.com/harryw1) (Harrison Weiss)

epa_pm_api_wrapper-0.3.3/CHANGELOG/0.3.3.md

@@ -0,0 +1,21 @@
+# 0.3.3 (2026-05-07)
+
+Two waste-meter bugs surfaced by round-trip testing against `wstest` after the 0.3.2 release.
+
+### Fixed
+
+- **`waste_data.add()`** — sent a bare `<wasteData>` request body, which the EPA rejects with HTTP 400. The endpoint requires records wrapped in `<wasteDataList>` (mirroring the `<meterData>` envelope used by `consumption_data.add`). 0.3.2 fixed the response parsing but the request shape was still broken upstream of it, so `waste_data.add()` never worked against the live API. The method now takes a `WasteDataList` and supports up to 120 records per call. The regression test additionally asserts the serialized request body uses `<wasteDataList>` as the root element so this can't regress silently.
+- **`meter.get()`** — `GET /meter/{id}` returns either `<meter>` (energy, water, etc.) or `<wasteMeter>` (trash, recycling, …) depending on the meter type, with disjoint child schemas. The method previously always parsed as `Meter`, so any waste-meter response raised `xsdata.exceptions.ParserError: Unknown property meter:dataEntryMethod`. It now dispatches via `_request_polymorphic` and returns `Meter | WasteMeter`; disambiguate with `isinstance`. Sync + async kept symmetric. The CLI's `get-meter` command prints the appropriate fields for each type.
+
+### Caller impact
+
+- **Breaking (signature only — runtime never worked):** `waste_data.add()` callers passing a single `WasteData` need to wrap it as `WasteDataList(waste_data=[record])`.
+- **Soft change:** `meter.get()` now returns `Meter | WasteMeter` instead of `Meter`. Callers that already only worked with energy meters keep working at runtime; type checkers will flag attribute access on the union and prompt for an `isinstance` narrow.
+
+### Known issues still open
+
+- **`property_use.use_details.revisions()`** raises `ParserError` because the response body is a polymorphic list of use-detail subtypes inside a `<response>` envelope. The wrapper has single-element polymorphic dispatch but no list equivalent yet. Tracked for a follow-up release.
+
+### Contributors
+
+[@harryw1](https://github.com/harryw1) (Harrison Weiss)

{epa_pm_api_wrapper-0.3.1 → epa_pm_api_wrapper-0.3.3}/CONTRIBUTING.md

@@ -99,6 +99,15 @@ Ready to contribute? Here's how to set up EPA-PM-API-Wrapper for local developme
 
 7. Submit a pull request through the GitHub website.
 
+## Generated models
+
+The dataclasses in `src/epa_pm_api_wrapper/models/` are generated from the
+2026 EPA Portfolio Manager XSD snapshot vendored under `schemas/` (rooted
+at `schemas/main.xsd`) using [`xsdata`](https://xsdata.readthedocs.io/).
+If the EPA publishes an updated schema, refresh `schemas/` from
+[`portfoliomanager.energystar.gov/webservices`](https://portfoliomanager.energystar.gov/webservices)
+and regenerate before editing the generated modules by hand.
+
 ## Pull Request Guidelines
 
 Before you submit a pull request, check that it meets these guidelines:

{epa_pm_api_wrapper-0.3.1 → epa_pm_api_wrapper-0.3.3}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: EPA-PM-API-Wrapper
-Version: 0.3.1
+Version: 0.3.3
 Summary: Client wrapper for the EPA Portfolio Manager API.
 Project-URL: bugs, https://github.com/harryw1/EPA-PM-API-Wrapper/issues
 Project-URL: changelog, https://github.com/harryw1/EPA-PM-API-Wrapper/releases

epa_pm_api_wrapper-0.3.3/docs/api.md

@@ -0,0 +1,104 @@
+# API Reference
+
+The public surface is the items listed below, plus everything exported from
+`epa_pm_api_wrapper.__all__`. Each resource is bound to `EpaPmClient` and
+`EpaPmAsyncClient` under an attribute matching the snake-cased name (e.g.
+the methods documented on `AccountResource` are reached via
+`client.account.<method>`); async resources expose the same methods as
+coroutines.
+
+## Clients
+
+::: epa_pm_api_wrapper.client.EpaPmClient
+
+::: epa_pm_api_wrapper.client.EpaPmAsyncClient
+
+## Exceptions
+
+::: epa_pm_api_wrapper.exceptions
+
+## Retry configuration
+
+::: epa_pm_api_wrapper.retry.RetryConfig
+
+## Resources
+
+### Account — `client.account`
+
+::: epa_pm_api_wrapper.resources.account
+
+### Association — `client.association`
+
+::: epa_pm_api_wrapper.resources.association
+
+### Building — `client.building`
+
+::: epa_pm_api_wrapper.resources.building
+
+### Consumption data — `client.consumption_data`
+
+::: epa_pm_api_wrapper.resources.consumption_data
+
+### Custom metric — `client.custom_metric`
+
+::: epa_pm_api_wrapper.resources.custom_metric
+
+### Customer — `client.customer`
+
+::: epa_pm_api_wrapper.resources.customer
+
+### Data exchange — `client.data_exchange`
+
+::: epa_pm_api_wrapper.resources.data_exchange
+
+### Green power — `client.green_power`
+
+::: epa_pm_api_wrapper.resources.green_power
+
+### Home Energy Yardstick — `client.hey`
+
+::: epa_pm_api_wrapper.resources.hey
+
+### Hierarchy — `client.hierarchy`
+
+::: epa_pm_api_wrapper.resources.hierarchy
+
+### Identifier — `client.identifier`
+
+::: epa_pm_api_wrapper.resources.identifier
+
+### Meter — `client.meter`
+
+::: epa_pm_api_wrapper.resources.meter
+
+### Metrics — `client.metrics`
+
+::: epa_pm_api_wrapper.resources.metrics
+
+### Property — `client.property`
+
+::: epa_pm_api_wrapper.resources.property
+
+### Property use — `client.property_use`
+
+::: epa_pm_api_wrapper.resources.property_use
+
+### Report — `client.report`
+
+::: epa_pm_api_wrapper.resources.report
+
+### Share — `client.share`
+
+::: epa_pm_api_wrapper.resources.share
+
+### Target finder — `client.target_finder`
+
+::: epa_pm_api_wrapper.resources.target_finder
+
+### Verification — `client.verification`
+
+::: epa_pm_api_wrapper.resources.verification
+
+### Waste data — `client.waste_data`
+
+::: epa_pm_api_wrapper.resources.waste_data

{epa_pm_api_wrapper-0.3.1 → epa_pm_api_wrapper-0.3.3}/docs/usage.md

@@ -228,8 +228,10 @@ with EpaPmClient(username, password) as client:
             # …more records up to 120 per call…
         ],
     )
-    response = client.consumption_data.add(meter_id, payload)
-    print(response.status)
+    created = client.consumption_data.add(meter_id, payload)
+    # The EPA echoes back each record with its server-assigned id populated.
+    for record in created.meter_consumption or []:
+        print(record.id, record.start_date, record.usage)
 ```
 
 ### Poll a report to completion
@@ -302,5 +304,7 @@ type is the concrete subtype.
 
 Every endpoint in the EPA Portfolio Manager web services API is bound to
 either a method or a sub-namespace on the client. See
-[the API reference](api.md) for the full surface, generated from the
-docstrings on each resource class.
+[the API reference](api.md) for the full surface — clients, exceptions,
+retry config, and one section per resource (`client.account`,
+`client.property`, `client.meter`, …) generated from the docstrings on
+each resource class.

{epa_pm_api_wrapper-0.3.1 → epa_pm_api_wrapper-0.3.3}/justfile

@@ -99,7 +99,11 @@ clean-test:
 	rm -fr htmlcov/
 	rm -fr .pytest_cache
 
-# Publish to PyPI (manual alternative to GitHub Actions)
+# Publish to PyPI (manual alternative to GitHub Actions). Sources
+# ~/.zshenv.local so UV_PUBLISH_TOKEN (or equivalent) is available.
 publish:
+    #!/usr/bin/env zsh
+    set -euo pipefail
+    [[ -f ~/.zshenv.local ]] && source ~/.zshenv.local
     uv build
     uv publish

{epa_pm_api_wrapper-0.3.1 → epa_pm_api_wrapper-0.3.3}/pyproject.toml

@@ -4,7 +4,7 @@ build-backend = "hatchling.build"
 
 [project]
 name = "EPA-PM-API-Wrapper"
-version = "0.3.1"
+version = "0.3.3"
 description = "Client wrapper for the EPA Portfolio Manager API."
 readme = "README.md"
 authors = [

{epa_pm_api_wrapper-0.3.1 → epa_pm_api_wrapper-0.3.3}/src/epa_pm_api_wrapper/cli.py

@@ -10,7 +10,7 @@ Each command takes ``--username`` / ``--password`` / ``--test`` (with matching
 variables, also picked up from a `.env` file in the working directory).
 """
 
-from typing import Annotated, Optional
+from typing import Annotated
 
 import typer
 from dotenv import load_dotenv
@@ -18,6 +18,7 @@ from rich.console import Console
 from rich.table import Table
 
 from epa_pm_api_wrapper.client import EpaPmClient
+from epa_pm_api_wrapper.models import WasteMeter
 
 # Load environment variables from .env file, allowing them to override existing env vars
 load_dotenv(override=True)
@@ -141,10 +142,15 @@ def get_meter(
         console.print("[bold green]Meter Details:[/bold green]")
         console.print(f"ID: {meter.id}")
         console.print(f"Name: {meter.name}")
-        console.print(f"Type: {meter.type_value}")
-        console.print(f"Unit: {meter.unit_of_measure}")
-        console.print(f"Metered: {meter.metered}")
-        console.print(f"In Use: {meter.in_use}")
+        if isinstance(meter, WasteMeter):
+            console.print(f"Type: {meter.type_value} (waste)")
+            console.print(f"Unit: {meter.unit_of_measure}")
+            console.print(f"In Use: {meter.in_use}")
+        else:
+            console.print(f"Type: {meter.type_value}")
+            console.print(f"Unit: {meter.unit_of_measure}")
+            console.print(f"Metered: {meter.metered}")
+            console.print(f"In Use: {meter.in_use}")
     except Exception as e:
         console.print(f"[bold red]Error:[/bold red] {e}")
 
@@ -155,8 +161,8 @@ def get_consumption(
     username: Username,
     password: Password,
     page: Annotated[int, typer.Option(help="1-based page number (120 records per page)")] = 1,
-    start_date: Annotated[Optional[str], typer.Option(help="ISO 8601 start date (YYYY-MM-DD)")] = None,
-    end_date: Annotated[Optional[str], typer.Option(help="ISO 8601 end date (YYYY-MM-DD)")] = None,
+    start_date: Annotated[str | None, typer.Option(help="ISO 8601 start date (YYYY-MM-DD)")] = None,
+    end_date: Annotated[str | None, typer.Option(help="ISO 8601 end date (YYYY-MM-DD)")] = None,
     test: Test = False,
 ) -> None:
     """Get consumption records for a meter."""

{epa_pm_api_wrapper-0.3.1 → epa_pm_api_wrapper-0.3.3}/src/epa_pm_api_wrapper/resources/consumption_data.py

@@ -26,9 +26,15 @@ class ConsumptionDataResource(Resource):
         params = {"page": page, "startDate": start_date, "endDate": end_date}
         return self._client._request("GET", f"meter/{meter_id}/consumptionData", params=params, response_type=MeterData)
 
-    def add(self, meter_id: int, data: MeterData) -> Response:
-        """Add consumption data records to a meter."""
-        return self._client._request("POST", f"meter/{meter_id}/consumptionData", data=data, response_type=Response)
+    def add(self, meter_id: int, data: MeterData) -> MeterData:
+        """Add consumption data records to a meter.
+
+        Returns the created records with their server-assigned IDs populated;
+        callers can read ``meter_consumption[i].id`` to track them without a
+        follow-up GET. The EPA responds with a ``<meterData>`` envelope here,
+        not the usual ``<response>``.
+        """
+        return self._client._request("POST", f"meter/{meter_id}/consumptionData", data=data, response_type=MeterData)
 
     def update(self, consumption_data_id: int, data: MeterConsumption) -> Response:
         """Update a single consumption data record."""
@@ -95,9 +101,9 @@ class AsyncConsumptionDataResource(AsyncResource):
             "GET", f"meter/{meter_id}/consumptionData", params=params, response_type=MeterData
         )
 
-    async def add(self, meter_id: int, data: MeterData) -> Response:
+    async def add(self, meter_id: int, data: MeterData) -> MeterData:
         return await self._client._arequest(
-            "POST", f"meter/{meter_id}/consumptionData", data=data, response_type=Response
+            "POST", f"meter/{meter_id}/consumptionData", data=data, response_type=MeterData
         )
 
     async def update(self, consumption_data_id: int, data: MeterConsumption) -> Response:

{epa_pm_api_wrapper-0.3.1 → epa_pm_api_wrapper-0.3.3}/src/epa_pm_api_wrapper/resources/meter.py

@@ -2,7 +2,7 @@
 
 from __future__ import annotations
 
-from typing import TYPE_CHECKING
+from typing import TYPE_CHECKING, cast
 
 from epa_pm_api_wrapper.models import (
     AdditionalIdentifier,
@@ -12,6 +12,7 @@ from epa_pm_api_wrapper.models import (
     IndividualMeter,
     Meter,
     Response,
+    WasteMeter,
 )
 
 from ._base import AsyncResource, Resource
@@ -118,9 +119,21 @@ class MeterResource(Resource):
         params = {"myAccessOnly": "true" if my_access_only else "false"} if my_access_only is not None else None
         return self._client._request("GET", f"property/{property_id}/meter/list", params=params, response_type=Response)
 
-    def get(self, meter_id: int) -> Meter:
-        """Get details for a specific meter."""
-        return self._client._request("GET", f"meter/{meter_id}", response_type=Meter)
+    def get(self, meter_id: int) -> Meter | WasteMeter:
+        """Get details for a specific meter (energy or waste).
+
+        The EPA returns either ``<meter>`` (energy/water/etc.) or
+        ``<wasteMeter>`` (trash, recycling, …) depending on the meter
+        type, with disjoint child schemas. The client auto-resolves to
+        the matching dataclass; use ``isinstance`` to disambiguate::
+
+            m = client.meter.get(meter_id)
+            if isinstance(m, WasteMeter):
+                ...
+            else:
+                ...  # energy meter — Meter
+        """
+        return cast(Meter | WasteMeter, self._client._request_polymorphic("GET", f"meter/{meter_id}"))
 
     def create(self, property_id: int, meter_data: Meter) -> Response:
         """Add a new meter under a property."""
@@ -242,8 +255,8 @@ class AsyncMeterResource(AsyncResource):
             "GET", f"property/{property_id}/meter/list", params=params, response_type=Response
         )
 
-    async def get(self, meter_id: int) -> Meter:
-        return await self._client._arequest("GET", f"meter/{meter_id}", response_type=Meter)
+    async def get(self, meter_id: int) -> Meter | WasteMeter:
+        return cast(Meter | WasteMeter, await self._client._arequest_polymorphic("GET", f"meter/{meter_id}"))
 
     async def create(self, property_id: int, meter_data: Meter) -> Response:
         return await self._client._arequest(

{epa_pm_api_wrapper-0.3.1 → epa_pm_api_wrapper-0.3.3}/src/epa_pm_api_wrapper/resources/waste_data.py

@@ -21,9 +21,19 @@ class WasteDataResource(Resource):
         params = {"page": page, "startDate": start_date, "endDate": end_date}
         return self._client._request("GET", f"meter/{meter_id}/wasteData", params=params, response_type=WasteDataList)
 
-    def add(self, meter_id: int, data: WasteData) -> Response:
-        """Add a waste data record to a meter."""
-        return self._client._request("POST", f"meter/{meter_id}/wasteData", data=data, response_type=Response)
+    def add(self, meter_id: int, data: WasteDataList) -> WasteDataList:
+        """Add one or more waste data records to a meter (up to 120 per call).
+
+        The EPA requires the request body to be wrapped in ``<wasteDataList>``
+        (mirroring the ``<meterData>`` envelope used by consumption_data.add);
+        a bare ``<wasteData>`` element is rejected with HTTP 400. Callers
+        wrap a single record as ``WasteDataList(waste_data=[record])``.
+
+        Returns a ``<wasteDataList>`` echoing the created records with their
+        server-assigned ``id`` populated; read ``result.waste_data[i].id``
+        to track them without a follow-up GET.
+        """
+        return self._client._request("POST", f"meter/{meter_id}/wasteData", data=data, response_type=WasteDataList)
 
     def delete_range(self, meter_id: int, start_date: str, end_date: str) -> Response:
         """Delete waste data records on a meter within a date range."""
@@ -81,8 +91,10 @@ class AsyncWasteDataResource(AsyncResource):
             "GET", f"meter/{meter_id}/wasteData", params=params, response_type=WasteDataList
         )
 
-    async def add(self, meter_id: int, data: WasteData) -> Response:
-        return await self._client._arequest("POST", f"meter/{meter_id}/wasteData", data=data, response_type=Response)
+    async def add(self, meter_id: int, data: WasteDataList) -> WasteDataList:
+        return await self._client._arequest(
+            "POST", f"meter/{meter_id}/wasteData", data=data, response_type=WasteDataList
+        )
 
     async def delete_range(self, meter_id: int, start_date: str, end_date: str) -> Response:
         return await self._client._arequest(

{epa_pm_api_wrapper-0.3.1 → epa_pm_api_wrapper-0.3.3}/tests/test_client.py

@@ -249,6 +249,38 @@ def test_get_meter():
     assert meter.name == "Main Electric"
 
 
+@respx.mock
+def test_get_meter_resolves_waste_meter_polymorphically():
+    """Regression: GET /meter/{id} can return <wasteMeter>, not just <meter>.
+
+    Previously meter.get() always parsed as Meter, so any waste meter
+    response raised ParserError. The client now auto-resolves to the
+    matching dataclass.
+    """
+    from epa_pm_api_wrapper.models import Meter, WasteMeter
+
+    waste_xml = """<?xml version="1.0" encoding="UTF-8"?>
+<wasteMeter>
+    <id>456</id>
+    <name>Trash dumpster</name>
+    <type>Disposed - Batteries</type>
+    <unitOfMeasure>tons</unitOfMeasure>
+    <dataEntryMethod>regular</dataEntryMethod>
+    <firstBillDate>2024-01-01</firstBillDate>
+    <inUse>true</inUse>
+</wasteMeter>"""
+
+    respx.get(f"{BASE}/meter/456").mock(return_value=httpx.Response(200, content=waste_xml))
+
+    client = EpaPmClient("user", "pass")
+    meter = client.meter.get(456)
+
+    assert isinstance(meter, WasteMeter)
+    assert not isinstance(meter, Meter)
+    assert meter.id == 456
+    assert meter.name == "Trash dumpster"
+
+
 @respx.mock
 def test_list_meters_with_my_access_only():
     list_xml = """<?xml version="1.0" encoding="UTF-8"?>

{epa_pm_api_wrapper-0.3.1 → epa_pm_api_wrapper-0.3.3}/tests/test_resources/test_consumption_data.py

@@ -7,7 +7,7 @@ import respx
 from xsdata.models.datatype import XmlDate
 
 from epa_pm_api_wrapper.client import EpaPmClient
-from epa_pm_api_wrapper.models import MeterConsumption, StatusType
+from epa_pm_api_wrapper.models import MeterConsumption, MeterData, StatusType
 
 BASE = EpaPmClient.LIVE_BASE_URL
 
@@ -54,6 +54,50 @@ def test_delete_range_passes_dates_as_query():
     assert params.get("endDate") == "2024-06-30"
 
 
+@respx.mock
+def test_add_returns_meter_data_with_assigned_ids():
+    """Regression: EPA returns <meterData> (not <response>) on POST.
+
+    Captured from the wstest environment; see BUGS.md #1 in the test repo.
+    Asserts the response parses as MeterData and surfaces server-assigned IDs.
+    """
+    add_xml = """<?xml version="1.0" encoding="UTF-8" standalone="yes"?>
+<meterData>
+  <meterConsumption estimatedValue="false">
+    <id>900238341</id>
+    <cost>250</cost>
+    <startDate>2025-01-01</startDate>
+    <endDate>2025-01-31</endDate>
+    <usage>1234.56</usage>
+  </meterConsumption>
+  <meterConsumption estimatedValue="true">
+    <id>900238342</id>
+    <startDate>2025-02-01</startDate>
+    <endDate>2025-02-28</endDate>
+    <usage>2000</usage>
+  </meterConsumption>
+</meterData>"""
+
+    route = respx.post(f"{BASE}/meter/123/consumptionData").mock(return_value=httpx.Response(201, content=add_xml))
+
+    client = EpaPmClient("user", "pass")
+    payload = MeterData(
+        meter_consumption=[
+            MeterConsumption(
+                start_date=XmlDate(2025, 1, 1),
+                end_date=XmlDate(2025, 1, 31),
+                usage=Decimal("1234.56"),
+            ),
+        ],
+    )
+    result = client.consumption_data.add(123, payload)
+
+    assert route.called
+    assert isinstance(result, MeterData)
+    assert result.meter_consumption is not None
+    assert [r.id for r in result.meter_consumption] == [900238341, 900238342]
+
+
 @respx.mock
 def test_what_changed_for_customer():
     route = respx.get(f"{BASE}/customer/501362/meter/consumptionData/whatChanged").mock(

{epa_pm_api_wrapper-0.3.1 → epa_pm_api_wrapper-0.3.3}/tests/test_resources/test_waste_data.py

@@ -1,8 +1,11 @@
+from decimal import Decimal
+
 import httpx
 import respx
+from xsdata.models.datatype import XmlDate
 
 from epa_pm_api_wrapper.client import EpaPmClient
-from epa_pm_api_wrapper.models import StatusType
+from epa_pm_api_wrapper.models import StatusType, WasteData, WasteDataList
 
 BASE = EpaPmClient.LIVE_BASE_URL
 
@@ -32,6 +35,56 @@ def test_list_with_date_range():
     assert len(data.waste_data) == 1
 
 
+@respx.mock
+def test_add_returns_waste_data_list_with_assigned_id():
+    """Regression: EPA returns <wasteDataList> (not <response>) on POST.
+
+    Body shape captured from the EPA's published wasteData/post endpoint
+    docs; mirrors the consumption_data.add() fix. Asserts the response
+    parses as WasteDataList and the created record's id is surfaced.
+    """
+    add_xml = """<?xml version="1.0" encoding="UTF-8"?>
+<wasteDataList>
+    <wasteData estimatedValue="true">
+        <id>783569362</id>
+        <startDate>2013-01-01</startDate>
+        <endDate>2013-01-31</endDate>
+        <quantity>50</quantity>
+        <cost>100</cost>
+        <disposalDestination>
+            <incinerationPercentage>25</incinerationPercentage>
+            <landfillPercentage>25</landfillPercentage>
+            <unknownDestPercentage>25</unknownDestPercentage>
+            <wasteToEnergyPercentage>25</wasteToEnergyPercentage>
+        </disposalDestination>
+    </wasteData>
+</wasteDataList>"""
+
+    route = respx.post(f"{BASE}/meter/123/wasteData").mock(return_value=httpx.Response(201, content=add_xml))
+
+    client = EpaPmClient("user", "pass")
+    payload = WasteDataList(
+        waste_data=[
+            WasteData(
+                start_date=XmlDate(2013, 1, 1),
+                end_date=XmlDate(2013, 1, 31),
+                quantity=Decimal("50"),
+            ),
+        ],
+    )
+    result = client.waste_data.add(123, payload)
+
+    assert route.called
+    # Request must use <wasteDataList> as root, not bare <wasteData>
+    # (EPA rejects the bare form with HTTP 400). Mirrors consumption_data.add.
+    sent_body = route.calls[0].request.content.decode()
+    assert "<wasteDataList" in sent_body
+    assert "<wasteData" in sent_body
+    assert isinstance(result, WasteDataList)
+    assert result.waste_data is not None
+    assert result.waste_data[0].id == 783569362
+
+
 @respx.mock
 def test_delete_range_passes_dates_as_query():
     response_xml = """<?xml version="1.0"?><response status="Ok"/>"""

{epa_pm_api_wrapper-0.3.1 → epa_pm_api_wrapper-0.3.3}/uv.lock

@@ -149,7 +149,7 @@ wheels = [
 
 [[package]]
 name = "epa-pm-api-wrapper"
-version = "0.3.1"
+version = "0.3.3"
 source = { editable = "." }
 dependencies = [
     { name = "httpx" },

epa_pm_api_wrapper-0.3.1/docs/api.md

@@ -1,3 +0,0 @@
-# API Reference
-
-::: epa_pm_api_wrapper