[Consumption] Fix #33907: az consumption usage list: Preserve meterDetails.totalIncludedQuantity

[Daz (8dazo)]

Related command
az consumption usage list -m
az consumption usage list -a -m
az consumption pricesheet show --include-meter-details

Description
Fixes #33907.

transform_usage_output and pricesheet_show_properties stringified meterDetails.totalIncludedQuantity from pretaxStandardRate, so included quantity was replaced with the listing price.

This restores the pre-AAZ mapping from #27152: each field is converted from its own service value.

Testing Guide
With --include-meter-details, if the service returns totalIncludedQuantity: 100 and pretaxStandardRate: 0.05, CLI output should be:

"meterDetails": {
  "totalIncludedQuantity": "100",
  "pretaxStandardRate": "0.05"
}

--debug already showed the correct REST payload; only the CLI transform was wrong.

History Notes
[Consumption] az consumption usage list/pricesheet show: Preserve meterDetails.totalIncludedQuantity when including meter details

---

This checklist is used to make sure that common guidelines for a pull request are followed.

• The PR title and description has followed the guideline in Submitting Pull Requests.

• I adhere to the Command Guidelines.

• I adhere to the Error Handling Guidelines.

Made with Cursor

[azure-client-tools-bot-prd]

Hi Daz (@8dazo),
Since the current milestone time is less than 7 days, this pr may not catch up with this release.

[azure-pipelines]

Azure Pipelines:
There may be pipelines that require an authorized user to comment /azp run to run.

[Copilot]

Pull request overview

Note

Copilot was unable to run its full agentic suite in this review.

Fixes incorrect serialization of meterDetails.totalIncludedQuantity by reading from the correct source field instead of pretaxStandardRate.

Changes:

• Corrects transform_usage_output to stringify meterDetails.totalIncludedQuantity from totalIncludedQuantity.
• Corrects pricesheet_show_properties to stringify meterDetails.totalIncludedQuantity from totalIncludedQuantity.

---

💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.

[microsoft-github-policy-service]

Thank you for your contribution Daz (@8dazo)! We will review the pull request and get back to you soon.

[Yong Zhang (yonzhan)]

Consumption

[azure-client-tools-agent]

Live test skipped

⏭️ Skipping the live test for this revision because no new test file was added (tests/**/test_*.py).

The live-test pipeline runs only the test files a PR changes, so there is nothing to execute for this commit. This is informational — a regression test is encouraged where it makes sense, but not required. If a test file is added in a later commit, the live test will run automatically.

[azure-client-tools-agent]

✅ All Checks Passed

All CI checks and live tests have passed. This PR looks good!

[azure-client-tools-agent]

Automated Review Summary

All checks are green for this PR.

• Live test (Tester): passed
• CI checks: 23/23 passed, 0 failed, 0 pending

Change: Fixes #33907 — az consumption usage list now preserves meterDetails.totalIncludedQuantity instead of dropping/overwriting it.

No outstanding issues found. Ready for human maintainer merge decision.